Checks run (2 succeeded)
flake8
passed.
JSHint
passed.
Add typing and missing docs for the main API unit tests mixins.
Review Request #13375 — Created Oct. 24, 2023 and submitted
The code for the API unit tests largely preceded much of our modern
codebase standards. It lacked docs, certainly lacked typing, and has a
lot of design choices that limit future expansion.In preparation for new API work that will be taking place, this change
cleans up this code, adding typing and docs and assertions to ensure
that we have a better foundation for building more manageable test
suites going forward.
Unit tests pass.
| Summary | ID |
|---|---|
| 5d24b73a13b04b45ffe8321ab68afa604682ab12 |
| Description | From | Last Updated |
|---|---|---|
|
Typo: "use" -> "user" |
maubin
|
|
|
Missing a trailing : after meth |
maubin
|
|
|
Should be 3-tuple. |
maubin
|
|
|
Should be 4-tuple. |
maubin
|
|
|
Should be 4-tuple. |
maubin
|
|
|
Should be 5-tuple. |
maubin
|
|
|
"post" instead of "delete". |
maubin
|
|
|
Missing a trailing : after meth |
maubin
|
|
|
Should be 6-tuple. |
maubin
|
|
|
Should say "put" instead of "delete". |
maubin
|
Review request changed
- Change Summary:
-
Fixed several issues in the docs.
- Commits:
-
Summary ID 4235a67ec1a1985a9157330e1760d391380c3458 5d24b73a13b04b45ffe8321ab68afa604682ab12 - Diff:
-
Revision 2 (+1904 -308)
Checks run (2 succeeded)
flake8
passed.
JSHint
passed.