kat-kan / kawqa-gad-playwright

4 stars 1 forks source link

[ISSUE#23]PUT articles/{id}- add tests #111

Closed Justyna-KO closed 1 month ago

Justyna-KO commented 4 months ago

Resolves #23

Scope of changes:

Tests PUT articles/{id} for responses:

How to use it:

Any other comments?

The test for the 404 response is missing, any ideas on how to get 404 in response for this endpoint?

Before submitting Pull Request, I've ensured that:

kat-kan commented 4 months ago

As for 404 - there is a 404 PATCH article test here https://github.com/kat-kan/kawqa-gad-playwright/pull/47/files maybe it can be reproduced this way with PUT as well?

Slawomir-DKl commented 4 months ago

@Justyna-KO The latest GAD version (2.7.2) brings another logic which is not yet tested:

using PUT with empty body will return error 422 (comments, users, articles)

If you think about developing the tests further, it would be quite nice extension