Split out the review request view tests into multiple files.

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

Christian Hammond
Review Board

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

All unit tests pass.

David Trowbridge
  1. Ship It!
Christian Hammond
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (4659634)