Merge lp:~reedobrien/charmworld/reindex_test into lp:charmworld
Proposed by
Reed O'Brien
Status: | Merged | ||||
---|---|---|---|---|---|
Approved by: | Reed O'Brien | ||||
Approved revision: | 511 | ||||
Merged at revision: | 510 | ||||
Proposed branch: | lp:~reedobrien/charmworld/reindex_test | ||||
Merge into: | lp:charmworld | ||||
Diff against target: |
28 lines (+2/-6) 1 file modified
charmworld/tests/test_search.py (+2/-6) |
||||
To merge this branch: | bzr merge lp:~reedobrien/charmworld/reindex_test | ||||
Related bugs: |
|
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Juju Gui Bot | continuous-integration | Approve | |
Brad Crittenden (community) | code | Approve | |
Richard Harding | Pending | ||
Review via email: mp+219569@code.launchpad.net |
Commit message
Fixes a destructive test in Bug #1317567.
Description of the change
Fixes the test that was destroying developer's charms indexes in ES.
Uses the base test class' use_index_client method to setup the test index instead.
To post a comment you must log in.
Looks good, thanks. See inline comments.