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

TST: Redirect test stdout to the qmflows logger and re-enable sphinx tests #297

Merged
merged 3 commits into from
May 10, 2022

Conversation

BvB93
Copy link
Collaborator

@BvB93 BvB93 commented May 9, 2022

No description provided.

@BvB93 BvB93 added the Tests Updated the unit-tests label May 9, 2022
@codecov
Copy link

codecov bot commented May 9, 2022

Codecov Report

Merging #297 (13086e7) into master (13e9f0f) will increase coverage by 0.61%.
The diff coverage is 90.00%.

@@            Coverage Diff             @@
##           master     #297      +/-   ##
==========================================
+ Coverage   83.83%   84.45%   +0.61%     
==========================================
  Files          60       60              
  Lines        2407     2431      +24     
  Branches      411      414       +3     
==========================================
+ Hits         2018     2053      +35     
+ Misses        290      282       -8     
+ Partials       99       96       -3     
Impacted Files Coverage Δ
src/qmflows/_settings.py 90.76% <ø> (ø)
src/qmflows/packages/_cp2k_mm.py 84.33% <ø> (ø)
src/qmflows/packages/_scm.py 68.75% <ø> (ø)
src/qmflows/packages/__init__.py 91.66% <75.00%> (-8.34%) ⬇️
src/qmflows/test_utils.py 81.45% <86.66%> (+2.36%) ⬆️
src/qmflows/__init__.py 100.00% <100.00%> (+17.85%) ⬆️
src/qmflows/_logger.py 100.00% <100.00%> (ø)
src/qmflows/packages/_package_wrapper.py 89.28% <100.00%> (ø)
src/qmflows/packages/_packages.py 84.15% <0.00%> (+1.50%) ⬆️
... and 2 more

📣 Codecov can now indicate which changes are the most critical in Pull Requests. Learn more

@BvB93 BvB93 force-pushed the logger branch 3 times, most recently from 6e9dd73 to 7710784 Compare May 9, 2022 23:09
@BvB93 BvB93 merged commit b9c2795 into master May 10, 2022
@BvB93 BvB93 deleted the logger branch May 10, 2022 11:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Tests Updated the unit-tests
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant