Merge lp://staging/~lukasz-czyzykowski/rnr-server/fix-404-in-rnr-client-tests into lp://staging/rnr-server
Proposed by
Łukasz Czyżykowski
| Status: | Merged |
|---|---|
| Approved by: | Michael Nelson |
| Approved revision: | 179 |
| Merged at revision: | 179 |
| Proposed branch: | lp://staging/~lukasz-czyzykowski/rnr-server/fix-404-in-rnr-client-tests |
| Merge into: | lp://staging/rnr-server |
| Diff against target: |
13 lines (+4/-1) 1 file modified
src/reviewsapp/tests/test_rnrclient.py (+4/-1) |
| To merge this branch: | bzr merge lp://staging/~lukasz-czyzykowski/rnr-server/fix-404-in-rnr-client-tests |
| Related bugs: |
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Michael Nelson (community) | Approve | ||
|
Review via email:
|
|||
Commit message
Fix rnr client test which was hitting a 404 page instead instead of proper handler.
Description of the change
Overview
========
This branch fixes an url for one of the rnr client tests. Before it was returning 404 which was fine for the rest of the tests. Also added explicit check for the status code to prevent this situation from happening in the future.
To post a comment you must log in.
Revision history for this message
| Michael Nelson (michael.nelson) | # |
review:
Approve
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
