Skip to content

Conversation

@urfeex
Copy link
Member

@urfeex urfeex commented Nov 3, 2025

This attempts to address some of the flakiness in our integration tests.

@codecov
Copy link

codecov bot commented Nov 3, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 15.29%. Comparing base (1b121b7) to head (3b8aaea).
⚠️ Report is 510 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff             @@
##            main    #1559       +/-   ##
==========================================
+ Coverage   3.59%   15.29%   +11.70%     
==========================================
  Files         13       34       +21     
  Lines        947     3878     +2931     
  Branches     152      467      +315     
==========================================
+ Hits          34      593      +559     
- Misses       843     3268     +2425     
+ Partials      70       17       -53     
Flag Coverage Δ
unittests 15.29% <ø> (+11.70%) ⬆️

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

☔ View full report in Codecov by Sentry.
📢 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.

@urfeex urfeex closed this Nov 4, 2025
@urfeex urfeex reopened this Nov 4, 2025
@urfeex urfeex requested a review from a team November 4, 2025 15:35
@urfeex urfeex added the CI label Nov 4, 2025
@urfeex urfeex marked this pull request as ready for review November 4, 2025 15:35
@urfeex urfeex marked this pull request as draft November 5, 2025 08:28
@urfeex urfeex closed this Nov 5, 2025
@urfeex urfeex reopened this Nov 5, 2025
Otherwise there is a race condition between sending the program ans
starting the tests.
@urfeex urfeex marked this pull request as ready for review November 10, 2025 14:42
@urfeex urfeex merged commit 32cf968 into UniversalRobots:main Nov 11, 2025
13 of 15 checks passed
@urfeex urfeex deleted the fix_flaky_tests branch November 11, 2025 08:09
mergify bot pushed a commit that referenced this pull request Nov 11, 2025
Wait for controller to become active

Otherwise there is a race condition between sending the program ans
starting the tests.

(cherry picked from commit 32cf968)
mergify bot pushed a commit that referenced this pull request Nov 11, 2025
Wait for controller to become active

Otherwise there is a race condition between sending the program ans
starting the tests.

(cherry picked from commit 32cf968)
urfeex pushed a commit that referenced this pull request Nov 12, 2025
Wait for controller to become active

Otherwise there is a race condition between sending the program ans
starting the tests.
urfeex pushed a commit that referenced this pull request Nov 12, 2025
Wait for controller to become active

Otherwise there is a race condition between sending the program and
starting the tests.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants