Summary

Split our four more classes from webapi tests.

Review Request #4514 — Created Aug. 30, 2013 and submitted Aug. 30, 2013, 3:10 p.m.

Information

Owner

chipx86

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description

Split our four more classes from webapi tests.

This splits DefaultReviewerResourceTests, RootResourceTests,
ServerInfoResourceTests, and SessionResourceTests into new files.

Testing Done

Unit tests pass.

Issues

Description	From	Last Updated
'DefaultReviewerResourceTests' imported but unused	reviewbot	Aug. 30, 2013, 2:19 p.m.
'RootResourceTests' imported but unused	reviewbot	Aug. 30, 2013, 2:19 p.m.
'ServerInfoResourceTests' imported but unused	reviewbot	Aug. 30, 2013, 2:19 p.m.
'SessionResourceTests' imported but unused	reviewbot	Aug. 30, 2013, 2:19 p.m.
'ValidateDiffResourceTests' imported but unused	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (82 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (99 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (80 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (81 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (86 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (82 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (103 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (84 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (85 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (81 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (90 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (98 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (103 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (84 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (89 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (81 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (106 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (85 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.
Col: 80 E501 line too long (88 > 79 characters)	reviewbot	Aug. 30, 2013, 2:19 p.m.

This is a review from Review Bot.
Tool: PEP8 Style Checker
Processed Files:
reviewboard/webapi/tests/server_info.py
reviewboard/webapi/tests/default_reviewer.py
reviewboard/webapi/tests/root.py
reviewboard/webapi/tests/session.py
reviewboard/webapi/tests/init.py
Ignored Files:

reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (82 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (99 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (80 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (81 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (86 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (82 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (103 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (84 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (85 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (81 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (90 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (98 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (103 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (84 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (89 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (81 > 79 characters)
```
reviewboard/webapi/tests/default_reviewer.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (106 > 79 characters)
```
reviewboard/webapi/tests/server_info.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (85 > 79 characters)
```
reviewboard/webapi/tests/session.py (Diff revision 1)
The issue has been dropped. Show all issues
```
Col: 80

 E501 line too long (88 > 79 characters)
```

This is a review from Review Bot.
Tool: Pyflakes
Processed Files:
reviewboard/webapi/tests/server_info.py
reviewboard/webapi/tests/default_reviewer.py
reviewboard/webapi/tests/root.py
reviewboard/webapi/tests/session.py
reviewboard/webapi/tests/init.py
Ignored Files:

reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been dropped. Show all issues
```
'DefaultReviewerResourceTests' imported but unused
```
reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been dropped. Show all issues
```
'RootResourceTests' imported but unused
```
reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been dropped. Show all issues
```
'ServerInfoResourceTests' imported but unused
```
reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been dropped. Show all issues
```
'SessionResourceTests' imported but unused
```
reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been dropped. Show all issues
```
'ValidateDiffResourceTests' imported but unused
```

Review Board 7.0 alpha 0 (dev)

Split our four more classes from webapi tests.

Information

Reviewers

Status: Closed (submitted)