• 
      

    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)