Skip to content
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

feat: mocks upload files in tests #139

Merged
merged 1 commit into from
Aug 30, 2024
Merged

Conversation

ReidyT
Copy link
Contributor

@ReidyT ReidyT commented Aug 16, 2024

Upload the image before the tests.
This PR propose a solution to the PR #136. The apps-query-client from graasp/graasp-apps-query-client# should be used, because MSW is mocking the upload and download of app settings files.

@ReidyT ReidyT requested review from spaenleh and LinaYahya August 16, 2024 14:46
@ReidyT ReidyT self-assigned this Aug 16, 2024
@ReidyT ReidyT force-pushed the 132-mocks-upload-files-in-tests branch from da4df39 to 7a10781 Compare August 16, 2024 14:52
@ReidyT ReidyT requested a review from pyphilia August 26, 2024 08:22
Copy link
Contributor

@LinaYahya LinaYahya left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Great work @ReidyT

@ReidyT ReidyT force-pushed the 132-mocks-upload-files-in-tests branch from 97c425e to b130b65 Compare August 28, 2024 08:24
@ReidyT ReidyT force-pushed the 132-mocks-upload-files-in-tests branch 2 times, most recently from 9156108 to 7e864fc Compare August 28, 2024 14:35
@ReidyT ReidyT marked this pull request as ready for review August 28, 2024 14:36
@ReidyT ReidyT force-pushed the 132-mocks-upload-files-in-tests branch 2 times, most recently from 450036d to 3998222 Compare August 30, 2024 09:15
- feat(test): setup uploaded image before the test

Co-Authored-By: ReidyT <thibault.reidy@hotmail.ch>
@ReidyT ReidyT force-pushed the 132-mocks-upload-files-in-tests branch from 3998222 to d808725 Compare August 30, 2024 11:50
@ReidyT
Copy link
Contributor Author

ReidyT commented Aug 30, 2024

There is an issue in the builder view where we have to click twice on the "Add the labels" step. I created a separated issue for that, because it was not introduced in this PR. #167

@ReidyT ReidyT merged commit 84dbc99 into main Aug 30, 2024
3 checks passed
@ReidyT ReidyT deleted the 132-mocks-upload-files-in-tests branch August 30, 2024 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants