gitlab_quality-test_tooling 3.3.0 → 3.5.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 32b52c2177aaf57f1ccf6a2bb922fd47e267b251ce74921c356f9631252051da
-  data.tar.gz: 8930c0fb427f3a946cd99902a230bcf90c48a051c9ffa1d19fb10a487107d231
+  metadata.gz: 0afeda440d33c3cb9e1e81adf43ce462d12b604cffdf96b04dbe83a9a4d1f853
+  data.tar.gz: 1a638b0a0b409ed28cb94da821a59f73967b6ae55cad7e87552a096b9c5c525e
 SHA512:
-  metadata.gz: f265a7b7c74d4ef36e06c53a945b96ab066a4304acd0e6c439cf10afee6d0483de9253ca9290b279db7ec3c1d2323146da3aea8cf92aa57d4d1914c9c1479b1f
-  data.tar.gz: b983b542d66e9ac5b5775dd97b80ce6b464a618a5adf573f18613014dcfecc86555fc8b7d20e32b33b41b23ce9be255811cc276a50a1ee7945f0e4062deb15cf
+  metadata.gz: 5bece6881a2ab2506f4d22b7da49b976af027a0f7d93c276d704f37fb52091ddde319232db0ccca1b717141267bbd5e283f30796a8bc06b14dbbbd38115327ca
+  data.tar.gz: 7136b103704d811fcdaee9a44ee815775d2eb0411934654131d6478a8d2b62c3225a94a5602de169284cd3121a1b464bcbcf65570a5f7f923add0cdb0b5b386a

data/Gemfile.lock

@@ -1,13 +1,12 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (3.3.0)
+    gitlab_quality-test_tooling (3.5.0)
       activesupport (>= 7.0)
       amatch (~> 0.4.1)
       fog-google (~> 1.24, >= 1.24.1)
       gitlab (>= 4.19, < 7.0)
       http (~> 5.0)
-      influxdb-client (~> 3.1)
       nokogiri (~> 1.10)
       parallel (>= 1, < 2)
       rainbow (>= 3, < 4)
@@ -200,7 +199,6 @@ GEM
     httpclient (2.8.3)
     i18n (1.14.6)
       concurrent-ruby (~> 1.0)
-    influxdb-client (3.1.0)
     jaro_winkler (1.6.0)
     json (2.7.2)
     jwt (2.9.3)

data/README.md

@@ -185,20 +185,6 @@ Usage: exe/failed-test-issues [options]
     -h, --help                       Show the usage
 ```
 
-### `exe/existing-test-health-issue`
-
-```shell
-Purpose: Checks whether tests coming from the rspec JSON report files has an existing test health issue opened.
-Usage: exe/existing-test-health-issue [options]
-    -i, --input-files INPUT_FILES    JSON rspec-retry report files
-    -p, --project PROJECT            Can be an integer or a group/project string
-    -t, --token TOKEN                A valid access token with `api` scope and Maintainer permission in PROJECT
-        --health-problem-type PROBLEM_TYPE
-                                     Look for the given health problem type (failures, pass-after-retry, slow)
-    -v, --version                    Show the version
-    -h, --help                       Show the usage
-```
-
 ### `exe/detect-infrastructure-failures`
 
 ```shell

data/exe/test-coverage

@@ -55,6 +55,10 @@ options = OptionParser.new do |opts|
     params[:clickhouse_username] = username
   end
 
+  opts.on('--clickhouse-shared-database DATABASE', 'ClickHouse shared database name (default: shared)') do |database|
+    params[:clickhouse_shared_database] = database
+  end
+
   opts.on('--responsibility-patterns PATH', 'Path to YAML file with responsibility classification patterns') do |path|
     params[:responsibility_patterns] = path
   end
@@ -166,7 +170,14 @@ if params.any? && (required_params - params.keys).none?
     password: clickhouse_password
   }
 
-  category_owners_table  = GitlabQuality::TestTooling::CodeCoverage::ClickHouse::CategoryOwnersTable.new(**clickhouse_data)
+  shared_clickhouse_data = {
+    url: params[:clickhouse_url],
+    database: params[:clickhouse_shared_database] || 'shared',
+    username: params[:clickhouse_username],
+    password: clickhouse_password
+  }
+
+  category_owners_table  = GitlabQuality::TestTooling::CodeCoverage::ClickHouse::CategoryOwnersTable.new(**shared_clickhouse_data)
   coverage_metrics_table = GitlabQuality::TestTooling::CodeCoverage::ClickHouse::CoverageMetricsTable.new(**clickhouse_data)
 
   if ENV['CLICKHOUSE_PUSH_CATEGORY_DATA'] == 'true'
@@ -178,7 +189,7 @@ if params.any? && (required_params - params.keys).none?
 
   # Export test-to-file mappings
   test_file_mapping_data = GitlabQuality::TestTooling::CodeCoverage::TestFileMappingData.new(test_to_sources)
-  test_file_mappings_table = GitlabQuality::TestTooling::CodeCoverage::ClickHouse::TestFileMappingsTable.new(**clickhouse_data)
+  test_file_mappings_table = GitlabQuality::TestTooling::CodeCoverage::ClickHouse::TestFileMappingsTable.new(**shared_clickhouse_data)
   test_file_mappings_table.push(test_file_mapping_data.as_db_table)
 else
   puts "Missing argument(s). Required arguments are: #{required_params}\nPassed arguments are: #{params}\n"

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

@@ -11,6 +11,8 @@ module GitlabQuality
 
           MissingMappingError = Class.new(StandardError)
 
+          KNOWN_UNOWNED = %w[shared not_owned tooling].freeze
+
           def truncate
             logger.debug("#{LOG_PREFIX} Truncating table #{full_table_name} ...")
 
@@ -24,6 +26,13 @@ module GitlabQuality
           # @param feature_category_name [String] the feature_category name
           # @return [Hash]
           def owners(feature_category_name)
+            if KNOWN_UNOWNED.include?(feature_category_name)
+              logger.info(
+                "#{LOG_PREFIX} #{feature_category_name} is a known feature category without owner..."
+              )
+              return {}
+            end
+
             records.fetch(feature_category_name)
           rescue KeyError
             raise(MissingMappingError, "Feature category '#{feature_category_name}' not found in table '#{table_name}'")

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

@@ -1,7 +1,5 @@
 # frozen_string_literal: true
 
-require 'erb'
-
 module GitlabQuality
   module TestTooling
     module Report
@@ -68,12 +66,12 @@ module GitlabQuality
         end
 
         def execution_graph_section(test)
-          formatted_title = ERB::Util.url_encode(test.name)
+          formatted_path = CGI.escape(test.relative_file)
 
           <<~MKDOWN.strip
             ### Executions
 
-            [Spec metrics on all environments](https://dashboards.quality.gitlab.net/d/cW0UMgv7k/single-spec-metrics?orgId=1&var-run_type=All&var-name=#{formatted_title})
+            [Spec metrics on all environments](https://dashboards.devex.gitlab.net/d/739c1bdd-a436-452b-bddc-fccb4d055768/single-test-overview?var-file_path=#{formatted_path})
           MKDOWN
         end
 

data/lib/gitlab_quality/test_tooling/runtime/env.rb

@@ -103,6 +103,10 @@ module GitlabQuality
           "#{ci_project_name}-#{test_subset}"
         end
 
+        def quarantine_disabled?
+          enabled?(ENV.fetch('GLCI_DISABLE_QUARANTINE', nil), default: false)
+        end
+
         private
 
         def enabled?(value, default: true)

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

@@ -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['name'])})
+                      | [Testcase](#{spec['testcase']}) | [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_meta/test_meta_updater.rb

@@ -348,11 +348,11 @@ module GitlabQuality
 
         # Returns the link to the Grafana dashboard for single spec metrics
         #
-        # @param [String] example_name the full example name
+        # @param [String] file_path the full path of spec
         # @return [String]
-        def single_spec_metrics_link(example_name)
-          base_url = "https://dashboards.quality.gitlab.net/d/cW0UMgv7k/single-spec-metrics?orgId=1&var-run_type=All&var-name="
-          base_url + CGI.escape(example_name)
+        def single_spec_metrics_link(file_path)
+          base_url = "https://dashboards.devex.gitlab.net/d/739c1bdd-a436-452b-bddc-fccb4d055768/single-test-overview?var-file_path="
+          base_url + CGI.escape(file_path)
         end
 
         # Returns any test description string within single or double quotes

data/lib/gitlab_quality/test_tooling/test_metrics_exporter/test_metrics.rb

@@ -51,6 +51,7 @@ module GitlabQuality
             exception_classes: exception_classes.map { |e| e.class.to_s }.uniq,
             failure_exception: failure_exception,
             quarantined: quarantined?,
+            quarantine_issue_url: quarantine_issue_url || "",
             feature_category: example.metadata[:feature_category] || "",
             test_retried: config.test_retried_proc.call(example),
             run_type: run_type,
@@ -98,6 +99,25 @@ module GitlabQuality
           example.execution_result.status == :pending
         end
 
+        # Extract quarantine issue URL from metadata
+        #
+        # @return [String, nil]
+        def quarantine_issue_url
+          return nil unless example.metadata.key?(:quarantine)
+
+          metadata = example.metadata[:quarantine]
+          case metadata
+          when String
+            # Direct URL: quarantine: 'https://gitlab.com/.../issues/123'
+            metadata if metadata.start_with?('http')
+          when Hash
+            # Hash format: quarantine: { issue: 'https://...', reason: '...' }
+            issue = metadata[:issue] || metadata['issue']
+            # Handle array of URLs (take the first one)
+            issue.is_a?(Array) ? issue.first : issue
+          end
+        end
+
         # Base ci job name
         #
         # @return [String]

data/lib/gitlab_quality/test_tooling/test_quarantine/quarantine_formatter.rb

@@ -0,0 +1,38 @@
+# frozen_string_literal: true
+
+require "rspec/core/formatters/base_formatter"
+
+module GitlabQuality
+  module TestTooling
+    module TestQuarantine
+      class QuarantineFormatter < ::RSpec::Core::Formatters::BaseFormatter
+        include QuarantineHelper
+
+        ::RSpec::Core::Formatters.register(
+          self,
+          :example_group_started,
+          :example_started
+        )
+
+        # Starts example group
+        # @param [RSpec::Core::Notifications::GroupNotification] example_group_notification
+        # @return [void]
+        def example_group_started(example_group_notification)
+          group = example_group_notification.group
+
+          skip_or_run_quarantined_tests_or_contexts(group)
+        end
+
+        # Starts example
+        # @param [RSpec::Core::Notifications::ExampleNotification] example_notification
+        # @return [void]
+        def example_started(example_notification)
+          example = example_notification.example
+
+          # if skip propagated from example_group, do not reset skip metadata
+          skip_or_run_quarantined_tests_or_contexts(example) unless example.metadata[:skip]
+        end
+      end
+    end
+  end
+end

data/lib/gitlab_quality/test_tooling/test_quarantine/quarantine_helper.rb

@@ -0,0 +1,76 @@
+# frozen_string_literal: true
+
+require 'rspec/core'
+
+module GitlabQuality
+  module TestTooling
+    module TestQuarantine
+      module QuarantineHelper
+        include ::RSpec::Core::Pending
+
+        extend self
+
+        # Skip tests in quarantine unless we explicitly focus on them or quarantine disabled
+        def skip_or_run_quarantined_tests_or_contexts(example)
+          return if Runtime::Env.quarantine_disabled?
+
+          if filters.key?(:quarantine)
+            included_filters = filters_other_than_quarantine
+
+            # If :quarantine is focused, skip the test/context unless its metadata
+            # includes quarantine and any other filters
+            # E.g., Suppose a test is tagged :smoke and :quarantine, and another is tagged
+            # :ldap and :quarantine. If we wanted to run just quarantined smoke tests
+            # using `--tag quarantine --tag smoke`, without this check we'd end up
+            # running that ldap test as well because of the :quarantine metadata.
+            # We could use an exclusion filter, but this way the test report will list
+            # the quarantined tests when they're not run so that we're aware of them
+            if should_skip_when_focused?(example.metadata, included_filters)
+              example.metadata[:skip] = "Only running tests tagged with :quarantine and any of #{included_filters.keys}"
+            end
+          elsif example.metadata.key?(:quarantine)
+            quarantine_tag = example.metadata[:quarantine]
+
+            example.metadata[:skip] = quarantine_message(quarantine_tag)
+          end
+        end
+
+        def filters_other_than_quarantine
+          filters.except(:quarantine)
+        end
+
+        def quarantine_message(quarantine_tag)
+          quarantine_message = %w[In quarantine]
+          quarantine_message << case quarantine_tag
+                                when String
+                                  ": #{quarantine_tag}"
+                                when Hash
+                                  quarantine_tag.key?(:issue) ? ": #{quarantine_tag[:issue]}" : ''
+                                else
+                                  ''
+                                end
+
+          quarantine_message.join(' ').strip
+        end
+
+        # Checks if a test or context should be skipped.
+        #
+        # Returns true if
+        # - the metadata does not includes the :quarantine tag
+        # or if
+        # - the metadata includes the :quarantine tag
+        # - and the filter includes other tags that aren't in the metadata
+        def should_skip_when_focused?(metadata, included_filters)
+          return true unless metadata.key?(:quarantine)
+          return false if included_filters.empty?
+
+          !metadata.keys.intersect?(included_filters.keys)
+        end
+
+        def filters
+          @filters ||= ::RSpec.configuration.inclusion_filter.rules
+        end
+      end
+    end
+  end
+end

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

@@ -14,6 +14,7 @@ module GitlabQuality
           "could not be found (502)",
           "Error reference number: 502",
           "(502): `GitLab is not responding`",
+          "(502): `502 Bad Gateway`",
           "<head><title>502 Bad Gateway</title></head>",
           "14:connections to all backends failing",
           "gitlab_canary=true cookie was set in browser but 'Next' badge was not shown on UI"

data/lib/gitlab_quality/test_tooling/version.rb

@@ -2,6 +2,6 @@
 
 module GitlabQuality
   module TestTooling
-    VERSION = "3.3.0"
+    VERSION = "3.5.0"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_quality-test_tooling
 version: !ruby/object:Gem::Version
-  version: 3.3.0
+  version: 3.5.0
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2025-12-18 00:00:00.000000000 Z
+date: 2026-01-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control
@@ -302,20 +302,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '5.0'
-- !ruby/object:Gem::Dependency
-  name: influxdb-client
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
-  type: :runtime
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
 - !ruby/object:Gem::Dependency
   name: nokogiri
   requirement: !ruby/object:Gem::Requirement
@@ -430,12 +416,10 @@ email:
 executables:
 - detect-infrastructure-failures
 - epic-readiness-notification
-- existing-test-health-issue
 - failed-test-issues
 - feature-readiness-checklist
 - feature-readiness-evaluation
 - flaky-test-issues
-- generate-test-session
 - knapsack-report-issues
 - post-to-slack
 - prepare-stage-reports
@@ -465,12 +449,10 @@ files:
 - Rakefile
 - exe/detect-infrastructure-failures
 - exe/epic-readiness-notification
-- exe/existing-test-health-issue
 - exe/failed-test-issues
 - exe/feature-readiness-checklist
 - exe/feature-readiness-evaluation
 - exe/flaky-test-issues
-- exe/generate-test-session
 - exe/knapsack-report-issues
 - exe/post-to-slack
 - exe/prepare-stage-reports
@@ -540,7 +522,6 @@ files:
 - lib/gitlab_quality/test_tooling/report/failed_test_issue.rb
 - lib/gitlab_quality/test_tooling/report/feature_readiness/report_on_epic.rb
 - lib/gitlab_quality/test_tooling/report/flaky_test_issue.rb
-- lib/gitlab_quality/test_tooling/report/generate_test_session.rb
 - lib/gitlab_quality/test_tooling/report/group_issues/error_message_normalizer.rb
 - lib/gitlab_quality/test_tooling/report/group_issues/error_pattern_matcher.rb
 - lib/gitlab_quality/test_tooling/report/group_issues/failure_processor.rb
@@ -563,7 +544,6 @@ files:
 - lib/gitlab_quality/test_tooling/report/results_in_issues.rb
 - lib/gitlab_quality/test_tooling/report/results_in_test_cases.rb
 - lib/gitlab_quality/test_tooling/report/slow_test_issue.rb
-- lib/gitlab_quality/test_tooling/report/test_health_issue_finder.rb
 - lib/gitlab_quality/test_tooling/report/update_screenshot_path.rb
 - lib/gitlab_quality/test_tooling/runtime/env.rb
 - lib/gitlab_quality/test_tooling/runtime/logger.rb
@@ -594,6 +574,8 @@ files:
 - lib/gitlab_quality/test_tooling/test_metrics_exporter/formatter.rb
 - lib/gitlab_quality/test_tooling/test_metrics_exporter/test_metrics.rb
 - lib/gitlab_quality/test_tooling/test_metrics_exporter/utils.rb
+- lib/gitlab_quality/test_tooling/test_quarantine/quarantine_formatter.rb
+- lib/gitlab_quality/test_tooling/test_quarantine/quarantine_helper.rb
 - lib/gitlab_quality/test_tooling/test_result/base_test_result.rb
 - lib/gitlab_quality/test_tooling/test_result/j_unit_test_result.rb
 - lib/gitlab_quality/test_tooling/test_result/json_test_result.rb

data/exe/existing-test-health-issue

@@ -1,59 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-require "bundler/setup"
-require "optparse"
-
-require_relative "../lib/gitlab_quality/test_tooling"
-
-params = {}
-HEALTH_PROBLEM_TYPES = GitlabQuality::TestTooling::Report::TestHealthIssueFinder::HEALTH_PROBLEM_TYPE_TO_LABEL.keys
-
-options = OptionParser.new do |opts|
-  opts.banner = "Usage: #{$PROGRAM_NAME} [options]"
-
-  opts.on('-i', '--input-files INPUT_FILES', String, 'JSON rspec-retry report files') do |input_files|
-    params[:input_files] = input_files
-  end
-
-  opts.on('-p', '--project PROJECT', String, 'Can be an integer or a group/project string') do |project|
-    params[:project] = project
-  end
-
-  opts.on('-t', '--token TOKEN', String, 'A valid access token with `api` scope and Maintainer permission in PROJECT') do |token|
-    params[:token] = token
-  end
-
-  opts.on("--health-problem-type PROBLEM_TYPE", String, "Look for the given health problem type (#{HEALTH_PROBLEM_TYPES.join(', ')})") do |value|
-    raise ArgumentError, "Invalid health problem type: #{value}. Valid options are: #{HEALTH_PROBLEM_TYPES.join(', ')}" unless HEALTH_PROBLEM_TYPES.include?(value)
-
-    params[:health_problem_type] = value
-  end
-
-  opts.on_tail('-v', '--version', 'Show the version') do
-    require_relative "../lib/gitlab_quality/test_tooling/version"
-    puts "#{$PROGRAM_NAME} : #{GitlabQuality::TestTooling::VERSION}"
-    exit
-  end
-
-  opts.on_tail('-h', '--help', 'Show the usage') do
-    puts "Purpose: Checks whether tests coming from the rspec JSON report files has an existing test health issue opened."
-    puts opts
-    exit
-  end
-
-  opts.parse(ARGV)
-end
-
-if params.any?
-  raise ArgumentError, "No health problem type given. Valid options are: #{HEALTH_PROBLEM_TYPES.join(', ')}" unless params.key?(:health_problem_type)
-
-  if GitlabQuality::TestTooling::Report::TestHealthIssueFinder.new(**params).found_existing_unhealthy_test_issue?
-    exit 0
-  else
-    exit 1
-  end
-else
-  puts options
-  exit 1
-end

data/exe/generate-test-session

@@ -1,70 +0,0 @@
-#!/usr/bin/env ruby
-# frozen_string_literal: true
-
-require "bundler/setup"
-require "optparse"
-
-require_relative "../lib/gitlab_quality/test_tooling"
-
-params = {}
-
-options = OptionParser.new do |opts|
-  opts.banner = "Usage: #{$PROGRAM_NAME} [options]"
-
-  opts.on('-i', '--input-files INPUT_FILES', String, 'RSpec report files (JSON or JUnit XML)') do |input_files|
-    params[:input_files] = input_files
-  end
-
-  opts.on('-p', '--project PROJECT', String, 'Can be an integer or a group/project string') do |project|
-    params[:project] = project
-  end
-
-  opts.on('-t', '--token TOKEN', String, 'A valid access token with `api` scope and Reporter permission in PROJECT') do |token|
-    params[:token] = token
-  end
-
-  opts.on('-c', '--ci-project-token CI_PROJECT_TOKEN', String, 'A valid access token with `read_api` scope permission in current ENV["CI_PROJECT_ID"]') do |ci_project_token|
-    params[:ci_project_token] = ci_project_token
-  end
-
-  opts.on('-f', '--issue-url-file ISSUE_URL_FILE', 'Output the created test session issue URL') do |issue_url_file|
-    params[:issue_url_file] = issue_url_file
-  end
-
-  opts.on('--pipeline-stages STAGES', String, 'Comma-separated list of pipeline stages to include in test session issue') do |pipeline_stages|
-    params[:pipeline_stages] = pipeline_stages.split(',')
-  end
-
-  opts.on('--confidential', "Makes test session issue confidential") do
-    params[:confidential] = true
-  end
-
-  opts.on('--dry-run', "Perform a dry-run (don't create or update issues or test cases)") do
-    params[:dry_run] = true
-  end
-
-  opts.on_tail('-v', '--version', 'Show the version') do
-    require_relative "../lib/gitlab_quality/test_tooling/version"
-    puts "#{$PROGRAM_NAME} : #{GitlabQuality::TestTooling::VERSION}"
-    exit
-  end
-
-  opts.on_tail('-h', '--help', 'Show the usage') do
-    puts "Purpose: Generate test session report based on RSpec report files (JSON or JUnit XML)"
-    puts opts
-    exit
-  end
-
-  opts.parse(ARGV)
-end
-
-issue_url_file = params.delete(:issue_url_file)
-
-if params.any?
-  issue_url = GitlabQuality::TestTooling::Report::GenerateTestSession.new(**params).invoke!
-
-  File.write(issue_url_file, issue_url) if issue_url_file && issue_url
-else
-  puts options
-  exit 1
-end

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

@@ -1,288 +0,0 @@
-# frozen_string_literal: true
-
-require 'erb'
-require 'date'
-
-module GitlabQuality
-  module TestTooling
-    module Report
-      class GenerateTestSession < ReportAsIssue
-        def initialize(ci_project_token:, pipeline_stages: nil, **kwargs)
-          super
-          @ci_project_token = ci_project_token
-          @pipeline_stages = Set.new(pipeline_stages)
-          @issue_type = 'issue'
-        end
-
-        private
-
-        attr_reader :ci_project_token, :pipeline_stages
-
-        # rubocop:disable Metrics/AbcSize
-        def run!
-          puts "Generating test results in `#{files.join(',')}` as issues in project `#{project}` via the API at `#{Runtime::Env.gitlab_api_base}`."
-
-          tests = Dir.glob(files).flat_map do |path|
-            puts "Loading tests in #{path}"
-
-            TestResults::JsonTestResults.new(path: path).to_a
-          end
-
-          tests = tests.select { |test| pipeline_stages.include? test.report["stage"] } unless pipeline_stages.empty?
-
-          issue = gitlab.create_issue(
-            title: "#{Time.now.to_date.iso8601} Test session report | #{Runtime::Env.qa_run_type}",
-            description: generate_description(tests),
-            labels: ['automation:bot-authored', 'E2E', 'triage report', pipeline_name_label, 'suppress-contributor-links'],
-            confidential: confidential
-          )
-
-          # Workaround for https://gitlab.com/gitlab-org/gitlab/-/issues/295493
-          unless Runtime::Env.qa_issue_url.to_s.empty?
-            gitlab.create_issue_note(
-              iid: issue.iid,
-              note: "/relate #{Runtime::Env.qa_issue_url}")
-          end
-
-          issue&.web_url # Issue isn't created in dry-run mode
-        end
-        # rubocop:enable Metrics/AbcSize
-
-        def generate_description(tests)
-          <<~MARKDOWN.rstrip
-          ## Session summary
-
-          * Deploy version: #{Runtime::Env.deploy_version}
-          * Deploy environment: #{Runtime::Env.deploy_environment}
-          * Pipeline: #{Runtime::Env.pipeline_from_project_name} [#{Runtime::Env.ci_pipeline_id}](#{Runtime::Env.ci_pipeline_url})
-          #{generate_summary(tests: tests)}
-
-          #{generate_failed_jobs_listing}
-
-          #{generate_stages_listing(tests)}
-
-          #{generate_qa_issue_relation}
-
-          #{generate_link_to_dashboard}
-          MARKDOWN
-        end
-
-        def generate_summary(tests:, tests_by_status: nil)
-          tests_by_status ||= tests.group_by(&:status)
-          total = tests.size
-          passed = tests_by_status['passed']&.size || 0
-          failed = tests_by_status['failed']&.size || 0
-          others = total - passed - failed
-
-          <<~MARKDOWN.chomp
-          * Total #{total} tests
-          * Passed #{passed} tests
-          * Failed #{failed} tests
-          * #{others} other tests (usually skipped)
-          MARKDOWN
-        end
-
-        def generate_failed_jobs_listing
-          failed_jobs = fetch_pipeline_failed_jobs
-          listings = failed_jobs.filter_map do |job|
-            next if pipeline_stages.any? && !pipeline_stages.include?(job.stage)
-
-            allowed_to_fail = ' (allowed to fail)' if job.allow_failure
-
-            "* [#{job.name}](#{job.web_url})#{allowed_to_fail}"
-          end.join("\n")
-
-          <<~MARKDOWN.chomp if failed_jobs.any?
-          ## Failed jobs
-
-          #{listings}
-          MARKDOWN
-        end
-
-        def generate_stages_listing(tests)
-          generate_tests_by_stage(tests).map do |stage, tests_for_stage|
-            tests_by_status = tests_for_stage.group_by(&:status)
-
-            <<~MARKDOWN.chomp
-            ### #{stage&.capitalize || 'Unknown'}
-
-            #{generate_summary(
-              tests: tests_for_stage, tests_by_status: tests_by_status)}
-
-            #{generate_testcase_listing_by_status(
-              tests: tests_for_stage, tests_by_status: tests_by_status)}
-            MARKDOWN
-          end.join("\n\n")
-        end
-
-        def generate_tests_by_stage(tests)
-          # https://about.gitlab.com/handbook/product/product-categories/#devops-stages
-          ordering = %w[
-            manage
-            plan
-            create
-            verify
-            package
-            release
-            configure
-            monitor
-            secure
-            defend
-            growth
-            fulfillment
-            enablement
-            self-managed
-            saas
-          ]
-
-          tests.sort_by do |test|
-            ordering.index(test.stage) || ordering.size
-          end.group_by(&:stage)
-        end
-
-        def generate_testcase_listing_by_status(tests:, tests_by_status:)
-          failed_tests = tests_by_status['failed']
-          passed_tests = tests_by_status['passed']
-          other_tests = tests.reject do |test|
-            test.status == 'failed' || test.status == 'passed'
-          end
-
-          [
-            (failed_listings(failed_tests) if failed_tests),
-            (passed_listings(passed_tests) if passed_tests),
-            (other_listings(other_tests) if other_tests.any?)
-          ].compact.join("\n\n")
-        end
-
-        def failed_listings(failed_tests)
-          generate_testcase_listing(failed_tests)
-        end
-
-        def passed_listings(passed_tests)
-          <<~MARKDOWN.chomp
-            <details><summary>Passed tests:</summary>
-
-            #{generate_testcase_listing(passed_tests, passed: true)}
-
-            </details>
-          MARKDOWN
-        end
-
-        def other_listings(other_tests)
-          <<~MARKDOWN.chomp
-            <details><summary>Other tests:</summary>
-
-            #{generate_testcase_listing(other_tests)}
-
-            </details>
-          MARKDOWN
-        end
-
-        def generate_testcase_listing(tests, passed: false)
-          body = tests.group_by(&:testcase).map do |testcase, tests_with_same_testcase|
-            tests_with_same_testcase.sort_by!(&:name)
-            [
-              generate_test_text(testcase, tests_with_same_testcase, passed),
-              generate_test_job(tests_with_same_testcase),
-              generate_test_status(tests_with_same_testcase),
-              generate_test_actions(tests_with_same_testcase)
-            ].join(' | ')
-          end.join("\n")
-
-          <<~MARKDOWN.chomp
-          | Test | Job | Status | Action |
-          | - | - | - | - |
-          #{body}
-          MARKDOWN
-        end
-
-        def generate_test_text(testcase, tests_with_same_testcase, passed)
-          text = tests_with_same_testcase.map(&:name).uniq.join(', ')
-
-          if testcase && !passed
-            "[#{text}](#{testcase})"
-          else
-            text
-          end
-        end
-
-        def generate_test_job(tests_with_same_testcase)
-          tests_with_same_testcase.map do |test|
-            ci_job_id = test.ci_job_url[/\d+\z/]
-
-            "[#{ci_job_id}](#{test.ci_job_url})#{' ~"quarantine"' if test.quarantine?}"
-          end.uniq.join(', ')
-        end
-
-        def generate_test_status(tests_with_same_testcase)
-          tests_with_same_testcase.map(&:status).uniq.map do |status|
-            %(~"#{status}")
-          end.join(', ')
-        end
-
-        def generate_test_actions(tests_with_same_testcase)
-          # All failed tests would be grouped together, meaning that
-          # if one failed, all the tests here would be failed too.
-          # So this check is safe. Same applies to 'passed'.
-          # But all other status might be mixing together,
-          # we cannot assume other statuses.
-          if tests_with_same_testcase.first.status == 'failed'
-            tests_having_failure_issue =
-              tests_with_same_testcase.select(&:failure_issue)
-
-            if tests_having_failure_issue.any?
-              items = tests_having_failure_issue.uniq(&:failure_issue).map do |test|
-                "<li>[ ] [failure issue](#{test.failure_issue})</li>"
-              end.join(' ')
-
-              "<ul>#{items}</ul>"
-            else
-              '<ul><li>[ ] failure issue exists or was created</li></ul>'
-            end
-          else
-            '-'
-          end
-        end
-
-        def generate_qa_issue_relation
-          return unless Runtime::Env.qa_issue_url
-
-          <<~MARKDOWN.chomp
-          ## Release QA issue
-
-          * #{Runtime::Env.qa_issue_url}
-
-          /relate #{Runtime::Env.qa_issue_url}
-          MARKDOWN
-        end
-
-        def generate_link_to_dashboard
-          return unless Runtime::Env.qa_run_type
-
-          <<~MARKDOWN.chomp
-          ## Link to Grafana dashboard for run-type of #{Runtime::Env.qa_run_type}
-
-          * https://dashboards.quality.gitlab.net/d/tR_SmBDVk/main-runs?orgId=1&refresh=1m&var-run_type=#{Runtime::Env.qa_run_type}
-          MARKDOWN
-        end
-
-        def fetch_pipeline_failed_jobs
-          failed_jobs = []
-
-          ci_project_client = Gitlab.client(
-            endpoint: Runtime::Env.ci_api_v4_url,
-            private_token: ci_project_token)
-
-          gitlab.handle_gitlab_client_exceptions do
-            failed_jobs = ci_project_client.pipeline_jobs(
-              Runtime::Env.ci_project_id,
-              Runtime::Env.ci_pipeline_id,
-              scope: 'failed')
-          end
-
-          failed_jobs
-        end
-      end
-    end
-  end
-end

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

@@ -1,79 +0,0 @@
-# frozen_string_literal: true
-
-require 'http'
-require 'json'
-
-module GitlabQuality
-  module TestTooling
-    module Report
-      class TestHealthIssueFinder < ReportAsIssue
-        HEALTH_PROBLEM_TYPE_TO_LABEL = {
-          'pass-after-retry' => 'test-health:pass-after-retry',
-          'slow' => 'test-health:slow',
-          'failures' => 'test-health:failures'
-        }.freeze
-
-        def initialize(health_problem_type: [], **kwargs)
-          super(**kwargs)
-
-          @health_problem_type = health_problem_type
-        end
-
-        def found_existing_unhealthy_test_issue?
-          issue_url = invoke!
-
-          !issue_url.nil? && !issue_url.empty?
-        end
-
-        def run!
-          existing_issue_found = nil
-
-          applicable_tests.each do |test|
-            issues = find_issues_by_hash(test_hash(test), state: 'opened', labels: search_labels)
-            next if issues.empty?
-
-            existing_issue_found = issues.first.web_url
-            puts "Found an existing test health issue of type #{health_problem_type} for test #{test.file}:#{test.line_number}: #{existing_issue_found}."
-            break
-          end
-
-          puts "Did not find an existing test health issue of type #{health_problem_type}." unless existing_issue_found
-
-          existing_issue_found
-        end
-
-        def applicable_tests
-          applicable_tests = []
-
-          TestResults::Builder.new(file_glob: files, token: token, project: project).test_results_per_file do |test_results|
-            applicable_tests = test_results.select { |test| test_is_applicable?(test) }
-          end
-
-          applicable_tests
-        end
-
-        private
-
-        attr_reader :health_problem_type
-
-        # Be mindful about the number of tests this method would return,
-        # as we will make at least one API request per test.
-        def test_is_applicable?(test)
-          expected_test_status =
-            case health_problem_type
-            when 'failures'
-              'failed'
-            else
-              'passed'
-            end
-
-          test.status == expected_test_status
-        end
-
-        def search_labels
-          ['test', HEALTH_PROBLEM_TYPE_TO_LABEL[health_problem_type]]
-        end
-      end
-    end
-  end
-end