~rogpeppe/juju-core/438-local-instance-Addresses

« back to all changes in this revision

Viewing changes to mstate/state.go

mstate: refactor tests like in state.

Tests are refactored into multiple suites to be compatible with
state. See issue 6348053. After mstate replaces state I plan to reverse
the dependencies between suites. Right now each suite embeds an UtilSuite
and has to call UtilSuite.SetUpTest resulting in error prone duplicate
code. Keeping the multiple suites as containers for data and embedding
all suites into a StateSuite shared by all tests is a better design. We
will keep tests in separate files.

R=niemeyer
CC=
https://codereview.appspot.com/6381045

Show diffs side-by-side

added added

removed removed

Lines of Context:
53
53
        change := bson.D{{"$set", bson.D{{"life", Dying}}}}
54
54
        err := s.machines.Update(sel, change)
55
55
        if err != nil {
56
 
                return fmt.Errorf("can't remove machine %d", id)
 
56
                return fmt.Errorf("can't remove machine %d: %v", id, err)
57
57
        }
58
58
        return nil
59
59
}
130
130
// RemoveService removes a service from the state. It will also remove all
131
131
// its units and break any of its existing relations.
132
132
func (s *State) RemoveService(svc *Service) (err error) {
133
 
        defer errorContextf(&err, "can't remove service %s", svc)
 
133
        defer errorContextf(&err, "can't remove service %q", svc)
134
134
 
135
135
        sel := bson.D{{"_id", svc.name}, {"life", Alive}}
136
136
        change := bson.D{{"$set", bson.D{{"life", Dying}}}}