Skip to content

Conversation

@kei-nan
Copy link
Contributor

@kei-nan kei-nan commented Oct 21, 2025

When we have a lot of tests then the outpus size can be massive and it is hard to find the problematic tests.
When running in github group all tests under a single group, print the summary outside of the group.
image

@codecov-commenter
Copy link

codecov-commenter commented Dec 2, 2025

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

Codecov Report

❌ Patch coverage is 7.14286% with 13 lines in your changes missing coverage. Please review.
✅ Project coverage is 32.56%. Comparing base (02cfbb9) to head (03e2720).
⚠️ Report is 3 commits behind head on master.

Files with missing lines Patch % Lines
RLTest/__main__.py 0.00% 12 Missing ⚠️
RLTest/utils.py 50.00% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #236      +/-   ##
==========================================
+ Coverage   32.46%   32.56%   +0.10%     
==========================================
  Files          17       17              
  Lines        2597     2641      +44     
==========================================
+ Hits          843      860      +17     
- Misses       1754     1781      +27     
Flag Coverage Δ
unittests 32.56% <7.14%> (+0.10%) ⬆️

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.

@kei-nan kei-nan force-pushed the master_jk_group_passing_tests_in_github branch 2 times, most recently from bdb4b35 to 02d8376 Compare December 2, 2025 17:48
kei-nan and others added 4 commits December 2, 2025 19:56
…dules#237)

* add depth and message for Query, and some related fixes

* remove unused import
* split nightly from generic CI

* version bump actions

* update poetry lock

* remove usage of pkg_resources

* bump poetry version

* update toml and poetry lock again

* fix whitespace

* Rename nightly-build job to build in biweekly.yml

* support version on python 3.7

* run CI on older and newer python version

* add old versions back and lock poetry

* fix run

* another attempt

* another attempt

* another attempt

* another attempt

* another attempt

* another attempt

* another attempt

* fix poetry lock

* CI improvement

* improve comment and concurrency group

* add failure notification
* refactor to use cluster slots to ensure agreement

* expose to Env

* wait for both OK and same topo

* minor improvements
@kei-nan kei-nan force-pushed the master_jk_group_passing_tests_in_github branch from 02d8376 to 1baa5fa Compare December 2, 2025 17:56
@kei-nan kei-nan requested a review from GuyAv46 December 8, 2025 08:13
GuyAv46
GuyAv46 previously approved these changes Dec 8, 2025
GuyAv46
GuyAv46 previously approved these changes Dec 8, 2025
Co-authored-by: GuyAv46 <47632673+GuyAv46@users.noreply.github.com>
@kei-nan kei-nan changed the title Group Successful Tests When Running In Github CI Group Tests When Running In Github CI Dec 8, 2025
@GuyAv46 GuyAv46 merged commit 9df1589 into RedisLabsModules:master Dec 8, 2025
9 checks passed
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.

4 participants