~thumper/juju-core/move-environs-tools

Viewing changes to worker/cleaner/cleaner_test.go

Committer: Tarmac
Author(s): John Arbash Meinel
Date: 2013-07-08 12:51:34 UTC
mfrom: (1390.3.4 use-notify-worker)
Revision ID: tarmac-20130708125134-nr74t8u11iw61vu0

[r=jameinel] worker/cleaner: use new NotifyWorker structure

Just reimplements the Cleaner worker as a WatchHandler passed to the
NotifyWorker. Cleaner was pretty simple, but I'm still able to remove about
20 lines of boilerplate. (And replace it with heavily tested logic.)

https://codereview.appspot.com/10876048/

files modified:
worker/cleaner/cleaner.go

worker/cleaner/cleaner_test.go

Show diffs side-by-side

added added

removed removed

worker/cleaner/cleaner_test.go

var _ = Suite(&CleanerSuite{})

var _ worker.Worker = (*cleaner.Cleaner)(nil)

var _ worker.WatchHandler = (*cleaner.Cleaner)(nil)

func (s *CleanerSuite) TestCleaner(c *C) {

cr := cleaner.NewCleaner(s.State)

Older »