Add typing and missing docs for the main API unit tests mixins.

Review Request #13375 — Created Oct. 24, 2023 and submitted — Latest diff uploaded

Information

Review Board
release-5.0.x

Reviewers

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.

Diff Revision 1

This is not the most recent revision of the diff. The latest diff is revision 2. See what's changed.

orig
1
2

Commits

First Last Summary ID Author
Add typing and missing docs for the main API unit tests mixins.
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.
4235a67ec1a1985a9157330e1760d391380c3458 Christian Hammond
reviewboard/webapi/tests/mixins.py
Loading...