Expand the Local Site query checks in the Users datagrid tests.

Review Request #13360 — 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 Users 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.

There's also a little bit of base class test setup, teardown, and
utilities that are introduced that will be needed for the test query
work.

Unit tests pass.

Changes between revision 1 and 2

orig
1
2

Commits

Summary ID Author
Expand the Local Site query checks in the Users datagrid tests.
This splits most of the unit tests in the Users 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.
bfade44c21c9d4b22bc81e02b83934a8a2ff4225 Christian Hammond
Expand the Local Site query checks in the Users datagrid tests.
This splits most of the unit tests in the Users 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. There's also a little bit of base class test setup, teardown, and utilities that are introduced that will be needed for the test query work.
2e4898394d70918b7e9e159387a32096570bdcbf Christian Hammond
reviewboard/datagrids/tests/base.py
reviewboard/datagrids/tests/test_dashboard.py
Loading...