• 
      

    Split out the review request view tests into multiple files.

    Review Request #9632 — Created Feb. 12, 2018 and submitted

    Information

    Review Board
    release-3.0.x
    0583a7a...

    Reviewers

    The test_views.py file was getting too large, with all the views it
    was testing. Most of these were intermixed in one big class, almost all
    with outdated function names and some placed wrong.

    This splits out the file into a series of per-view files. The tests
    haven't changed (and many of them are definitely still very outdated),
    with a few exceptions:

    1. The test names have changed to remove any view namespacing (leaving
      that to the class)
    2. The docstrings specify the new class names.
    3. Calls to self._get_context_var(response, varname) are replaced with
      response.context[varname].

    All unit tests pass.

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