Split reviewboard.site.tests into multiple test modules.

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

Information

Review Board
release-5.0.x

Reviewers

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 ID
Split reviewboard.site.tests into multiple test modules.
`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`.
9ba156fee3abfb650d056eecee447e7b7ca70258
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...