gitlab_quality-test_tooling 1.14.0 → 1.17.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 970354551e0118a942865386f3e844762cf1fdfb06cfc02aa7bca602e55abc8f
-  data.tar.gz: f512f45f80e5a3949b07a237c1f29c6bc73fcd8e320a92b6b3fcd0a23ccc311f
+  metadata.gz: b8709476aafe6dc96d2b49d1d74b18b2e3dbf6c37cb8ebb0acd631d05dbfe2ac
+  data.tar.gz: 26d4d54613a522014b5f3b93f38c0cb62027c2e209573727265a5c2b630a6d9d
 SHA512:
-  metadata.gz: 639c54db934d3fb5ee5cc57264957a6ffff2714fff9dce6517701b7ff4e5ce74fca87a518f35b1e8d52852952d875f6b9aa45e79a506fbb9416d9712bba2ab95
-  data.tar.gz: 1dbfcfab9f4161827ae2775a69f6203008cc4ea07dc23424a971c5451a354ebb28d7a698e922aecf8ca321a9612a05e5adf966c1bc0e38686c249aaa87ce6446
+  metadata.gz: 194294ffed245e88a96e35480485f98c1748a8417766cdca1cf737ece2b090a4e29efc444fa449f5f569e292333ba50954f7c07a72fbb2f9225c9195c4b6f411
+  data.tar.gz: 30cc7f3a5da1c15b5c3508e0850c007bddf9f22a85c9ec1361821be7bc330afd3869cd9fe74d3b424afc2d0ad2d180c4c0649b5f502f4740ca68f1b2c8119933

data/Gemfile.lock

@@ -1,8 +1,8 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (1.14.0)
-      activesupport (>= 6.1, < 7.2)
+    gitlab_quality-test_tooling (1.17.0)
+      activesupport (>= 6.1, < 7.1)
       amatch (~> 0.4.1)
       gitlab (~> 4.19)
       http (~> 5.0)
@@ -21,15 +21,14 @@ GEM
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)
       tzinfo (~> 2.0)
-      zeitwerk (~> 2.3)
-    addressable (2.8.4)
+    addressable (2.8.6)
       public_suffix (>= 2.0.2, < 6.0)
     amatch (0.4.1)
       mize
       tins (~> 1.0)
     ast (2.4.2)
     backport (1.2.0)
-    benchmark (0.2.1)
+    benchmark (0.3.0)
     binding_of_caller (1.0.0)
       debug_inspector (>= 0.0.1)
     byebug (11.1.3)
@@ -41,23 +40,23 @@ GEM
     climate_control (1.2.0)
     coderay (1.1.3)
     colored2 (3.1.2)
-    concurrent-ruby (1.2.2)
+    concurrent-ruby (1.2.3)
     cork (0.3.0)
       colored2 (~> 3.1)
     crack (0.4.5)
       rexml
-    danger (9.3.0)
+    danger (9.4.2)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
       cork (~> 0.1)
       faraday (>= 0.9.0, < 3.0)
       faraday-http-cache (~> 2.0)
-      git (~> 1.13.0)
+      git (~> 1.13)
       kramdown (~> 2.3)
       kramdown-parser-gfm (~> 1.0)
       no_proxy_fix
-      octokit (~> 5.0)
+      octokit (>= 4.0)
       terminal-table (>= 1, < 4)
     danger-gitlab (8.0.0)
       danger
@@ -65,37 +64,36 @@ GEM
     debug_inspector (1.2.0)
     diff-lcs (1.5.0)
     docile (1.4.0)
-    domain_name (0.5.20190701)
-      unf (>= 0.0.5, < 1.0.0)
+    domain_name (0.6.20240107)
     e2mmap (0.1.0)
-    faraday (2.7.4)
-      faraday-net_http (>= 2.0, < 3.1)
-      ruby2_keywords (>= 0.0.4)
-    faraday-http-cache (2.5.0)
+    faraday (2.9.0)
+      faraday-net_http (>= 2.0, < 3.2)
+    faraday-http-cache (2.5.1)
       faraday (>= 0.8)
-    faraday-net_http (3.0.2)
-    ffi (1.15.5)
+    faraday-net_http (3.1.0)
+      net-http
+    ffi (1.16.3)
     ffi-compiler (1.0.1)
       ffi (>= 1.0.0)
       rake
     formatador (1.1.0)
-    git (1.13.2)
+    git (1.19.1)
       addressable (~> 2.8)
       rchardet (~> 1.8)
     gitlab (4.19.0)
       httparty (~> 0.20)
       terminal-table (>= 1.5.1)
-    gitlab-dangerfiles (3.8.0)
+    gitlab-dangerfiles (3.13.0)
       danger (>= 8.4.5)
       danger-gitlab (>= 8.0.0)
       rake
-    gitlab-styles (10.0.0)
-      rubocop (~> 1.43.0)
+    gitlab-styles (10.1.0)
+      rubocop (~> 1.50.2)
       rubocop-graphql (~> 0.18)
       rubocop-performance (~> 1.15)
       rubocop-rails (~> 2.17)
-      rubocop-rspec (~> 2.18)
-    guard (2.18.0)
+      rubocop-rspec (~> 2.22)
+    guard (2.18.1)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
       lumberjack (>= 1.0.12, < 2.0)
@@ -109,7 +107,7 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    hashdiff (1.0.1)
+    hashdiff (1.1.0)
     http (5.1.1)
       addressable (~> 2.8)
       http-cookie (~> 1.0)
@@ -121,45 +119,48 @@ GEM
     httparty (0.21.0)
       mini_mime (>= 1.0.0)
       multi_xml (>= 0.5.2)
-    i18n (1.13.0)
+    i18n (1.14.1)
       concurrent-ruby (~> 1.0)
-    jaro_winkler (1.5.4)
-    json (2.6.3)
+    jaro_winkler (1.5.6)
+    json (2.7.1)
     kramdown (2.4.0)
       rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
-    lefthook (1.3.11)
+    lefthook (1.6.1)
     listen (3.8.0)
       rb-fsevent (~> 0.10, >= 0.10.3)
       rb-inotify (~> 0.9, >= 0.9.10)
     llhttp-ffi (0.4.0)
       ffi-compiler (~> 1.0)
       rake (~> 13.0)
-    lumberjack (1.2.8)
+    lumberjack (1.2.10)
     method_source (1.0.0)
-    mini_mime (1.1.2)
-    mini_portile2 (2.8.1)
-    minitest (5.18.0)
+    mini_mime (1.1.5)
+    mini_portile2 (2.8.5)
+    minitest (5.21.2)
     mize (0.4.1)
       protocol (~> 2.0)
     multi_xml (0.6.0)
     nap (1.1.0)
     nenv (0.3.0)
+    net-http (0.4.1)
+      uri
     no_proxy_fix (0.1.2)
-    nokogiri (1.14.3)
-      mini_portile2 (~> 2.8.0)
+    nokogiri (1.16.0)
+      mini_portile2 (~> 2.8.2)
       racc (~> 1.4)
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    octokit (5.6.1)
+    octokit (8.0.0)
       faraday (>= 1, < 3)
       sawyer (~> 0.9)
     open4 (1.3.4)
-    parallel (1.23.0)
-    parser (3.2.2.1)
+    parallel (1.24.0)
+    parser (3.3.0.5)
       ast (~> 2.4.1)
+      racc
     proc_to_ast (0.1.0)
       coderay
       parser
@@ -172,20 +173,20 @@ GEM
     pry-byebug (3.10.1)
       byebug (~> 11.0)
       pry (>= 0.13, < 0.15)
-    public_suffix (5.0.1)
-    racc (1.6.2)
-    rack (3.0.7)
+    public_suffix (5.0.4)
+    racc (1.7.3)
+    rack (3.0.8)
     rainbow (3.1.1)
-    rake (13.0.6)
+    rake (13.1.0)
     rb-fsevent (0.11.2)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
     rbs (2.8.4)
     rchardet (1.8.0)
-    regexp_parser (2.8.0)
+    regexp_parser (2.9.0)
     reverse_markdown (2.1.1)
       nokogiri
-    rexml (3.2.5)
+    rexml (3.2.6)
     rspec (3.12.0)
       rspec-core (~> 3.12.0)
       rspec-expectations (~> 3.12.0)
@@ -195,7 +196,7 @@ GEM
     rspec-expectations (3.12.3)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.12.0)
-    rspec-mocks (3.12.5)
+    rspec-mocks (3.12.6)
       diff-lcs (>= 1.2.0, < 2.0)
       rspec-support (~> 3.12.0)
     rspec-parameterized (1.0.0)
@@ -209,41 +210,45 @@ GEM
     rspec-parameterized-table_syntax (1.0.1)
       binding_of_caller
       rspec-parameterized-core (< 2)
-    rspec-support (3.12.0)
-    rubocop (1.43.0)
+    rspec-support (3.12.1)
+    rubocop (1.50.2)
       json (~> 2.3)
       parallel (~> 1.10)
       parser (>= 3.2.0.0)
       rainbow (>= 2.2.2, < 4.0)
       regexp_parser (>= 1.8, < 3.0)
       rexml (>= 3.2.5, < 4.0)
-      rubocop-ast (>= 1.24.1, < 2.0)
+      rubocop-ast (>= 1.28.0, < 2.0)
       ruby-progressbar (~> 1.7)
       unicode-display_width (>= 2.4.0, < 3.0)
-    rubocop-ast (1.28.0)
+    rubocop-ast (1.30.0)
       parser (>= 3.2.1.0)
-    rubocop-capybara (2.18.0)
+    rubocop-capybara (2.20.0)
+      rubocop (~> 1.41)
+    rubocop-factory_bot (2.25.1)
       rubocop (~> 1.41)
     rubocop-graphql (0.19.0)
       rubocop (>= 0.87, < 2)
-    rubocop-performance (1.17.1)
-      rubocop (>= 1.7.0, < 2.0)
-      rubocop-ast (>= 0.4.0)
-    rubocop-rails (2.19.1)
+    rubocop-performance (1.20.2)
+      rubocop (>= 1.48.1, < 2.0)
+      rubocop-ast (>= 1.30.0, < 2.0)
+    rubocop-rails (2.23.1)
       activesupport (>= 4.2.0)
       rack (>= 1.1)
       rubocop (>= 1.33.0, < 2.0)
-    rubocop-rspec (2.20.0)
-      rubocop (~> 1.33)
+      rubocop-ast (>= 1.30.0, < 2.0)
+    rubocop-rspec (2.26.1)
+      rubocop (~> 1.40)
       rubocop-capybara (~> 2.17)
+      rubocop-factory_bot (~> 2.22)
     ruby-progressbar (1.13.0)
-    ruby2_keywords (0.0.5)
-    ruby_parser (3.20.3)
+    ruby_parser (3.21.0)
+      racc (~> 1.5)
       sexp_processor (~> 4.16)
     sawyer (0.9.2)
       addressable (>= 2.3.5)
       faraday (>= 0.17.3, < 3)
-    sexp_processor (4.17.0)
+    sexp_processor (4.17.1)
     shellany (0.0.1)
     simplecov (0.22.0)
       docile (~> 1.1)
@@ -254,7 +259,7 @@ GEM
       simplecov (~> 0.19)
     simplecov-html (0.12.3)
     simplecov_json_formatter (0.1.4)
-    solargraph (0.49.0)
+    solargraph (0.50.0)
       backport (~> 1.2)
       benchmark
       bundler (~> 2.0)
@@ -274,26 +279,24 @@ GEM
     table_print (1.5.7)
     terminal-table (3.0.2)
       unicode-display_width (>= 1.1.1, < 3)
-    thor (1.2.1)
-    tilt (2.1.0)
-    timecop (0.9.6)
+    thor (1.3.0)
+    tilt (2.3.0)
+    timecop (0.9.8)
     tins (1.32.1)
       sync
     tzinfo (2.0.6)
       concurrent-ruby (~> 1.0)
-    unf (0.1.4)
-      unf_ext
-    unf_ext (0.0.8.2)
-    unicode-display_width (2.4.2)
-    unparser (0.6.8)
+    unicode-display_width (2.5.0)
+    unparser (0.6.12)
       diff-lcs (~> 1.3)
-      parser (>= 3.2.0)
+      parser (>= 3.2.2.4)
+    uri (0.13.0)
     webmock (3.7.0)
       addressable (>= 2.3.6)
       crack (>= 0.3.2)
       hashdiff (>= 0.4.0, < 2.0.0)
     yard (0.9.34)
-    zeitwerk (2.6.7)
+    zeitwerk (2.6.12)
 
 PLATFORMS
   ruby
@@ -315,4 +318,4 @@ DEPENDENCIES
   webmock (= 3.7.0)
 
 BUNDLED WITH
-   2.4.2
+   2.5.4

data/exe/flaky-test-issues

@@ -19,10 +19,6 @@ options = OptionParser.new do |opts|
     params[:project] = project
   end
 
-  opts.on('-m', '--merge_request_iid MERGE_REQUEST_IID', String, 'An integer merge request IID') do |merge_request_iid|
-    params[:merge_request_iid] = merge_request_iid
-  end
-
   opts.on('--base-issue-labels BASE_ISSUE_LABELS', String,
     'Comma-separated labels (without tilde) to add to new flaky test issues') do |base_issue_labels|
     params[:base_issue_labels] = base_issue_labels.split(',')
@@ -32,6 +28,10 @@ options = OptionParser.new do |opts|
     params[:token] = token
   end
 
+  opts.on('-r', '--related-issues-file RELATED_ISSUES_FILE', String, 'The file path for the related issues') do |related_issues_file|
+    params[:related_issues_file] = related_issues_file
+  end
+
   opts.on('--dry-run', "Perform a dry-run (don't create issues)") do
     params[:dry_run] = true
   end

data/lefthook.yml

@@ -13,3 +13,16 @@ pre-push:
     rubocop:
       run: bundle exec rubocop
       glob: '*.rb'
+
+    # Changelog git trailer for the first commit of the branch
+    changelog-on-first-commit:
+      run: |
+        first_commit_message=$(git log --format=%B -n 1 $(git log main..HEAD --pretty=format:"%h" | tail -1))
+        if ! echo ${first_commit_message} | grep "Changelog:"; then
+          echo Could not find a Changelog: git trailer on the first commit for this branch.
+          echo
+          echo Please add a trailer by amending the git commit message.
+          echo
+          echo See https://docs.gitlab.com/ee/development/changelog.html#overview for more info.
+          exit 1
+        fi

data/lib/gitlab_quality/test_tooling/gitlab_client/branches_client.rb

@@ -9,7 +9,7 @@ module GitlabQuality
             client.create_branch(project, branch_name, ref)
           end
 
-          Runtime::Logger.debug("Created branch #{branch['name']} (#{branch['web_url']})")
+          Runtime::Logger.debug("Created branch #{branch['name']} (#{branch['web_url']})") if branch
           branch
         end
       end

data/lib/gitlab_quality/test_tooling/gitlab_client/commits_client.rb

@@ -5,11 +5,13 @@ module GitlabQuality
     module GitlabClient
       class CommitsClient < GitlabClient
         def create(branch_name, file_path, new_content, message)
-          commit = client.create_commit(project, branch_name, message, [
-            { action: :update, file_path: file_path, content: new_content }
-          ])
+          commit = handle_gitlab_client_exceptions do
+            client.create_commit(project, branch_name, message, [
+              { action: :update, file_path: file_path, content: new_content }
+            ])
+          end
 
-          Runtime::Logger.debug("Created commit #{commit['id']} (#{commit['web_url']}) on #{branch_name}")
+          Runtime::Logger.debug("Created commit #{commit['id']} (#{commit['web_url']}) on #{branch_name}") if commit
           commit
         end
       end

data/lib/gitlab_quality/test_tooling/gitlab_client/gitlab_client.rb

@@ -15,24 +15,32 @@ module GitlabQuality
           @retry_backoff = 0
         end
 
-        def handle_gitlab_client_exceptions # rubocop:disable Metrics/AbcSize
+        def handle_gitlab_client_exceptions
           yield
         rescue Gitlab::Error::NotFound
           # This error could be raised in assert_user_permission!
           # If so, we want it to terminate at that point
           raise
         rescue SystemCallError, OpenSSL::SSL::SSLError, Net::OpenTimeout, Net::ReadTimeout,
-          Gitlab::Error::InternalServerError, Gitlab::Error::Parsing => e
+          Gitlab::Error::InternalServerError, Gitlab::Error::BadRequest, Gitlab::Error::ResponseError, Gitlab::Error::Parsing => e
           @retry_backoff += RETRY_BACK_OFF_DELAY
 
           raise if @retry_backoff > RETRY_BACK_OFF_DELAY * MAX_RETRY_ATTEMPTS
 
-          warn_exception(e)
+          warn("#{error.class.name} #{error.message}")
           warn("Sleeping for #{@retry_backoff} seconds before retrying...")
           sleep @retry_backoff
 
           retry
         rescue StandardError => e
+          post_exception_to_slack(e) if Runtime::Env.ci_commit_ref_name == Runtime::Env.default_branch
+
+          raise e
+        end
+
+        def post_exception_to_slack(error)
+          return unless ENV['CI_SLACK_WEBHOOK_URL']
+
           pipeline = Runtime::Env.pipeline_from_project_name
           channel = case pipeline
                     when "canary"
@@ -42,9 +50,6 @@ module GitlabQuality
                     else
                       "qa-#{pipeline}"
                     end
-          error_msg = warn_exception(e)
-
-          return unless Runtime::Env.ci_commit_ref_name == Runtime::Env.default_branch
 
           slack_options = {
             slack_webhook_url: ENV.fetch('CI_SLACK_WEBHOOK_URL', nil),
@@ -54,7 +59,7 @@ module GitlabQuality
             message: <<~MSG
               An unexpected error occurred while reporting test results in issues.
               The error occurred in job: #{Runtime::Env.ci_job_url}
-              `#{error_msg}`
+              `#{error.class.name} #{error.message}`
             MSG
           }
           puts "Posting Slack message to channel: #{channel}"
@@ -79,12 +84,6 @@ module GitlabQuality
             private_token: token
           )
         end
-
-        def warn_exception(error)
-          error_msg = "#{error.class.name} #{error.message}"
-          warn(error_msg)
-          error_msg
-        end
       end
     end
   end

data/lib/gitlab_quality/test_tooling/gitlab_client/issues_client.rb

@@ -48,6 +48,12 @@ module GitlabQuality
           end
         end
 
+        def find_issue_notes(iid:)
+          handle_gitlab_client_exceptions do
+            client.issue_notes(project, iid, order_by: 'created_at', sort: 'asc').auto_paginate
+          end
+        end
+
         def find_issue_discussions(iid:)
           handle_gitlab_client_exceptions do
             client.issue_discussions(project, iid, order_by: 'created_at', sort: 'asc').auto_paginate
@@ -75,12 +81,6 @@ module GitlabQuality
           end
         end
 
-        def find_issue_notes(iid:)
-          handle_gitlab_client_exceptions do
-            client.issue_notes(project, iid, order_by: 'created_at', sort: 'asc')&.auto_paginate
-          end
-        end
-
         def create_issue_note(iid:, note:)
           handle_gitlab_client_exceptions do
             client.create_issue_note(project, iid, note)

data/lib/gitlab_quality/test_tooling/gitlab_client/merge_requests_client.rb

@@ -5,10 +5,12 @@ module GitlabQuality
     module GitlabClient
       class MergeRequestsClient < GitlabClient
         def find_merge_request_changes(merge_request_iid:)
-          client.merge_request_changes(project, merge_request_iid)
+          handle_gitlab_client_exceptions do
+            client.merge_request_changes(project, merge_request_iid)
+          end
         end
 
-        def create_merge_request(title:, source_branch:, target_branch:, description:, labels:, assignee_id: nil)
+        def create_merge_request(title:, source_branch:, target_branch:, description:, labels:, assignee_id: nil, reviewer_ids: [])
           attrs = {
             source_branch: source_branch,
             target_branch: target_branch,
@@ -16,7 +18,8 @@ module GitlabQuality
             labels: labels,
             assignee_id: assignee_id,
             squash: true,
-            remove_source_branch: true
+            remove_source_branch: true,
+            reviewer_ids: reviewer_ids
           }.compact
 
           merge_request = handle_gitlab_client_exceptions do

data/lib/gitlab_quality/test_tooling/gitlab_client/merge_requests_dry_client.rb

@@ -26,9 +26,11 @@ module GitlabQuality
           puts "The following note would have been updated id: #{id} with body: #{note} for mr_iid: #{merge_request_iid}"
         end
 
-        def create_merge_request(title:, source_branch:, target_branch:, description:, labels:)
+        def create_merge_request(title:, source_branch:, target_branch:, description:, labels:, assignee_id:, reviewer_ids:)
           puts "A merge request would be created with title: #{title} " \
-               "source_branch: #{source_branch} target_branch: #{target_branch} description: #{description} labels: #{labels}"
+               "source_branch: #{source_branch} target_branch: #{target_branch} " \
+               "description: #{description} labels: #{labels}, assignee_id: #{assignee_id}" \
+               "reviewer_ids: #{reviewer_ids}"
         end
       end
     end

data/lib/gitlab_quality/test_tooling/report/concerns/issue_reports.rb

@@ -1,13 +1,16 @@
 # frozen_string_literal: true
 
+require 'active_support/core_ext/object/blank'
+
 module GitlabQuality
   module TestTooling
     module Report
       module Concerns
         module IssueReports
-          JOB_URL_REGEX                = %r{(?<job_url>https://(?<host>[\w.]+)/(?<project_path>[\w\-./]+)/-/jobs/\d+)}
+          JOB_URL_REGEX = %r{(?<job_url>https://(?<host>[\w.]+)/(?<project_path>[\w\-./]+)/-/jobs/\d+)}
           FAILED_JOB_DESCRIPTION_REGEX = /First happened in #{JOB_URL_REGEX}\./m
-          REPORT_ITEM_REGEX            = /^1\. \d{4}-\d{2}-\d{2}: #{JOB_URL_REGEX} \((?<pipeline_url>.+)\)$/
+          REPORT_ITEM_REGEX = /^1\. \d{4}-\d{2}-\d{2}: #{JOB_URL_REGEX} \((?<pipeline_url>\S+)\)/
+          LATEST_REPORTS_TO_SHOW = 10
 
           def initial_reports_section(test)
             <<~REPORTS
@@ -17,22 +20,21 @@ module GitlabQuality
             REPORTS
           end
 
-          def add_report_to_issue_description(issue, test)
-            issue_description = issue.description
-
-            # We include the number of reports in the header, for visibility.
-            new_issue_description =
-              if issue_description.include?('### Reports')
-                # We count the number of existing reports.
-                reports_count = issue_description
-                  .scan(REPORT_ITEM_REGEX)
-                  .size.to_i + 1
-                issue_description.sub(/^### Reports.*$/, "### Reports (#{reports_count})")
-              else # For issue with the legacy format, we add the Reports section
-                update_legacy_issue_description(issue_description)
-              end
+          def increment_reports(
+            current_reports_content:,
+            test:,
+            reports_section_header: '### Reports',
+            item_extra_content: nil,
+            reports_extra_content: nil)
+            preserved_content = current_reports_content.split(reports_section_header).first&.strip
+            reports = report_lines(current_reports_content) + [report_list_item(test, item_extra_content: item_extra_content)]
 
-            [new_issue_description, report_list_item(test)].join("\n")
+            [
+              preserved_content,
+              "#{reports_section_header} (#{reports.size})",
+              reports_list(reports),
+              reports_extra_content
+            ].reject(&:blank?).compact.join("\n\n")
           end
 
           def failed_issue_job_url(issue)
@@ -49,8 +51,28 @@ module GitlabQuality
 
           private
 
-          def report_list_item(test)
-            "1. #{Time.new.utc.strftime('%F')}: #{test.ci_job_url} (#{ENV.fetch('CI_PIPELINE_URL', 'pipeline url is missing')})"
+          def report_lines(content)
+            content.lines.grep(REPORT_ITEM_REGEX).map(&:strip)
+          end
+
+          def reports_list(reports)
+            sorted_reports = reports.sort.reverse
+
+            if sorted_reports.size > LATEST_REPORTS_TO_SHOW
+              [
+                "Last 10 reports:",
+                sorted_reports[...LATEST_REPORTS_TO_SHOW].join("\n"),
+                "<details><summary>See #{sorted_reports.size - LATEST_REPORTS_TO_SHOW} more reports</summary>",
+                sorted_reports[LATEST_REPORTS_TO_SHOW..].join("\n"),
+                "</details>"
+              ].join("\n\n")
+            else
+              sorted_reports.join("\n")
+            end
+          end
+
+          def report_list_item(test, item_extra_content: nil)
+            "1. #{Time.new.utc.strftime('%F')}: #{test.ci_job_url} (#{ENV.fetch('CI_PIPELINE_URL', 'pipeline url is missing')}) #{item_extra_content}".strip
           end
 
           def job_urls_from_description(issue_description, regex)
@@ -60,15 +82,6 @@ module GitlabQuality
             end
           end
 
-          def update_legacy_issue_description(issue_description)
-            test_captures = issue_description.scan(JOB_URL_REGEX)
-            reports_count = test_captures.size.to_i + 1
-
-            updated_description = "#{issue_description}\n\n### Reports (#{reports_count})\n"
-            updated_description = [updated_description, *test_captures_to_report_items(test_captures)].join("\n") unless test_captures.empty?
-            updated_description
-          end
-
           def test_captures_to_report_items(test_captures)
             test_captures.map do |ci_job_url, _, _|
               report_list_item(GitlabQuality::TestTooling::TestResult::JsonTestResult.new(

data/lib/gitlab_quality/test_tooling/report/concerns/utils.rb

@@ -16,7 +16,7 @@ module GitlabQuality
           end
 
           def new_issue_title(test)
-            "#{partial_file_path(test.file)} | #{search_safe(test.name)}".strip
+            "[Test] #{partial_file_path(test.file)} | #{search_safe(test.name)}".strip
           end
 
           def partial_file_path(path)