gitlab_quality-test_tooling 3.19.0 → 3.19.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: de285d350eb535db983a229c6fbe2ea8565fd92ac757ba4371a7abf2e86a0e44
-  data.tar.gz: bb1143033fdd4d8e0c71c4a0ca27574391138feed3b5198f934df1356684c866
+  metadata.gz: 5fb5356dd9a684e76ccf4de875f66d77ee818bef86d1cd9792eb8db9d688cfe9
+  data.tar.gz: 75e5afd55644a124fbf09f4fe0edd8f298fb2614f54dd0884a8a18002d2fe6d8
 SHA512:
-  metadata.gz: 43d770b36e5b946f287a5f309f0e242033432c9b9a2f7618e396abc894067314d26a0b88416c41f880d1a16c5ffe7a595e306f074a057478fb139f2361e77e56
-  data.tar.gz: 8f61352e3441e40c2b09d45621e91028f8d1f48623859ba34f5aabc7db449b03bc80f18ede9cbb558c47776d4a6b10c7fdb8e39e22b1dd0158477f2d85d97d8c
+  metadata.gz: 35ff364a8fbbd437bef5e94a6bedccf66b8288965fd6c136ce6c21baa3d9052ac44281e54dd3a11eb92dae156f6e9e123756fa526ddb24fd39c00450b1d7187e
+  data.tar.gz: 1de79819dcb8aadf7ef3994f54387008ff9d1e15ea4a0b0590caebe9cf2af99a2821dcc7008b7bff5b2df9da69d4103622624983ce7434272ec2e5663c1e6695

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (3.19.0)
+    gitlab_quality-test_tooling (3.19.1)
       activesupport (>= 7.0)
       amatch (~> 0.4.1)
       fog-google (~> 1.24, >= 1.24.1)

data/lib/gitlab_quality/test_tooling/code_coverage/click_house/test_health_risk_aggregation.sql

@@ -71,14 +71,35 @@ WITH
     GROUP BY test_file
     HAVING is_flaky
   ),
+  test_ownership AS (
+    -- Authoritative test-file ownership from test_metrics, which resolves
+    -- feature_category to group/stage/section from a current source. The
+    -- per-test coverage export's own CategoryOwners mapping is stale (it fetches
+    -- a dated stages.yml), so many rows arrive with a blank group; this is the
+    -- fallback. argMax pins the latest ownership per file over the risk window.
+    SELECT
+      file_path,
+      argMax(`group`, timestamp) AS owner_group,
+      argMax(stage, timestamp) AS owner_stage,
+      argMax(section, timestamp) AS owner_section
+    FROM test_metrics.test_results
+    WHERE timestamp >= now() - INTERVAL {risk_window}
+      AND `group` NOT IN ('', 'unknown')
+    GROUP BY file_path
+  ),
   per_test_status AS (
     SELECT
-      tc.source_file,
-      tc.`group`,
-      tc.stage,
-      tc.section,
-      tc.total_lines,
-      tc.covered_lines,
+      tc.source_file AS source_file,
+      -- Fall back to test_metrics ownership when the coverage row has no group
+      -- (stale CategoryOwners mapping). Resolve all three off the same condition
+      -- so the team stays internally consistent. Alias the passthrough columns to
+      -- bare names too: the analyzer otherwise keeps the `tc.` qualifier once the
+      -- ownership join is in play, and per_file references them unqualified.
+      if(tc.`group` != '', tc.`group`, own.owner_group) AS `group`,
+      if(tc.`group` != '', tc.stage, own.owner_stage) AS stage,
+      if(tc.`group` != '', tc.section, own.owner_section) AS section,
+      tc.total_lines AS total_lines,
+      tc.covered_lines AS covered_lines,
       -- Jest quarantine is a flat list (quarantined_vue3_specs.txt), loaded into
       -- jest_quarantined_tests_today by JestQuarantinedTestsTable before each run.
       (coalesce(qs.is_quarantined, FALSE)
@@ -87,6 +108,7 @@ WITH
     FROM code_coverage.test_coverage_per_file tc FINAL
     LEFT JOIN quarantine_status qs ON qs.test_file = tc.test_file
     LEFT JOIN flaky_status      fs ON fs.test_file = tc.test_file
+    LEFT JOIN test_ownership own ON own.file_path = tc.test_file
     WHERE tc.timestamp >= now() - INTERVAL {coverage_window}
   ),
   per_file AS (

data/lib/gitlab_quality/test_tooling/code_coverage/per_test_coverage_exporter.rb

@@ -131,8 +131,8 @@ module GitlabQuality
         end
 
         # `tests_to_categories` comes from the test report JSON; without it, rows
-        # carry blank categories. `CategoryOwners` fetches stages.yml over HTTP,
-        # so only build it when there are reports to enrich.
+        # carry blank categories. The category-to-team map is a ClickHouse read,
+        # so only build both when there are reports to enrich.
         def resolve_categories
           return [{}, {}] unless test_reports
 
@@ -141,7 +141,19 @@ module GitlabQuality
           report_files = Dir.glob(patterns)
           return [{}, {}] if report_files.empty?
 
-          [tests_to_categories_from(report_files), CategoryOwners.new.feature_categories_to_teams]
+          [tests_to_categories_from(report_files), feature_categories_to_teams]
+        end
+
+        # Category to group/stage/section, read from `code_coverage.category_owners`
+        # (populated by sync-category-owners). This is the source the full
+        # coverage export already uses, and it retains categories the live
+        # stages.yml fetch has since dropped (global_search and the Duo/AI groups),
+        # so more rows resolve to a real group instead of blank. `owner_records`
+        # gives string-keyed values; PerTestCoverageData reads symbol keys.
+        def feature_categories_to_teams
+          ClickHouse::CategoryOwnersTable.new(**clickhouse_credentials).owner_records.transform_values do |owner|
+            { group: owner["group"], stage: owner["stage"], section: owner["section"] }
+          end
         end
 
         def tests_to_categories_from(report_files)

data/lib/gitlab_quality/test_tooling/version.rb

@@ -2,6 +2,6 @@
 
 module GitlabQuality
   module TestTooling
-    VERSION = "3.19.0"
+    VERSION = "3.19.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_quality-test_tooling
 version: !ruby/object:Gem::Version
-  version: 3.19.0
+  version: 3.19.1
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-06-27 00:00:00.000000000 Z
+date: 2026-07-02 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control