cmd/juju/commands: fix unreliable test #6119

Merged
merged 1 commit into from Aug 31, 2016

Conversation

Projects
None yet
2 participants
Owner

rogpeppe commented Aug 30, 2016

The order of the auth types passed to Bootstrap is not defined
because it is populated from a map.
Sort before testing that value so that the tests fail less often.

(Review request: http://reviews.vapour.ws/r/5558/)

cmd/juju/commands: fix unreliable test
The order of the auth types passed to Bootstrap is not defined
because it is populated from a map.
Sort before testing that value so that the tests fail less often.
Owner

rogpeppe commented Aug 30, 2016

$$merge$$

Contributor

jujubot commented Aug 30, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 30, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8981

Owner

rogpeppe commented Aug 30, 2016

$$merge$$

Contributor

jujubot commented Aug 30, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

Contributor

jujubot commented Aug 30, 2016

Build failed: Tests failed
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/8985

Owner

rogpeppe commented Aug 30, 2016

$$merge $$

Owner

rogpeppe commented Aug 31, 2016

$$merge$$

Contributor

jujubot commented Aug 31, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 54b153e into juju:master Aug 31, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment