Removed dependence of slice order in test. #5318

Merged
merged 2 commits into from Apr 29, 2016

Conversation

Projects
None yet
2 participants
Contributor

kat-co commented Apr 29, 2016

When given a map[interface{}][]interface{} jc.DeepEquals does not compare the slice in an order independent manner.

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

kat-co added some commits Apr 29, 2016

Removed dependence of slice order in test.
When given a map[interface{}][]interface{} jc.DeepEquals does not compare the slice in an order independent manner.
Contributor

kat-co commented Apr 29, 2016

$$fixes-1576705$$

Contributor

jujubot commented Apr 29, 2016

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

Contributor

jujubot commented Apr 29, 2016

Build failed: Does not match ['fixes-1576509', 'fixes-1576527', 'fixes-1576313', 'fixes-1576368', 'fixes-1576021']
build url: http://juju-ci.vapour.ws:8080/job/github-merge-juju/7621

Contributor

kat-co commented Apr 29, 2016

$$fixes-1576705$$

Contributor

jujubot commented Apr 29, 2016

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

@jujubot jujubot merged commit 1497e01 into juju:master Apr 29, 2016

@kat-co kat-co deleted the kat-co:fix-1576705-order-in-test branch Apr 29, 2016

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