Split the datagrid unit tests into per-suite modules.

Review Request #13359 — Created Oct. 22, 2023 and submitted

Information

Review Board
release-5.0.x

Reviewers

The reviewboard.datagrids unit tests were all contained in a single
large tests.py. Some work is being done to audit and optimize queries
for the datagrid, which will significantly grow this test suite.

To prepare for that work, this change splits up the test suite into a
file for each tested datagrid and for each tested column.

No code changes have been made to the test suites, with the exception
that two suites were merged together, as they were testing the same
datagrid.

Unit tests pass.

Summary ID
Split the datagrid unit tests into per-suite modules.
The `reviewboard.datagrids` unit tests were all contained in a single large `tests.py`. Some work is being done to audit and optimize queries for the datagrid, which will significantly grow this test suite. To prepare for that work, this change splits up the test suite into a file for each tested datagrid and for each tested column. No code changes have been made to the test suites.
09b82e1e6812fca050a793ef344dfada16693a03
Description From Last Updated

redefinition of unused 'ReviewRequest' from line 20 Column: 1 Error code: F811

reviewbotreviewbot

'kgb' imported but unused Column: 1 Error code: F401

reviewbotreviewbot

'django.core.cache.cache' imported but unused Column: 1 Error code: F401

reviewbotreviewbot
Checks run (1 failed, 1 succeeded)
flake8 failed.
JSHint passed.

flake8

chipx86
david
  1. Ship It!
  2. 
      
maubin
  1. Ship It!
  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-5.0.x (ab91723)