Split reviewboard.site.tests into multiple test modules.

Review Request #12300 — Created May 24, 2022 and submitted

chipx86
Review Board
release-5.0.x
reviewboard

reviewboard.site.tests was getting a bit big, covering many areas of
Local Sites. With some new tests coming, it seemed time to split this
up.

This creates a series of new test cases for different areas of the app,
moving tests into them. The tests themselves have not changed, aside
from the removal of the old import reviewboard.site.templatetags hack
for our {% url %} replacement, which is not needed with modern Django
template configuration.

One test for managing OAuth2 Application access when Local Site
membership changes has been moved into reviewboard/oauth/tests.py.

Unit tests pass.

Summary
Split reviewboard.site.tests into multiple test modules.
Description From Last Updated

We can now just use super().setUp() for most of these.

daviddavid

Same here.

daviddavid

And here.

daviddavid

Module docstring.

daviddavid
david
  1. 
      
  2. We can now just use super().setUp() for most of these.

  3. Same here.

  4. 
      
chipx86
david
  1. 
      
  2. Module docstring.

  3. 
      
chipx86
Review request changed

Status: Closed (submitted)

Change Summary:

Pushed to release-5.0.x (348e58e)
Loading...