Skip to content

Commit

Permalink
Remove an unnecessary await in tests.
Browse files Browse the repository at this point in the history
  • Loading branch information
jaydenseric committed Oct 7, 2024
1 parent 1135035 commit e7a051c
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 1 deletion.
1 change: 1 addition & 0 deletions changelog.md
Original file line number Diff line number Diff line change
Expand Up @@ -42,6 +42,7 @@
- Removed the Node.js CLI option `--unhandled-rejections=throw` in the package script `tests` as it’s now the default for all supported Node.js versions.
- Avoid hardcoding a default value in the type `FileUploadCreateReadStreamOptions` property `highWaterMark` description and use the function `getDefaultHighWaterMark` from `node:stream` in tests.
- Replaced the test helper class `Deferred` with polyfilled `Promise.withResolvers`.
- Removed an unnecessary `await` in tests.
- Omit unused catch bindings in the function `processRequest`.
- Corrected the JSDoc type `FileUploadCreateReadStreamOptions` in the module `processRequest.mjs`.
- Avoid using `return` in the middleware.
Expand Down
2 changes: 1 addition & 1 deletion processRequest.test.mjs
Original file line number Diff line number Diff line change
Expand Up @@ -686,7 +686,7 @@ describe(
const { createReadStream } =
await operation.variables.files[0].promise;

await throws(
throws(
() => {
createReadStream();
},
Expand Down

0 comments on commit e7a051c

Please sign in to comment.