Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Make test GetLatest stricter about timestamps #1719

Merged
merged 2 commits into from
Sep 13, 2019

Conversation

astafan8
Copy link
Contributor

... by adding time.sleep(1) to ensure that timestamps
should be different regardless of the resolution
of the function that produces the timestamps.

... by adding `time.sleep(1)` to ensure that timestamps 
should be different regardless of the resolution 
of the function that produces the timestamps.
@codecov
Copy link

codecov bot commented Sep 13, 2019

Codecov Report

Merging #1719 into master will increase coverage by <.01%.
The diff coverage is n/a.

@@            Coverage Diff             @@
##           master    #1719      +/-   ##
==========================================
+ Coverage   67.21%   67.22%   +<.01%     
==========================================
  Files         145      145              
  Lines       17951    17951              
==========================================
+ Hits        12066    12067       +1     
+ Misses       5885     5884       -1

Copy link
Collaborator

@jenshnielsen jenshnielsen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes

@astafan8 astafan8 merged commit bfee6d9 into master Sep 13, 2019
@astafan8 astafan8 deleted the getlatest-strict-timestamp-test branch September 13, 2019 12:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants