Split reviewboard.site.tests into multiple test modules.

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

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.

Diff Revision 2 (Latest)

orig
1
2

Commits

First Last Summary ID Author
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 Christian Hammond
reviewboard/oauth/tests.py
reviewboard/site/tests/__init__.py
reviewboard/site/tests/test_check_local_site_access_view_mixin.py
reviewboard/site/tests/test_context_processors.py
reviewboard/site/tests/test_local_site.py
reviewboard/site/tests.py
reviewboard/site/tests/test_local_site_middleware.py
reviewboard/site/tests/test_template_tags.py
reviewboard/site/tests/test_urlresolvers.py
Loading...