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

🐛 Fix flaky test_disk_usage #5364

Merged

Conversation

GitHK
Copy link
Contributor

@GitHK GitHK commented Feb 23, 2024

What do these changes do?

As usual comparing floats is hard. Done the right way. Added regression tests.

Related issue/s

How to test

Dev Checklist

DevOps Checklist

@GitHK GitHK self-assigned this Feb 23, 2024
@GitHK GitHK added this to the Schoggilebe milestone Feb 23, 2024
@GitHK GitHK added the t:maintenance Some planned maintenance work label Feb 23, 2024
@GitHK GitHK marked this pull request as ready for review February 23, 2024 07:36
Copy link

Quality Gate Passed Quality Gate passed

Issues
0 New issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

Copy link

codecov bot commented Feb 23, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.7%. Comparing base (889e0ff) to head (afc34dc).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff            @@
##           master   #5364     +/-   ##
========================================
+ Coverage    86.8%   87.7%   +0.8%     
========================================
  Files        1187    1095     -92     
  Lines       49076   48186    -890     
  Branches      923     553    -370     
========================================
- Hits        42643   42290    -353     
+ Misses       6236    5775    -461     
+ Partials      197     121     -76     
Flag Coverage Δ
integrationtests 65.1% <ø> (+<0.1%) ⬆️
unittests 85.3% <ø> (+0.8%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

see 354 files with indirect coverage changes

@GitHK GitHK merged commit f23a597 into ITISFoundation:master Feb 23, 2024
55 checks passed
@GitHK GitHK deleted the pr-osparc-fix-flaky-disk-usage-test branch February 23, 2024 08:27
jsaq007 pushed a commit to jsaq007/osparc-simcore that referenced this pull request Feb 27, 2024
jsaq007 pushed a commit to jsaq007/osparc-simcore that referenced this pull request Mar 4, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
a:models-library t:maintenance Some planned maintenance work
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Flaky: Random failure tests/test_api_schemas_dynamic_sidecar_telemetry.py::test_disk_usage
3 participants