Expand the Local Site query checks in the All Review Requests datagrid tests.

Review Request #13361 — Created Oct. 22, 2023 and submitted — Latest diff uploaded

Information

Review Board
release-5.0.x

Reviewers

This splits most of the unit tests in the All Review Requests datagrid
test suite into a global site test, a Local Site test, and a global
site + Local Sites in the database test.

This gives us thorough query coverage, which unveils a number of
sub-optimal queries that will need to be addressed in upcoming changes.

The queries are intentionally verbose right now, with no query-building
being shared across tests, and with the query-by-query breakdowns
removed. This is intentional, as upcoming work is expected to heavily
impact these queries.

Unit tests pass.

Changes between revision 1 and 2

orig
1
2

Commits

Summary ID Author
Expand the Local Site query checks in the All Review Requests datagrid tests.
This splits most of the unit tests in the All Review Requests datagrid test suite into a global site test, a Local Site test, and a global site + Local Sites in the database test. This gives us thorough query coverage, which unveils a number of sub-optimal queries that will need to be addressed in upcoming changes. The queries are intentionally verbose right now, with no query-building being shared across tests, and with the query-by-query breakdowns removed. This is intentional, as upcoming work is expected to heavily impact these queries.
8639b88fd9a93fd9f3a22b18e997b924e4cf0a26 Christian Hammond
Expand the Local Site query checks in the All Review Requests datagrid tests.
This splits most of the unit tests in the All Review Requests datagrid test suite into a global site test, a Local Site test, and a global site + Local Sites in the database test. This gives us thorough query coverage, which unveils a number of sub-optimal queries that will need to be addressed in upcoming changes. The queries are intentionally verbose right now, with no query-building being shared across tests, and with the query-by-query breakdowns removed. This is intentional, as upcoming work is expected to heavily impact these queries.
b8ef11b6584366f94b216674e4e8f8fa3b224379 Christian Hammond
reviewboard/datagrids/tests/test_all_review_requests.py
Loading...