state: Add networks doc for services

state/service_test.go

 // Copyright 2012, 2013 Canonical Ltd.
 // Licensed under the AGPLv3, see LICENCE file for details.
 
 package state_test
 
 import (
         "fmt"
         "sort"
 
         jc "github.com/juju/testing/checkers"
@@ skipping 1411 matching lines @@
 // SCHEMACHANGE
 // TODO(mattyw) remove when schema upgrades are possible
 // Check that GetOwnerTag returns user-admin even
 // when the service has no owner
 func (s *ServiceSuite) TestOwnerTagSchemaProtection(c *gc.C) {
         service := s.AddTestingService(c, "foobar", s.charm)
         state.SetServiceOwnerTag(service, "")
         c.Assert(state.GetServiceOwnerTag(service), gc.Equals, "")
         c.Assert(service.GetOwnerTag(), gc.Equals, "user-admin")
 }
+
+func (s *ServiceSuite) TestNetworks(c *gc.C) {

[dimitern, 2014/03/21 14:21:56]

Please add a test to check we can set networks at service creation and we can
read them back.

[gz, 2014/03/21 15:46:00]

That is deliberately not included in this mp, but I am missing some test
coverage for the functionality that is here.

[dimitern, 2014/03/21 16:11:32]

I'm fine with adding this in a follow-up.

+        service, err := s.State.Service(s.mysql.Name())
+        c.Assert(err, gc.IsNil)
+        includedNetworks, excludedNetworks, err := service.Networks()
+        c.Assert(err, gc.IsNil)
+        c.Check(includedNetworks, gc.DeepEquals, []string{})
+        c.Check(excludedNetworks, gc.DeepEquals, []string{})
+}