Skip to content

Commit 90274bc

Browse files
committed
Ignore empty test suites in GitHub job summary report
1 parent a96fa31 commit 90274bc

File tree

1 file changed

+14
-12
lines changed

1 file changed

+14
-12
lines changed

src/ci/citool/src/metrics.rs

+14-12
Original file line numberDiff line numberDiff line change
@@ -119,18 +119,20 @@ fn test_metadata_name(metadata: &TestSuiteMetadata) -> String {
119119
fn aggregate_test_suites(suites: &[&TestSuite]) -> BTreeMap<String, TestSuiteRecord> {
120120
let mut records: BTreeMap<String, TestSuiteRecord> = BTreeMap::new();
121121
for suite in suites {
122-
let name = test_metadata_name(&suite.metadata);
123-
let record = records.entry(name).or_default();
124-
for test in &suite.tests {
125-
match test.outcome {
126-
TestOutcome::Passed => {
127-
record.passed += 1;
128-
}
129-
TestOutcome::Failed => {
130-
record.failed += 1;
131-
}
132-
TestOutcome::Ignored { .. } => {
133-
record.ignored += 1;
122+
if !suite.tests.is_empty() {
123+
let name = test_metadata_name(&suite.metadata);
124+
let record = records.entry(name).or_default();
125+
for test in &suite.tests {
126+
match test.outcome {
127+
TestOutcome::Passed => {
128+
record.passed += 1;
129+
}
130+
TestOutcome::Failed => {
131+
record.failed += 1;
132+
}
133+
TestOutcome::Ignored { .. } => {
134+
record.ignored += 1;
135+
}
134136
}
135137
}
136138
}

0 commit comments

Comments
 (0)