Reduce docstring line lengths for webapi unit tests.
Review Request #4585 — Created Sept. 17, 2013 and submitted
Reduce docstring line lengths for webapi unit tests.
An annoying thing about submitting webapi unit test code for review is
that ReviewBot would be very sad about the docstring line lengths. These
were long because unit tests, by default, only show the first line.
However, that's under our control, it turns out.Now, the new base TestCase will grab the summary, even if it's wrapped
across lines, and turn it into one line.All the docstrings have been updated in the webapi tests for this. The
rules I used for formatting them is as follows:1) If it can fit on one line, then keep it on one line.
2) If it needs to wrap, wrap the details part after "API" (such as
"with ...") on the next line.
3) If it still needs to wrap, wrap the "METHOD URL API" part onto its
own line, keeping it together as much as possible.
4) If it still needs to wrap, break that part up as needed.Some URLs are long enough by themselves where they will still trigger
warnings, but this is limited to only a few instances now.
Unit tests pass, and look correct.
| Description | From | Last Updated |
|---|---|---|
|
'TestTool' imported but unused |
 reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (85 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
|
|
Col: 80 E501 line too long (80 > 79 characters) |
reviewbot
|
-
This is a review from Review Bot.
Tool: Pyflakes
Processed Files:
reviewboard/webapi/tests/test_review_reply_screenshot_comment.py
reviewboard/webapi/tests/test_review_screenshot_comment.py
reviewboard/webapi/tests/test_file_diff_comment.py
reviewboard/webapi/tests/test_watched_review_request.py
reviewboard/webapi/tests/test_review.py
reviewboard/webapi/tests/test_default_reviewer.py
reviewboard/webapi/tests/test_draft_review_screenshot_comment.py
reviewboard/webapi/tests/test_screenshot.py
reviewboard/webapi/tests/test_repository_commits.py
reviewboard/webapi/tests/test_review_comment.py
reviewboard/webapi/tests/test_review_group.py
reviewboard/webapi/tests/test_file_attachment_draft.py
reviewboard/webapi/tests/test_change.py
reviewboard/webapi/tests/test_review_reply.py
reviewboard/webapi/tests/test_file_attachment_comment.py
reviewboard/webapi/tests/test_server_info.py
reviewboard/webapi/tests/test_repository_branches.py
reviewboard/webapi/tests/test_user.py
reviewboard/testing/testcase.py
reviewboard/webapi/tests/test_screenshot_comment.py
reviewboard/webapi/tests/test_session.py
reviewboard/webapi/tests/test_watched_review_group.py
reviewboard/webapi/tests/test_review_request.py
reviewboard/webapi/tests/test_review_request_draft.py
reviewboard/webapi/tests/test_draft_review_file_attachment_comment.py
reviewboard/webapi/tests/test_diff.py
reviewboard/webapi/tests/test_repository.py
reviewboard/webapi/tests/test_validate_diff.py
reviewboard/webapi/tests/test_file_attachment.py
reviewboard/webapi/tests/test_review_reply_file_attachment_comment.py
reviewboard/webapi/tests/test_review_group_user.py
reviewboard/webapi/tests/test_repository_info.py
reviewboard/webapi/tests/test_review_reply_diff_comment.py
reviewboard/webapi/tests/test_screenshot_draft.py
Ignored Files: -
