Code review comment for lp:~wallyworld/juju-core/joyent-gccgo-test-failure

Revision history for this message
Ian Booth (wallyworld) wrote :

Reviewers: mp+220743_code.launchpad.net,

Message:
Please take a look.

Description:
Trivial joyent test fix for gccgo

Use SameContents not DeepEquals

https://code.launchpad.net/~wallyworld/juju-core/joyent-gccgo-test-failure/+merge/220743

(do not edit description out of merge proposal)

Please review this at https://codereview.appspot.com/97650046/

Affected files (+3, -1 lines):
   A [revision details]
   M provider/joyent/local_test.go

Index: [revision details]
=== added file '[revision details]'
--- [revision details] 2012-01-01 00:00:00 +0000
+++ [revision details] 2012-01-01 00:00:00 +0000
@@ -0,0 +1,2 @@
+Old revision: tarmac-20140523013529-f0gwneum0rsp1vfe
+New revision: <email address hidden>

Index: provider/joyent/local_test.go
=== modified file 'provider/joyent/local_test.go'
--- provider/joyent/local_test.go 2014-05-19 21:42:14 +0000
+++ provider/joyent/local_test.go 2014-05-23 03:55:47 +0000
@@ -422,7 +422,7 @@
   cons := constraints.MustParse("arch=amd64 tags=bar cpu-power=10")
   unsupported, err := validator.Validate(cons)
   c.Assert(err, gc.IsNil)
- c.Assert(unsupported, gc.DeepEquals, []string{"cpu-power", "tags"})
+ c.Assert(unsupported, jc.SameContents, []string{"cpu-power", "tags"})
  }

  func (s *localServerSuite) TestConstraintsValidatorVocab(c *gc.C) {

« Back to merge proposal