Skip to content

test(integration): make a test stable by removing time constraints#214

Merged
mdevolde merged 1 commit into
jxmorris12:masterfrom
mdevolde:test/non_stable
Jul 2, 2026
Merged

test(integration): make a test stable by removing time constraints#214
mdevolde merged 1 commit into
jxmorris12:masterfrom
mdevolde:test/non_stable

Conversation

@mdevolde

@mdevolde mdevolde commented Jul 2, 2026

Copy link
Copy Markdown
Collaborator

test(integration): make a test stable by removing time constraints

Why the pull request was made

To make the test CI stable, and avoid to depend on the CI runners performances.

Summary of changes

Screenshots (if appropriate):

Not applicable.

How has this been tested?

Applied local tests.

Resources

Not applicable.

Types of changes

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Documentation update (changes to documentation only)
  • Refactor / code style update (non-breaking change that improves code structure or readability)
  • Tests / CI improvement (adding or updating tests or CI configuration only)
  • Chore / maintenance (non-breaking change that does not affect functionality, such as updating dependencies or fixing typos)
  • Other (please describe):

Checklist

  • Followed the project's contributing guidelines.
  • Updated any relevant tests.
  • Updated any relevant documentation.
  • Added comments to your code where necessary.
  • Formatted your code, run the linters, checked types and tests.
  • Added your changes to the CHANGELOG file, if applicable.

@mdevolde mdevolde self-assigned this Jul 2, 2026
@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (be97cfc) to head (ec9fa45).
❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##           master      #214      +/-   ##
===========================================
+ Coverage   99.88%   100.00%   +0.11%     
===========================================
  Files          13        13              
  Lines        1699      1696       -3     
===========================================
- Hits         1697      1696       -1     
+ Misses          2         0       -2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@mdevolde mdevolde merged commit 4477151 into jxmorris12:master Jul 2, 2026
13 checks passed
@mdevolde mdevolde deleted the test/non_stable branch July 2, 2026 14:14
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