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

[Draft] Split unit tests in CI #17608

Closed
Closed
Changes from 1 commit
Commits
Show all changes
27 commits
Select commit Hold shift + click to select a range
e38d20f
Split processing_modules_test into 2 ranges
Akshat-Jain Jan 7, 2025
b57e01b
Try commenting unnecessary checks
Akshat-Jain Jan 7, 2025
f3b7265
Fix issue with test_range not present in inputs
Akshat-Jain Jan 7, 2025
40cac46
Disable failIfNoSpecifiedTests
Akshat-Jain Jan 7, 2025
82c7497
Split indexing_modules_test and other_modules_test
Akshat-Jain Jan 7, 2025
0e0372d
Add test_range to job name
Akshat-Jain Jan 7, 2025
89c3877
Try splits from A-J and K-Z
Akshat-Jain Jan 7, 2025
8dad5f7
Remove Test from regex pattern matching
Akshat-Jain Jan 7, 2025
03a7136
Try splits from A-K and L-Z
Akshat-Jain Jan 7, 2025
e4d9e46
Revert "Remove Test from regex pattern matching"
Akshat-Jain Jan 7, 2025
de3ed41
Use A*,B* approach
Akshat-Jain Jan 7, 2025
f32487d
Use A*,B* approach
Akshat-Jain Jan 7, 2025
95dfcde
Revert "Use A*,B* approach"
Akshat-Jain Jan 7, 2025
b173d66
Revert "Use A*,B* approach"
Akshat-Jain Jan 7, 2025
d94ba1f
Restore A*,B* approach and make a couple of classes abstract
Akshat-Jain Jan 7, 2025
c13e965
Investigating jacoco issues of azure module because of no tests in split
Akshat-Jain Jan 7, 2025
ab70a11
Remove hardcoded jacoco config limits from azure module
Akshat-Jain Jan 7, 2025
3f0605b
Revert "Investigating jacoco issues of azure module because of no tes…
Akshat-Jain Jan 7, 2025
4c77f42
Remove echo
Akshat-Jain Jan 7, 2025
edcdbc4
Revert "Try commenting unnecessary checks"
Akshat-Jain Jan 7, 2025
2d36e84
Try to fix reading matrix outputs
Akshat-Jain Jan 7, 2025
f7492cb
Try to fix reading matrix outputs v2
Akshat-Jain Jan 7, 2025
25b1582
Run check_results always
Akshat-Jain Jan 7, 2025
703099f
Debug reading matrix values
Akshat-Jain Jan 7, 2025
df04ce8
Reduce tests and try reading job result and outputs
Akshat-Jain Jan 8, 2025
5cbb8ed
Change default to X-X
Akshat-Jain Jan 8, 2025
4f000f0
Comment out last layer of tests
Akshat-Jain Jan 8, 2025
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Try to fix reading matrix outputs
  • Loading branch information
Akshat-Jain committed Jan 7, 2025
commit 2d36e84f025daea03a93a339f8f14b9837b800f9
13 changes: 5 additions & 8 deletions .github/workflows/unit-tests.yml
Original file line number Diff line number Diff line change
@@ -26,14 +26,11 @@ on:
description: 'Flag to decide if next tests need to run incase coverage issue failures'
value: |
${{
(jobs.indexing_modules_test.result == 'success' || jobs.indexing_modules_test.result == 'skipped' ||
fromJson(jobs.indexing_modules_test.outputs.coverage_failure)) &&
(jobs.processing_modules_test.result == 'success' || jobs.processing_modules_test.result == 'skipped' ||
fromJson(jobs.processing_modules_test.outputs.coverage_failure)) &&
(jobs.server_modules_test.result == 'success' || jobs.server_modules_test.result == 'skipped' || fromJson
(jobs.server_modules_test.outputs.coverage_failure)) &&
(jobs.other_modules_test.result == 'success' || jobs.other_modules_test.result == 'skipped' || fromJson(jobs
.other_modules_test.outputs.coverage_failure))
!contains(needs.*.result, 'failure') ||
(contains(fromJson(needs.indexing_modules_test.outputs.*.coverage_failure || '[]'), true) &&
contains(fromJson(needs.processing_modules_test.outputs.*.coverage_failure || '[]'), true) &&
fromJson(needs.server_modules_test.outputs.coverage_failure || 'false') &&
contains(fromJson(needs.other_modules_test.outputs.*.coverage_failure || '[]'), true))
}}

jobs: