gitlab_quality-test_tooling 3.20.2 → 3.21.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6d63cdc47b44dd111330eaffb4e37c889d8785eb395094eec7398c97870f889f
-  data.tar.gz: bbe3389c97486d566bb8a948da3a88cf94633857d8eee5ad9fbb53e6ba38aef5
+  metadata.gz: cbd765eef13187839210bb0dde07988168415170236b8734d3faf396ce95b9a9
+  data.tar.gz: '0877e97fd029bdb06028dad82aeae1a15f18b73e3f8ad2ad9c0448819335ca71'
 SHA512:
-  metadata.gz: 81d28ad1a352c06db086546c5eef55f9f82aa982904e4f3e03ab1c7e509d0a18fde12f6e77e7a09eda3c549dd15ff1d0090f532a2da60eda68d31eba77cee93f
-  data.tar.gz: 8aeb35ec5c0fc54e86c9cb029f44c62feb9dbc1e5031f6d4aaba07a9a3e3791a739841da7a1080bf1c28c05c81baee57a42742dca5ab60d1420e8e74360b9b9c
+  metadata.gz: f242e07dbdefd766fd8f2bec91b779c93bf68f91a913314982b8e3053db99704fa0330cb31dadeb8829c570f43b7f2f04c52265fcdd75cc1beb60f1486096554
+  data.tar.gz: b6cd28e7ad33b233fb6f7c68db39ade7b72118913eac8e3b3281e018d59a9e9e5ca99b50a7730d28f189b5a2517c2c933e254d15f7a50cff6800476346aa4b37

data/AGENTS.md

@@ -75,6 +75,13 @@ Reference: https://docs.gitlab.com/ee/development/changelog.html
 - **CodeCoverage** — Coverage analysis and responsibility patterns
 - **ClickHouse::Client** — HTTP query/insert client for ClickHouse, used by the code coverage subsystem
 
+## Code Comments
+
+Comments carry the non-obvious why, not what the code already says. Keep them
+short. A header block that restates the query or narrates each column gets
+trimmed to the design decisions a reader can't infer from the code. This
+applies to AI-generated comments too: trim before committing.
+
 ## Review Guidance
 
 - `RelateFailureIssue#trigger_duo_analysis` deliberately rescues all `StandardError` (best-effort): a Duo/API failure must never block failure-issue creation. Preserve the broad rescue — don't narrow or remove it.

data/Gemfile.lock

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

data/exe/test-coverage

@@ -73,7 +73,9 @@ options = OptionParser.new do |opts|
       'Inserts rows into code_coverage.test_coverage_per_file and runs the daily ' \
       'test_health_risk aggregation unless --skip-aggregation. Needs only the ' \
       '--clickhouse-* options; --test-reports and --jest-quarantine-file are ' \
-      'optional enrichment. (e.g., "tmp/per-test-coverage-*.ndjson")') do |pattern|
+      'optional enrichment. --test-reports also requires --clickhouse-shared-database, ' \
+      'since category ownership is read from the shared database. ' \
+      '(e.g., "tmp/per-test-coverage-*.ndjson")') do |pattern|
     params[:per_test_coverage] = pattern
   end
 
@@ -135,7 +137,16 @@ mode =
 clickhouse_required = [:clickhouse_url, :clickhouse_database, :clickhouse_username]
 full_export_required =
   [:test_reports, :coverage_report, :test_map, :responsibility_patterns, :clickhouse_shared_database] + clickhouse_required
-required_params = mode == :full_export ? full_export_required : clickhouse_required
+# Per-test mode reads category_owners from the shared database only when it enriches
+# rows (--test-reports present), so require the shared database exactly then.
+per_test_enriching = mode == :per_test && params[:test_reports]
+required_params =
+  case mode
+  when :full_export
+    full_export_required
+  else
+    per_test_enriching ? clickhouse_required + [:clickhouse_shared_database] : clickhouse_required
+  end
 
 unless params.any? && (required_params - params.keys).none?
   puts "Missing argument(s). Required arguments are: #{required_params}\nPassed arguments are: #{params}\n"
@@ -149,7 +160,13 @@ if clickhouse_password.to_s.strip.empty?
   exit 1
 end
 
-non_empty_params = mode == :full_export ? clickhouse_required + [:clickhouse_shared_database] : clickhouse_required
+non_empty_params =
+  if mode == :full_export || per_test_enriching
+    clickhouse_required + [:clickhouse_shared_database]
+  else
+    clickhouse_required
+  end
+
 non_empty_params.each do |param|
   next unless params[param].to_s.strip.empty?
 
@@ -189,6 +206,7 @@ when :per_test
   GitlabQuality::TestTooling::CodeCoverage::PerTestCoverageExporter.new(
     coverage_glob: params[:per_test_coverage],
     clickhouse: clickhouse_data,
+    clickhouse_shared_database: params[:clickhouse_shared_database],
     test_reports: params[:test_reports],
     jest_quarantine_file: params[:jest_quarantine_file],
     skip_aggregation: params.fetch(:skip_aggregation, false)

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

@@ -1,52 +1,28 @@
--- Daily aggregation of per-test coverage data into per-group risk summary.
+-- Daily aggregation of per-test coverage into per-group risk rows: one
+-- (snapshot_date, group, stage, section) row inserted into
+-- code_coverage.test_health_risk_per_group.
 --
--- Reads the bitmap line sets from `code_coverage.test_coverage_per_file`,
--- joins per-test quarantine and flaky status from `test_metrics`, then
--- computes for each source file:
---   quarantined_union        = union of line sets covered by *quarantined* tests
---   flaky_union              = union of line sets covered by *flaky* tests
---   quarantined_or_flaky_union = union of line sets covered by *quarantined OR flaky* tests
---   healthy_union            = union of line sets covered by tests that are neither
+-- "At risk" lines are covered only by quarantined/flaky tests. The healthy
+-- baseline excludes flaky tests by design: a flaky test isn't a reliable
+-- safety net, so its lines shouldn't offset quarantine-only risk.
 --
--- The "at risk" line counts are:
---   at_risk_lines_quarantine = lines uniquely covered by quarantined tests
---                              (not covered by any healthy test)
---   at_risk_lines_flaky      = lines uniquely covered by flaky tests
---   at_risk_lines_combined   = lines uniquely covered by quarantined OR flaky
+-- Parameters use `{name}` braces, substituted as literal text by gsub in
+-- TestHealthRiskAggregator#build_sql after validation. A typoed placeholder
+-- silently survives gsub, so match the names exactly.
+--   {snapshot_date}  date stamp for this run
+--   {coverage_start} day of the most recent full per-test capture
+--   {risk_window}    quarantine/flaky lookback, default '30 DAY'
 --
--- Healthy union excludes flaky tests too, by design: a flaky test isn't a
--- reliable coverage safety net, so its lines shouldn't count toward the
--- baseline against which quarantine-only risk is measured.
+-- The INSERT is unconditional. Idempotency comes from the target table's
+-- ReplacingMergeTree(version) engine keyed by the row tuple: re-runs and
+-- overlapping cross-run inserts replace rather than duplicate, latest wins.
 --
--- Result: one row per (snapshot_date, group, stage, section) tuple inserted
--- into `code_coverage.test_health_risk_per_group`.
---
--- Parameter substitution: this template uses `{name}` braces (not %{name} or
--- :name). They're replaced by literal text via `gsub` in
--- TestHealthRiskAggregator#build_sql, after each parameter passes a regex
--- check. A typoed placeholder name will silently fall through gsub as a
--- literal — match the names exactly.
---
---   {snapshot_date}     : the date stamp for this run, e.g. '2026-05-07'
---   {coverage_window}   : interval to look back for fresh per-test rows,
---                         default '2 DAY' (see Resilience note below)
---   {risk_window}       : interval to consider quarantine/flaky status from
---                         the test_metrics summaries, default '30 DAY'
-
--- Idempotency: this INSERT is unconditional. The target table must use
--- SharedReplacingMergeTree(version) (or ReplacingMergeTree on non-Cloud
--- ClickHouse; Cloud silently rewrites to Shared...) keyed by
--- (snapshot_date, group, stage, section), with
--- `version UInt64 MATERIALIZED toUnixTimestamp64Milli(now64(3))`, so re-running
--- for the same snapshot_date produces a higher version that replaces the
--- previous row on merge. Without that engine, re-runs would duplicate rows.
---
--- Resilience: `{coverage_window}` defaults to 2 DAY (see
--- TestHealthRiskAggregator::DEFAULT_COVERAGE_WINDOW). One missed nightly
--- run is recovered on the next night because the aggregation still sees
--- the previous day's per-test rows. Cross-run race: if two jobs land
--- inserts overlapping with the aggregation, the later aggregation wins
--- because of the version-based replacement.
+-- Per-test capture is sparse between full sweeps (roughly weekly), so a
+-- fixed trailing window would flap. {coverage_start} anchors the universe at
+-- the last day that cleared the full-capture floor (#find_coverage_start);
+-- when no day qualifies within the lookback, nothing is written and the
+-- previous snapshot carries forward. FINAL on the source keeps the latest
+-- version per (test_file, source_file), so the wide span never double-counts.
 INSERT INTO code_coverage.test_health_risk_per_group
 WITH
   quarantine_status AS (
@@ -109,7 +85,7 @@ WITH
     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}
+    WHERE tc.timestamp >= toDate('{coverage_start}')
   ),
   per_file AS (
     SELECT

data/lib/gitlab_quality/test_tooling/code_coverage/click_house/test_health_risk_aggregator.rb

@@ -19,14 +19,25 @@ module GitlabQuality
 
           SQL_FILE = File.expand_path('test_health_risk_aggregation.sql', __dir__)
 
-          # 2 DAY rather than 1 DAY makes the aggregation self-healing across
-          # a single missed nightly run: if last night's export failed, this
-          # night's run still sees yesterday's per-test rows and produces a
-          # current snapshot. ReplacingMergeTree FINAL on the source table
-          # ensures we read only the latest version per (test_file, source_file).
-          # ClickHouse accepts both '2 DAY' and '2 DAYS'; we use the singular
-          # form for consistency with `30 DAY` below.
-          DEFAULT_COVERAGE_WINDOW = '2 DAY'
+          # Per-test capture is sparse between full captures (a full sweep of
+          # the suite lands roughly weekly; weekdays contribute only a small
+          # delta), so the coverage universe anchors at the most recent day
+          # whose capture volume clears this floor instead of a fixed trailing
+          # interval. Counted in distinct test files, because the weekend
+          # sweep enumerates the whole suite by test file: full sweeps land
+          # 22-27k test files, a single sweep bucket ~13k, and delta days at
+          # most ~16k even at their historical worst. Source files would not
+          # separate the cases: a few thousand tests already touch tens of
+          # thousands of source files through app boot, so a healthy delta
+          # day could pass a source-file floor and anchor the window at a
+          # partial capture.
+          DEFAULT_FULL_CAPTURE_FLOOR = 18_000
+          # Far enough back to bridge two missed weekly full captures before
+          # giving up, and well inside the source table's 90-day TTL. When no
+          # day within the lookback clears the floor, the run writes nothing
+          # so the previous snapshot carries forward rather than publishing a
+          # denominator built from sparse data.
+          DEFAULT_ANCHOR_LOOKBACK = '21 DAY'
           DEFAULT_RISK_WINDOW = '30 DAY'
 
           # `snapshot_date` is YYYY-MM-DD; intervals are `<integer> <unit>`
@@ -36,35 +47,38 @@ module GitlabQuality
 
           def initialize(
             url:, database:, username: nil, password: nil, logger: nil,
-            coverage_window: DEFAULT_COVERAGE_WINDOW, risk_window: DEFAULT_RISK_WINDOW)
+            full_capture_floor: DEFAULT_FULL_CAPTURE_FLOOR, anchor_lookback: DEFAULT_ANCHOR_LOOKBACK,
+            risk_window: DEFAULT_RISK_WINDOW)
             @url = url
             @database = database
             @username = username
             @password = password
             @logger = logger || ::Logger.new($stdout, level: 1)
-            @coverage_window = coverage_window
+            @full_capture_floor = full_capture_floor
+            @anchor_lookback = anchor_lookback
             @risk_window = risk_window
           end
 
           # @param snapshot_date [Date, String] date stamp for this run; defaults to today.
           # @return [void]
           def run(snapshot_date: Date.today) # rubocop:disable Metrics/AbcSize
-            sql = build_sql(snapshot_date: snapshot_date)
-            logger.info(
-              "#{LOG_PREFIX} Running test_health_risk aggregation snapshot_date=#{snapshot_date} " \
-                "coverage_window=#{coverage_window} risk_window=#{risk_window}"
-            )
-            client.query(sql, format: "TabSeparated")
-            inserted = fetch_row_count(snapshot_date)
-            if inserted.is_a?(Integer) && inserted.zero?
+            date = validate_date(snapshot_date)
+            coverage_start = find_coverage_start
+            if coverage_start.nil?
               logger.warn(
-                "#{LOG_PREFIX} Aggregation wrote 0 rows for snapshot_date=#{snapshot_date}. " \
-                  "This is valid if no per-test data is in the coverage_window, but worth checking " \
-                  "test_coverage_per_file directly if a recent export ran."
+                "#{LOG_PREFIX} No day within #{anchor_lookback} captured >= #{full_capture_floor} " \
+                  "distinct test files; skipping snapshot for #{date} so the previous row carries forward."
               )
-            else
-              logger.info("#{LOG_PREFIX} Aggregation wrote #{inserted} rows for snapshot_date=#{snapshot_date}")
+              return
             end
+
+            sql = build_sql(snapshot_date: date, coverage_start: coverage_start)
+            logger.info(
+              "#{LOG_PREFIX} Running test_health_risk aggregation snapshot_date=#{date} " \
+                "coverage_start=#{coverage_start} risk_window=#{risk_window}"
+            )
+            client.query(sql, format: "TabSeparated")
+            report_row_count(date)
           rescue StandardError => e
             logger.error("#{LOG_PREFIX} Aggregation failed for #{snapshot_date}: #{e.message}")
             raise
@@ -72,24 +86,62 @@ module GitlabQuality
 
           private
 
-          attr_reader :url, :database, :username, :password, :logger, :coverage_window, :risk_window
+          attr_reader :url, :database, :username, :password, :logger,
+            :full_capture_floor, :anchor_lookback, :risk_window
+
+          # Most recent day whose capture volume looks like a full sweep of the
+          # suite. maxOrNull (rather than max) distinguishes "no qualifying day"
+          # from a real date when the subquery comes back empty.
+          def find_coverage_start
+            sql = <<~SQL
+              SELECT maxOrNull(day) AS coverage_start
+              FROM (
+                SELECT toDate(timestamp) AS day
+                FROM code_coverage.test_coverage_per_file
+                WHERE timestamp >= now() - INTERVAL #{validate_interval(anchor_lookback)}
+                GROUP BY day
+                HAVING uniqExact(test_file) >= #{validate_positive_integer(full_capture_floor)}
+              )
+            SQL
+            anchor = client.query(sql, format: "JSONCompact")&.dig('data', 0, 0)
+            anchor.nil? ? nil : validate_date(anchor, 'coverage_start')
+          end
+
+          def report_row_count(date)
+            inserted = fetch_row_count(date)
+            if inserted.is_a?(Integer) && inserted.zero?
+              logger.warn(
+                "#{LOG_PREFIX} Aggregation wrote 0 rows for snapshot_date=#{date}. " \
+                  "This is valid if no per-test data landed since the coverage anchor, but worth " \
+                  "checking test_coverage_per_file directly if a recent export ran."
+              )
+            else
+              logger.info("#{LOG_PREFIX} Aggregation wrote #{inserted} rows for snapshot_date=#{date}")
+            end
+          end
 
-          def build_sql(snapshot_date:)
+          def build_sql(snapshot_date:, coverage_start:)
             template = File.read(SQL_FILE)
             template
               .gsub('{snapshot_date}', validate_date(snapshot_date))
-              .gsub('{coverage_window}', validate_interval(coverage_window))
+              .gsub('{coverage_start}', coverage_start)
               .gsub('{risk_window}', validate_interval(risk_window))
           end
 
           # DateTime/Time `to_s` includes the time portion and is rejected.
-          def validate_date(value)
+          def validate_date(value, name = 'snapshot_date')
             str = value.to_s
-            raise ArgumentError, "Invalid snapshot_date: #{value.inspect}" unless DATE_PATTERN.match?(str)
+            raise ArgumentError, "Invalid #{name}: #{value.inspect}" unless DATE_PATTERN.match?(str)
 
             str
           end
 
+          def validate_positive_integer(value)
+            raise ArgumentError, "Invalid full_capture_floor: #{value.inspect}" unless value.is_a?(Integer) && value.positive?
+
+            value.to_s
+          end
+
           def validate_interval(value)
             raise ArgumentError, "Invalid interval expression: #{value.inspect}" unless INTERVAL_PATTERN.match?(value.to_s)
 

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

@@ -23,12 +23,15 @@ module GitlabQuality
       # aggregation once at the end.
       class PerTestCoverageExporter
         # @param clickhouse [Hash] connection params (:url, :database, :username, :password)
+        # @param clickhouse_shared_database [String, nil] the shared database holding the
+        #   canonical `category_owners`. Required only when enriching (test_reports present).
         def initialize(
-          coverage_glob:, clickhouse:,
+          coverage_glob:, clickhouse:, clickhouse_shared_database: nil,
           test_reports: nil, jest_quarantine_file: nil,
           captured_sha: ENV.fetch('CI_COMMIT_SHA', ''), skip_aggregation: false, logger: nil)
           @coverage_glob = coverage_glob
           @clickhouse = clickhouse
+          @clickhouse_shared_database = clickhouse_shared_database
           @test_reports = test_reports
           @jest_quarantine_file = jest_quarantine_file
           @captured_sha = captured_sha.to_s
@@ -65,8 +68,8 @@ module GitlabQuality
 
         private
 
-        attr_reader :coverage_glob, :clickhouse, :test_reports, :jest_quarantine_file,
-          :captured_sha, :skip_aggregation, :logger
+        attr_reader :coverage_glob, :clickhouse, :clickhouse_shared_database, :test_reports,
+          :jest_quarantine_file, :captured_sha, :skip_aggregation, :logger
 
         def insert(coverage_files)
           tests_to_categories, feature_categories_to_teams = resolve_categories
@@ -144,14 +147,21 @@ module GitlabQuality
           [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.
+        # Category to group/stage/section, read from `category_owners` in the shared
+        # database, the same source the full coverage export uses. The shared copy is
+        # refreshed by sync-category-owners; the legacy `code_coverage.category_owners`
+        # stopped updating in December 2025, so reading it would drop every category
+        # added since. `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|
+          if clickhouse_shared_database.to_s.strip.empty?
+            raise ArgumentError,
+              'clickhouse_shared_database is required to enrich per-test rows with category owners'
+          end
+
+          shared_credentials = clickhouse_credentials.merge(database: clickhouse_shared_database)
+
+          ClickHouse::CategoryOwnersTable.new(**shared_credentials).owner_records.transform_values do |owner|
             { group: owner["group"], stage: owner["stage"], section: owner["section"] }
           end
         end

data/lib/gitlab_quality/test_tooling/report/relate_failure_issue.rb

@@ -151,7 +151,6 @@ module GitlabQuality
               line_number: extract_line_number(test),
               exception: extract_exception_data(test),
               ci_job_url: test.ci_job_url,
-              testcase: extract_test_id_or_name(test),
               product_group: extract_product_group(test),
               level: extract_level(test)
             }
@@ -184,14 +183,6 @@ module GitlabQuality
           }
         end
 
-        def extract_test_id_or_name(test)
-          return test.example_id if test.respond_to?(:example_id)
-          return test.id if test.respond_to?(:id)
-          return test.name if test.respond_to?(:name)
-
-          "#{test.relative_file}:#{test.respond_to?(:line_number) ? test.line_number : 'unknown'}"
-        end
-
         def new_issue_labels(test)
           up_to_date_labels(test: test, new_labels: NEW_ISSUE_LABELS + group_and_category_labels_for_test(test))
         end

data/lib/gitlab_quality/test_tooling/report/report_as_issue.rb

@@ -63,7 +63,6 @@ module GitlabQuality
           | Test level | `#{test.level}` |
           | Hash | `#{test_hash(test)}` |
           | Max expected duration | < #{test.max_duration_for_test} seconds |
-          #{"| Test case | #{test.testcase} |" if test.testcase}
           <!-- Don't modify this section as it's auto-generated -->
           DESCRIPTION
         end

data/lib/gitlab_quality/test_tooling/report/report_results.rb

@@ -49,16 +49,11 @@ module GitlabQuality
 
               report_test(test)
             end
-
-            test_results.write
           end
         end
 
         def report_test(test)
           testcase = testcase_project_reporter.find_or_create_testcase(test)
-          # The API returns the test case with an issue URL since it is technically a type of issue.
-          # This updates the URL to a valid test case link.
-          test.testcase = testcase.web_url.sub('/issues/', '/quality/test_cases/')
 
           result_issue, is_new = results_issue_project_reporter.get_related_issue(testcase, test)
 

data/lib/gitlab_quality/test_tooling/report/results_in_test_cases.rb

@@ -18,18 +18,17 @@ module GitlabQuality
           find_testcase(test) || create_issue(test)
         end
 
-        def add_result_issue_link_to_testcase(testcase, result_issue, test)
+        def add_result_issue_link_to_testcase(testcase, result_issue, _test)
           results_section = testcase.description.include?(TEST_CASE_RESULTS_SECTION_TEMPLATE) ? '' : TEST_CASE_RESULTS_SECTION_TEMPLATE
 
           gitlab.edit_issue(iid: testcase.iid,
             options: { description: (testcase.description + results_section + "\n\n#{result_issue.web_url}") })
-          # We are using test.testcase for the url here instead of testcase.web_url since it has the updated test case path
-          puts "Added results issue #{result_issue.web_url} link to test case #{test.testcase}"
+          puts "Added results issue #{result_issue.web_url} link to test case #{testcase.web_url}"
         end
 
         def update_testcase(testcase, test)
-          puts "Labels updated for test case #{test.testcase}." if update_labels(testcase, test)
-          puts "Quarantine section updated for test case #{test.testcase}." if update_quarantine_section(testcase, test)
+          puts "Labels updated for test case #{testcase.web_url}." if update_labels(testcase, test)
+          puts "Quarantine section updated for test case #{testcase.web_url}." if update_quarantine_section(testcase, test)
         end
 
         private

data/lib/gitlab_quality/test_tooling/test_meta/processor/meta_processor.rb

@@ -37,7 +37,7 @@ module GitlabQuality
               description_start_index + description_length
             end
 
-            # List specs in markdown with details such as link to code, testcase, metrics and failure issue
+            # List specs in markdown with details such as link to code, metrics and failure issue
             #
             # @param [Hash<String,Hash>] commits The commits hash to use for spec details
             # @return String
@@ -45,7 +45,7 @@ module GitlabQuality
               commits.each_with_index.map do |(changed_line_number, spec), index|
                 <<~MARKDOWN
                   #{index + 1}. [`#{spec['name']}`](https://gitlab.com/#{context.project}/-/blob/#{context.ref}/#{spec['file_path']}#L#{changed_line_number.to_i + 1})
-                      | [Testcase](#{spec['testcase']}) | [Spec metrics](#{context.single_spec_metrics_link(spec['file_path'])})
+                      | [Spec metrics](#{context.single_spec_metrics_link(spec['file_path'])})
                       #{failure_issue_text(spec)}
                 MARKDOWN
               end.join("\n")

data/lib/gitlab_quality/test_tooling/test_result/j_unit_test_result.rb

@@ -4,8 +4,6 @@ module GitlabQuality
   module TestTooling
     module TestResult
       class JUnitTestResult < BaseTestResult
-        attr_accessor :testcase # Ignore it for now
-
         def name
           report['name']
         end

data/lib/gitlab_quality/test_tooling/test_result/json_test_result.rb

@@ -60,14 +60,6 @@ module GitlabQuality
           report.fetch('ci_job_url', '')
         end
 
-        def testcase
-          report.fetch('testcase', '')
-        end
-
-        def testcase=(new_testcase)
-          report['testcase'] = new_testcase
-        end
-
         def quarantine_type
           quarantine['type'] if quarantine?
         end

data/lib/gitlab_quality/test_tooling/version.rb

@@ -2,6 +2,6 @@
 
 module GitlabQuality
   module TestTooling
-    VERSION = "3.20.2"
+    VERSION = "3.21.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_quality-test_tooling
 version: !ruby/object:Gem::Version
-  version: 3.20.2
+  version: 3.21.1
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2026-07-16 00:00:00.000000000 Z
+date: 2026-08-26 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control