Conditionally return total_results from list resources.

Review Request #13274 — Created Sept. 15, 2023 and submitted — Latest diff uploaded

Information

RBTools
release-4.x

Reviewers

We have some resources that behave like lists but don't contain a
total_results field. When we access this, we crash, because we assume
this will always be present.

This change makes that field optional, setting it to None if not
found.

Unit tests pass.

Tested this with the watched review request resource, which exhibits
this failure.

Diff Revision 1 (Latest)

Commits

First Last Summary ID Author
Conditionally return total_results from list resources.
We have some resources that behave like lists but don't contain a `total_results` field. When we access this, we crash, because we assume this will always be present. This change makes that field optional, setting it to `None` if not found.
562e55f3f6f0a490dc12b4752f70cd8addc61bc2 Christian Hammond
rbtools/api/resource.py
rbtools/api/tests/test_resource.py
Loading...