Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(480)

Unified Diff: state/apiserver/client_test.go

Issue 10172045: state: Service.SetConfigYAML
Patch Set: state: Service.SetConfigYAML Created 10 years, 9 months ago
Use n/p to move between diff chunks; N/P to move between comments. Please Sign in to add in-line comments.
Jump to:
View side-by-side diff with in-line comments
Download patch
« no previous file with comments | « juju/conn_test.go ('k') | state/apiserver/perm_test.go » ('j') | no next file with comments »
Expand Comments ('e') | Collapse Comments ('c') | Show Comments Hide Comments ('s')
Index: state/apiserver/client_test.go
=== modified file 'state/apiserver/client_test.go'
--- state/apiserver/client_test.go 2013-06-04 13:51:16 +0000
+++ state/apiserver/client_test.go 2013-06-10 22:46:06 +0000
@@ -47,7 +47,7 @@
func (s *clientSuite) TestClientServiceSetYAML(c *C) {
dummy, err := s.State.AddService("dummy", s.AddTestingCharm(c, "dummy"))
c.Assert(err, IsNil)
- err = s.APIState.Client().ServiceSetYAML("dummy", "title: aaa\nusername: bbb")
+ err = s.APIState.Client().ServiceSetYAML("dummy", "dummy:\n title: aaa\n username: bbb")
c.Assert(err, IsNil)
conf, err := dummy.Config()
c.Assert(err, IsNil)
@@ -302,20 +302,17 @@
var serviceDeployTests = []struct {
about string
- serviceName string
charmUrl string
numUnits int
expectedNumUnits int
constraints constraints.Value
}{{
about: "Normal deploy",
- serviceName: "mywordpress",
charmUrl: "local:series/wordpress",
expectedNumUnits: 1,
constraints: constraints.MustParse("mem=1G"),
}, {
about: "Two units",
- serviceName: "mywordpress",
charmUrl: "local:series/wordpress",
numUnits: 2,
expectedNumUnits: 2,
@@ -332,14 +329,15 @@
localRepo, err := charm.InferRepository(parsedUrl, coretesting.Charms.Path)
c.Assert(err, IsNil)
withRepo(localRepo, func() {
- _, err = s.State.Service(test.serviceName)
+ serviceName := "mywordpress"
+ _, err = s.State.Service(serviceName)
c.Assert(errors.IsNotFoundError(err), Equals, true)
err = s.APIState.Client().ServiceDeploy(
- test.charmUrl, test.serviceName, test.numUnits, "", test.constraints,
+ test.charmUrl, serviceName, test.numUnits, "mywordpress: {}", test.constraints,
)
c.Assert(err, IsNil)
- service, err := s.State.Service(test.serviceName)
+ service, err := s.State.Service(serviceName)
c.Assert(err, IsNil)
defer removeServiceAndUnits(c, service)
scons, err := service.Constraints()
« no previous file with comments | « juju/conn_test.go ('k') | state/apiserver/perm_test.go » ('j') | no next file with comments »

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b