-
Notifications
You must be signed in to change notification settings - Fork 189
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
Refactor local API tests of JSON responses #3179
Comments
@ScharfViktor I am thinking about this, and will work on a proposed core solution for issue owncloud/core#39796 and you can review. Once we agree on what is a good refactoring, then someone can also implement here in oCIS local API tests. |
We have used to check the response of the json using a library in TODO:
|
All test suite related to the json schema assertion has been replaced and added. So closing this issue . |
There are scenarios that test the JSON response to various spaces API requests. For example:
Those drill-down into the JSON response structure and have specific numeric array indices mentioned. (see the "0" and "1" mentioned in the above) Those are brittle because they depend on the exact order of entries in the JSON response.
refactor the code so that it can look for and find the data anywhere in the numeric-indexed arrays
implement a nicer way of referring to the nested array structure than having those "@@@" in the scenario steps.
Note: (2) can also be done in the core API tests, they are where the "@@@" thing came from. See owncloud/core#39796
The text was updated successfully, but these errors were encountered: