This change has been marked as completed.

Describe the completed change (optional):

Summary

Split out RepositoryBranches and RepositoryCommits webapi tests.

Review Request #4517 — Created Aug. 30, 2013 and submitted 11 years, 9 months ago

Information

Owner

chipx86*

Repository

Review Board

Branch

master

Bugs

Depends On

Reviewers

Groups

reviewboard

People

Description*

Split out RepositoryBranches and RepositoryCommits webapi tests.

This moves out the RepositoryBranchesResourceTests and
RepositoryCommitsResourceTests out into their own files.

Testing Done

Unit tests pass.

Issues

Description	From	Last Updated
'Tool' imported but unused	reviewbot	11 years, 9 months ago
'REPO_NOT_IMPLEMENTED' imported but unused	reviewbot	11 years, 9 months ago
'DefaultReviewerResourceTests' imported but unused	reviewbot	11 years, 9 months ago
'RepositoryBranchesResourceTests' imported but unused	reviewbot	11 years, 9 months ago
'RepositoryCommitsResourceTests' imported but unused	reviewbot	11 years, 9 months ago
'RepositoryInfoResourceTests' imported but unused	reviewbot	11 years, 9 months ago
'RootResourceTests' imported but unused	reviewbot	11 years, 9 months ago
'ServerInfoResourceTests' imported but unused	reviewbot	11 years, 9 months ago
'SessionResourceTests' imported but unused	reviewbot	11 years, 9 months ago
'ValidateDiffResourceTests' imported but unused	reviewbot	11 years, 9 months ago
Col: 80 E501 line too long (107 > 79 characters)	reviewbot	11 years, 9 months ago
Col: 80 E501 line too long (106 > 79 characters)	reviewbot	11 years, 9 months ago
Col: 80 E501 line too long (96 > 79 characters)	reviewbot	11 years, 9 months ago
Col: 80 E501 line too long (82 > 79 characters)	reviewbot	11 years, 9 months ago
Col: 80 E501 line too long (106 > 79 characters)	reviewbot	11 years, 9 months ago
Col: 80 E501 line too long (105 > 79 characters)	reviewbot	11 years, 9 months ago
There are no open issues

This is a review from Review Bot.
Tool: PEP8 Style Checker
Processed Files:
reviewboard/webapi/tests/repository_commits.py
reviewboard/webapi/tests/repository_branches.py
reviewboard/webapi/tests/init.py
Ignored Files:

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

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

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

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

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

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

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

This is a review from Review Bot.
Tool: Pyflakes
Processed Files:
reviewboard/webapi/tests/repository_commits.py
reviewboard/webapi/tests/repository_branches.py
reviewboard/webapi/tests/init.py
Ignored Files:

reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been resolved. Show all issues
```
'Tool' imported but unused
```
reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been resolved. Show all issues
```
'REPO_NOT_IMPLEMENTED' imported but unused
```
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
```
'RepositoryBranchesResourceTests' imported but unused
```
reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been dropped. Show all issues
```
'RepositoryCommitsResourceTests' imported but unused
```
reviewboard/webapi/tests/__init__.py (Diff revision 1)
The issue has been dropped. Show all issues
```
'RepositoryInfoResourceTests' 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
```

Status:: Completed