flake8
-
reviewboard/webapi/resources/review_request.py (Diff revision 1) Show all issues
Review Request #9496 — Created Jan. 19, 2018 and submitted
When fetching the JSON from the ReviewRequest API using
GET review-requests/<id>/
, the fieldissue_verifying_count
is
missing and should be included.Note that this field already exists in the review request model, and
counts issues which have the statusVERIFYING_RESOLVED
or
VERIFYING_DROPPED
, and these issues are not included in the currently
displayed fields which areissue_dropped_count
,issue_open_count
andissue_resolved_count
.With this fix, we display the field
issue_verifying_count
in the JSON.
Also, a unit test is included to assert that all four issue count fields
are visible.
httpie
, run http :8080/api/review-requests/<id>/
"issue_dropped_count": 0,
"issue_open_count": 1,
"issue_resolved_count": 0,
"issue_verifying_count": 1,
./tests/runtests.py
reviewboard.webapi.tests.test_review_request:
ResourceItemTests.test_get_contains_all_issue_counts
Description | From | Last Updated |
---|---|---|
You'll need to update unit tests for this as well, to ensure this works as expected and never regresses. |
chipx86 | |
The description needs a bit of work. See https://www.reviewboard.org/docs/codebase/dev/writing-good-descriptions/ for what we expect on all the fields. |
chipx86 | |
The bug number should go in the bugs field. |
brennie | |
Can you wrap your description and testing done at 72 characters? |
brennie | |
Summaries shouldn't end in a period. |
brennie | |
E501 line too long (81 > 79 characters) |
reviewbot | |
2.0 is an old release. We'll target this fix for 3.0.3 instead. Can you update this to specify that version? |
chipx86 |
Summary: |
|
|||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Description: |
|
|||||||||||||||||||||||||||||||||||||||||||||
Testing Done: |
|
|||||||||||||||||||||||||||||||||||||||||||||
Commit: |
|
|||||||||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 2 (+10 -2) |
You'll need to update unit tests for this as well, to ensure this works as expected and never regresses.
The description needs a bit of work. See https://www.reviewboard.org/docs/codebase/dev/writing-good-descriptions/ for what we expect on all the fields.
reviewboard/webapi/resources/review_request.py (Diff revision 2) |
---|
2.0 is an old release. We'll target this fix for 3.0.3 instead. Can you update this to specify that version?
Description: |
|
|||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Testing Done: |
|
|||||||||||||||||||||||||||||||||||||||
Commit: |
|
|||||||||||||||||||||||||||||||||||||||
Diff: |
Revision 3 (+28 -2) |
Summary: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|---|
Description: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Testing Done: |
|
||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Bugs: |
|
Description: |
|
---|