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

Hide intentional "coroutine '...' was never awaited" in test suite #189

Conversation

njsmith
Copy link
Member

@njsmith njsmith commented Jun 7, 2017

In addition to cluttering up the test output now that pytest collects
and shows warnings, this causes segfaults on some CI builds when run
with -W error (see gh-187).

In addition to cluttering up the test output now that pytest collects
and shows warnings, this causes segfaults on some CI builds when run
with -W error (see python-triogh-187).
@codecov
Copy link

codecov bot commented Jun 7, 2017

Codecov Report

Merging #189 into master will increase coverage by <.01%.
The diff coverage is 100%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #189      +/-   ##
==========================================
+ Coverage   98.44%   98.45%   +<.01%     
==========================================
  Files          53       53              
  Lines        6129     6132       +3     
  Branches      476      476              
==========================================
+ Hits         6034     6037       +3     
  Misses         80       80              
  Partials       15       15
Impacted Files Coverage Δ
trio/_core/tests/test_run.py 100% <100%> (ø) ⬆️

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 b2e043b...3b831cc. Read the comment docs.

@njsmith njsmith merged commit 77f2962 into python-trio:master Jun 7, 2017
@njsmith njsmith deleted the hide-never-awaited-warning-in-test-error-in-run-loop branch June 8, 2017 00:37
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