Closed gnaaruag closed 3 months ago
[!IMPORTANT]
Review skipped
Auto reviews are disabled on base/target branches other than the default branch.
Please check the settings in the CodeRabbit UI or the
.coderabbit.yaml
file in this repository. To trigger a single review, invoke the@coderabbitai review
command.You can disable this status message by setting the
reviews.review_status
tofalse
in the CodeRabbit configuration file.
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Merging this thanks for the effort @gnaaruag
Hello @techsavvyash Will you pls close the corresponding issues as well if the required PR is merged? Thanks.
Hey there, Thanks for your contribution. As part of Augtoberfest, we want to reward you for your contributions, so pls fill out this form by 19th Sept. Thanks.
Checklist
Make sure you have
@nestjs/swagger
decorators wherever requiredDescription
Tests in #50
This PR adds tests for local uploads described in #50 to
file-upload.interceptor.spec.ts
also adds e2e tests in the file upload sample app insample/04-file-upload/test/app.e2e-spec.ts
fix for issue #156
file-upload.controller.ts
inpackages/common/src
process.env.STORAGE_MODE
file-upload.interceptor
the linerequest.raw = request.raw || request;
allows for valid request body to pass throughSteps to test #50
Navigate to
packages/common/test
and runnpm run test interceptor/file-upload.interceptor.spec.ts
Navigate tosample/04-file-upload
and runnpm run test:e2e
Steps to test #156
to test manually create a post request to
/file/upload-file
test with empty file and non empty file via
multipart/form-data
to run automated test navigate to
samples/04-file-upload
and runnpm run test:e2e