Ignore empty test suites in GitHub job summary report

Author: Kobzol

src/ci/citool/src/metrics.rs

@@ -119,18 +119,20 @@ fn test_metadata_name(metadata: &TestSuiteMetadata) -> String {
 fn aggregate_test_suites(suites: &[&TestSuite]) -> BTreeMap<String, TestSuiteRecord> {
  let mut records: BTreeMap<String, TestSuiteRecord> = BTreeMap::new();
  for suite in suites {
- let name = test_metadata_name(&suite.metadata);
- let record = records.entry(name).or_default();
- for test in &suite.tests {
- match test.outcome {
- TestOutcome::Passed => {
- record.passed += 1;
- }
- TestOutcome::Failed => {
- record.failed += 1;
- }
- TestOutcome::Ignored { .. } => {
- record.ignored += 1;
+ if !suite.tests.is_empty() {
+ let name = test_metadata_name(&suite.metadata);
+ let record = records.entry(name).or_default();
+ for test in &suite.tests {
+ match test.outcome {
+ TestOutcome::Passed => {
+ record.passed += 1;
+ }
+ TestOutcome::Failed => {
+ record.failed += 1;
+ }
+ TestOutcome::Ignored { .. } => {
+ record.ignored += 1;
+ }
  }
  }
  }