/juju

Use the mock clock in TestMeterStatusSuccesfulSend. #6446

Merged
merged 1 commit into from Oct 13, 2016

Conversation

Reviewers

@tasdomas tasdomas

@natefinch natefinch

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
5 participants
@cmars @jujubot @nskaggs @tasdomas @natefinch
@cmars
Owner

cmars commented Oct 13, 2016

This fixes an "intermittent" test failure which could have been caused
by the real-world clock time from setup to the affected test being > 1s.
@cmars
Use the mock clock in TestMeterStatusSuccesfulSend. 
This fixes an "intermittent" test failure which could have been caused
by the real-world clock time from setup to the affected test being > 1s.
9291126
@cmars
Owner

cmars commented Oct 13, 2016

@natefinch PTAL
@natefinch

natefinch approved these changes Oct 13, 2016
View changes

LGTM

@cmars
Owner

cmars commented Oct 13, 2016

$$merge$$
@jujubot
Contributor

jujubot commented Oct 13, 2016

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

nskaggs commented Oct 13, 2016

Sorry cmars, merges still disabled for release. They'll open soon.

@jujubot jujubot merged commit 82bbf52 into juju:develop Oct 13, 2016
1 check passed

1 check passed

github-check-merge-juju Built PR, ran unit tests, and tested LXD deploy.
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment