-
Notifications
You must be signed in to change notification settings - Fork 25
Enhance test duration workflow and script #2984
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
base: main
Are you sure you want to change the base?
Conversation
7947864 to
d4f185c
Compare
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #2984 +/- ##
=======================================
Coverage 62.65% 62.65%
=======================================
Files 156 156
Lines 13266 13266
=======================================
Hits 8312 8312
Misses 4954 4954 ☔ View full report in Codecov by Sentry. |
3576368 to
4ca94d2
Compare
4ca94d2 to
d945390
Compare
| """ | ||
| Create a simple Excel file with headers and rows and basic styling. | ||
| """ | ||
| workbook = Workbook() |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Excel? Ahm, why not using python set? test_durations is quite a simple file...
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Dumping the test duration updates to an Excel report provides a clear, visual comparison of test duration changes. It helps us easily identify whether a test case is newly added or if its duration has been updated, along with the previous and current values side by side.
Attached the generated excel for better visualization,
This pull request enhances the Update Test Durations workflow with several important improvements and configurable features for better control and reporting.
Summary of Enhancements
1. Selective Duration Overriding (New Input:
override_test_durations).test_durationsfile using only the tests from the latest nightly and push runs.override_test_durationsinput that enables merge mode, where only the durations of existing tests are updated instead of erasing the full file.2. Offset-Based Update Control (New Input:
duration_offset)override_test_durationsis enabled, a test duration is only updated if the absolute difference between the new (nightly/push) duration and the existing duration exceeds the specifiedduration_offsetvalue.3. Detailed Test Duration Report Generation
test_duration_report.xlsx) containing:4. Automatic Pull Request Merging (New Input:
enable_auto_pr_merge)enable_auto_pr_mergeinput to allow automatic merging of the generated PR once it’s successfully created.