-
Notifications
You must be signed in to change notification settings - Fork 32
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Check empty file uploads, tests for the same and tests for local uploads on file interceptor #178
Conversation
Rebase empty
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the 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? TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (invoked as PR comments)
Additionally, you can add CodeRabbit Configuration File (
|
it('should throw error when process.env.STORAGE_ENDPOINT is not defined or empty', async () => { | ||
// mocking env vars | ||
const OLD_ENV = process.env; | ||
|
||
}); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
is this expected.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
seems to have been a mix up, will fix it asap
Merging this thanks for the effort @gnaaruag |
Hello @techsavvyash |
Hey there, |
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 in
sample/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 to
sample/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