state/api: implement APIHostPorts methods

state/api/machiner/machiner_test.go

 // Copyright 2012, 2013 Canonical Ltd.
 // Licensed under the AGPLv3, see LICENCE file for details.
 
 package machiner_test
 
 import (
         stdtesting "testing"
 
         jc "github.com/juju/testing/checkers"
         gc "launchpad.net/gocheck"
@@ skipping 23 matching lines @@
 
         machiner *machiner.State
 }
 
 var _ = gc.Suite(&machinerSuite{})
 
 func (s *machinerSuite) SetUpTest(c *gc.C) {
         s.JujuConnSuite.SetUpTest(c)
         m, err := s.State.AddMachine("quantal", state.JobManageEnviron)
         c.Assert(err, gc.IsNil)
-        m.SetAddresses(instance.NewAddresses([]string{"127.0.0.1"}))
+        err = m.SetAddresses(instance.NewAddresses([]string{"127.0.0.1"}))
+        c.Assert(err, gc.IsNil)
 
         s.st, s.machine = s.OpenAPIAsNewMachine(c)
         // Create the machiner API facade.
         s.machiner = s.st.Machiner()
         c.Assert(s.machiner, gc.NotNil)
-        s.APIAddresserTests = apitesting.NewAPIAddresserTests(s.BackingState, s.machiner)
+        s.APIAddresserTests = apitesting.NewAPIAddresserTests(s.machiner, s.BackingState)
 }
 
 func (s *machinerSuite) TestMachineAndMachineTag(c *gc.C) {
         machine, err := s.machiner.Machine("machine-42")
         c.Assert(err, gc.ErrorMatches, "permission denied")
         c.Assert(err, jc.Satisfies, params.IsCodeUnauthorized)
         c.Assert(machine, gc.IsNil)
 
         machine, err = s.machiner.Machine("machine-1")
         c.Assert(err, gc.IsNil)
@@ skipping 102 matching lines @@
         wc.AssertNoChange()
 
         // Make the machine dead and check it's detected.
         err = machine.EnsureDead()
         c.Assert(err, gc.IsNil)
         wc.AssertOneChange()
 
         statetesting.AssertStop(c, w)
         wc.AssertClosed()
 }