Summary

Split reviewboard.extensions.tests into multiple test modules.

Review Request #11637 — Created May 29, 2021 and submitted June 2, 2021, 6:48 p.m.

Information

Owner

chipx86

Repository

Review Board

Branch

release-3.0.x

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

This change simply splits the tests.py into a series of test_*.py
files. No logic or documentation has been changed, though module-level
docstrings were added. Cleanup would happen in another change.

Testing Done

Unit tests passed.

Commits

Summary	ID
Split reviewboard.extensions.tests into multiple test modules. This test suite had grown too big to manage, and a lot of the testing patterns being used are far from ideal. With more work on the horizon, we want to split this up and make it easier to maintain a test suite. This change simply splits the `tests.py` into a series of `test_*.py` files. No logic or documentation has been changed. That would come in a future change.	f90130f52c154fbbfa12e7c8dccb5d4c12c71fcb

Summary

Split reviewboard.extensions.tests into multiple test modules.

This test suite had grown too big to manage, and a lot of the testing patterns being used are far from ideal. With more work on the horizon, we want to split this up and make it easier to maintain a test suite. This change simply splits the `tests.py` into a series of `test_*.py` files. No logic or documentation has been changed. That would come in a future change.

f90130f52c154fbbfa12e7c8dccb5d4c12c71fcb

Issues

Description	From	Last Updated
E501 line too long (81 > 79 characters)	reviewbot	May 29, 2021, 4:49 p.m.
E501 line too long (81 > 79 characters)	reviewbot	May 29, 2021, 4:49 p.m.
W391 blank line at end of file	reviewbot	May 29, 2021, 4:50 p.m.

flake8 failed.

JSHint passed.

flake8

reviewboard/extensions/tests/test_action_hooks.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (81 > 79 characters)
```
reviewboard/extensions/tests/test_action_hooks.py (Diff revision 1)
The issue has been resolved. Show all issues
```
E501 line too long (81 > 79 characters)
```
reviewboard/extensions/tests/test_avatar_service_hook.py (Diff revision 1)
The issue has been resolved. Show all issues
```
W391 blank line at end of file
```

Ship it!

```
Ship It!
```

Status: Closed (submitted)

Change Summary:

Pushed to release-3.0.x (bd54043)