Merge lp://staging/~thisfred/ubuntuone-couch/add-tests into lp://staging/~thisfred/ubuntuone-couch/trunk
Proposed by
Eric Casteleijn
| Status: | Merged | ||||||||
|---|---|---|---|---|---|---|---|---|---|
| Approved by: | Eric Casteleijn | ||||||||
| Approved revision: | 14 | ||||||||
| Merged at revision: | 2 | ||||||||
| Proposed branch: | lp://staging/~thisfred/ubuntuone-couch/add-tests | ||||||||
| Merge into: | lp://staging/~thisfred/ubuntuone-couch/trunk | ||||||||
| Diff against target: |
614 lines (+391/-181) 3 files modified
bin/u1couch-query (+32/-181) tests/test_u1couchquery.py (+164/-0) u1couch/query.py (+195/-0) |
||||||||
| To merge this branch: | bzr merge lp://staging/~thisfred/ubuntuone-couch/add-tests | ||||||||
| Related bugs: |
|
| Reviewer | Review Type | Date Requested | Status |
|---|---|---|---|
| Natalia Bidart (community) | Approve | ||
| Alejandro J. Cura (community) | Approve | ||
| Eric Casteleijn | Pending | ||
|
Review via email:
|
|||
Commit message
Fixed and refactored u1couch-query and added tests
Description of the change
Fixed and refactored u1couch-query and added tests
To post a comment you must log in.
Preview Diff
[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
Great to see this being tested and tidied up. Good work!
I've added bug #720917 so the request method returns only json or Exceptions, and thisfred agreed to work on that on a subsequent branch.