/juju

Migrate state tests from time.Now() to [Non]ZeroTime() #6333

Merged
merged 1 commit into from Sep 27, 2016

Conversation

Reviewers

@dimitern dimitern

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@macgreagoir @jujubot @dimitern
@macgreagoir
Contributor

macgreagoir commented Sep 27, 2016

Continue the migration of state tests to use ZeroTime() or NonZeroTime()
from the testing package in place of time.Now().
@macgreagoir
Migrate state tests from time.Now() to [Non]ZeroTime() 
Continue the migration of state tests to use ZeroTime() or NonZeroTime()
from the testing package in place of time.Now().
e77a733
@dimitern

dimitern approved these changes Sep 27, 2016
View changes

LGTM, thanks!

@macgreagoir
Contributor

macgreagoir commented Sep 27, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Sep 27, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit aac2675 into juju:master Sep 27, 2016

@macgreagoir macgreagoir deleted the macgreagoir:state-time-tests branch Oct 5, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment