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

Decouple tests from unspecified data ordering (part 2) #2652

Merged

Conversation

AndrewKvalheim
Copy link
Member

Continuing #2650—I just ran across a few more.

These tests have been passing with SQLite, but don't reflect guaranteed
behavior and with PostgreSQL failed intermittently.
@differentreality
Copy link
Contributor

Hmmm I think travis may be stuck...

@codecov
Copy link

codecov bot commented Apr 1, 2020

Codecov Report

Merging #2652 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2652   +/-   ##
=======================================
  Coverage   82.15%   82.15%           
=======================================
  Files         146      146           
  Lines        4780     4780           
=======================================
  Hits         3927     3927           
  Misses        853      853

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e709d08...a179a38. Read the comment docs.

@AndrewKvalheim
Copy link
Member Author

continuous-integration/travis-ci Expected — Waiting for status to be reported

The tests ran fine so apparently Travis and GitHub just aren't communicating correctly.

@differentreality differentreality merged commit 303129b into openSUSE:master Apr 6, 2020
@AndrewKvalheim AndrewKvalheim deleted the test-unspecified-order-2 branch October 31, 2020 00:23
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.

2 participants