~axwalk/juju-core/api-uniter-machine

« back to all changes in this revision

Viewing changes to state/service_test.go

  • Committer: Tarmac
  • Author(s): William Reade
  • Date: 2014-05-22 11:37:17 UTC
  • mfrom: (1679.1.11 juju-core)
  • Revision ID: tarmac-20140522113717-99dhnikcznpjpwrw
[r=fwereade],[bug=1192433] state: set departed on dying units' relations

This ended up including:

  * stopping using magic strings in cleanup
  * drawing a clear distinction between "joined" and "in scope" (which
    accounts for most of the lines of changes)

...but did *not* include:

  * fixing the name of the Uniter.JoinedRelations API, because I want to
    wait for API versioning

https://codereview.appspot.com/94540043/

Show diffs side-by-side

added added

removed removed

Lines of Context:
1102
1102
                }
1103
1103
        }
1104
1104
 
 
1105
        // Check for queued unit cleanups, and run them.
 
1106
        dirty, err = s.State.NeedsCleanup()
 
1107
        c.Assert(err, gc.IsNil)
 
1108
        c.Assert(dirty, gc.Equals, true)
 
1109
        err = s.State.Cleanup()
 
1110
        c.Assert(err, gc.IsNil)
 
1111
 
1105
1112
        // Check we're now clean.
1106
1113
        dirty, err = s.State.NeedsCleanup()
1107
1114
        c.Assert(err, gc.IsNil)