Merge lp://qastaging/~divmod-dev/divmod.org/829872-createnew-type-check into lp://qastaging/divmod.org
Proposed by
Tristan Seligmann
Status: | Superseded | ||||
---|---|---|---|---|---|
Proposed branch: | lp://qastaging/~divmod-dev/divmod.org/829872-createnew-type-check | ||||
Merge into: | lp://qastaging/divmod.org | ||||
Prerequisite: | lp://qastaging/~divmod-dev/divmod.org/829869-explicit-type-dropping | ||||
Diff against target: |
202 lines (+40/-14) 3 files modified
Axiom/axiom/substore.py (+9/-0) Axiom/axiom/test/test_batch.py (+7/-5) Axiom/axiom/test/test_substore.py (+24/-9) |
||||
To merge this branch: | bzr merge lp://qastaging/~divmod-dev/divmod.org/829872-createnew-type-check | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Jean-Paul Calderone | Approve | ||
Jonathan Jacobs | Approve | ||
Review via email: mp+72277@code.qastaging.launchpad.net |
This proposal has been superseded by a proposal from 2013-07-07.
To post a comment you must log in.
1. I think this should either be %r or you should mention the type to make figuring out what you're *actually* passing a little easier, __str__ can be deceiving:
10 + 'Received %s instead of a sequence' % (pathSegments,))
2. Can you assert a little bit more about the exception in test_createNewS tringPath?
Merge when you're happy with those.