Split the reviewboard.search unit tests into multiple files.

Review Request #11346 — Created Dec. 23, 2020 and submitted

Information

Review Board
release-4.0.x

Reviewers

reviewboard.search.tests was getting a bit hard to manage, and new
tests for new classes will soon be written. To prepare for this, this
change splits the test suite into 3 new files.

The test code is largely unchanged, with a few exceptions:

  1. Missing/broken docstrings were added/corrected.
  2. Some Django 1.6-specific code has been removed.
  3. A KGB call has been updated for modern best practices.

Unit tests pass on Python 2.7 and 3.6 through 3.9.

Ensured I didn't have a stale tests.pyc or __pycache__ around.

Summary ID
Split the reviewboard.search unit tests into multiple files.
`reviewboard.search.tests` was getting a bit hard to manage, and new tests for new classes will soon be written. To prepare for this, this change splits the test suite into 3 new files. The test code is largely unchanged, with a few exceptions: 1. Missing/broken docstrings were added/corrected. 2. Some Django 1.6-specific code has been removed. 3. A KGB call has been updated for modern best practices.
29f37b44a9ea674acb4d3f441b02295973c205c4
Description From Last Updated

W391 blank line at end of file

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

flake8

david
  1. Once Review Bot is happy, this looks good.

  2. 
      
chipx86
Review request changed
Status:
Completed
Change Summary:
Pushed to release-4.0.x (d41b0e7)