Code review comment for lp:~themue/juju-core/031-fix-resumer-test

Revision history for this message
Dimiter Naydenov (dimitern) wrote :

LGTM modulo a single suggestion.

https://codereview.appspot.com/10550043/diff/1/worker/resumer/resumer_test.go
File worker/resumer/resumer_test.go (right):

https://codereview.appspot.com/10550043/diff/1/worker/resumer/resumer_test.go#newcode47
worker/resumer/resumer_test.go:47: c.Assert(len(tr.timestamps) > 0,
Equals, true)
This code block is a bit obscure to my puny mind - can you put some
comment describing what we're actually doing here please?

https://codereview.appspot.com/10550043/

« Back to merge proposal