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.

Sign up for GitHub

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

Jump to bottom

Add system tests for the image upload endpoint #69

Open
matheusgomes28 opened this issue Mar 28, 2024 · 0 comments
Open

Add system tests for the image upload endpoint #69

matheusgomes28 opened this issue Mar 28, 2024 · 0 comments
Assignees
@matheusgomes28
Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed

Comments

@matheusgomes28
Copy link
Owner

After #64 , the coverage went down due to not testing certain endpoints on e2e tests:

  • Database image get.
  • Database image delete.
  • Database image add.

These can be added under the tests/system_tests/admin_app where we test endpoints.
@matheusgomes28 matheusgomes28 added enhancement New feature or request help wanted Extra attention is needed good first issue Good for newcomers labels Mar 28, 2024
@matheusgomes28 matheusgomes28 mentioned this issue Mar 28, 2024
Merged
@matheusgomes28 matheusgomes28 self-assigned this Mar 29, 2024
This was referenced Mar 29, 2024
Closed
Merged
matheusgomes28 added a commit that referenced this issue Mar 29, 2024
@matheusgomes28
Image System Tests, Fix & Refactor (#72)
e28a20d 
Implements #69 as well as some minor refactoring of functions to be
reused by system tests and the unit tests.

Bugfix: we were not previously returning the `uuid` field for the `GET
images/{id}` request as the marshalling field name differed from the
value of the response.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@matheusgomes28 matheusgomes28

Labels
enhancement New feature or request good first issue Good for newcomers help wanted Extra attention is needed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@matheusgomes28