gitlab_quality-test_tooling 1.5.1 → 1.5.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 7ba60bceafbfb950d7cc588d11b14b457f74cdf3f39271b33b171d7de935c6a2
-  data.tar.gz: 3129a7a6da1a0c6f46df06ccd69f4d6d84a2f926a1dca268ea73455054fdbf4d
+  metadata.gz: 421778dfb1e380d4b8e0fc3e43a8d4147d50c0a11304ac36e964aeaf99330404
+  data.tar.gz: 1f09fb87a864818b6728364454c0f8b9043cc73c5cec1ce4aa0104261d5dbfea
 SHA512:
-  metadata.gz: 0cfd0c911a463fe793c701ae13bef14dcd4e5220bc0e41819126e6659f1bcb6589ce1dba5baa33f6410f92a8ddc6414623cb7952638f66a3d79a55099089f1a6
-  data.tar.gz: 12600147b76a659a725662119d08318999a85f1483a5d68bb6da14f464085f3d67fc6748a2f94dcee9bef82357c62a17f874d3a3feb2ee6961fbcb094f61aec0
+  metadata.gz: 7cdbc73cd4ac08dc3f78a27b4575ec3a6005b7cfd95577bd61c8868d6836b3f1d549b1744d27d6cc2939e08d9b777eb74e84e4fb473952c7a9c982dc7a9b719e
+  data.tar.gz: ddd08dc76c0a45cae09283f0a2c8b91a731a203ffa624cac641254b4933b49585d585b5ada0169f85cccd4c59af562ffca9b16df5d733ee98d38f0c15af08334

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (1.5.1)
+    gitlab_quality-test_tooling (1.5.2)
       activesupport (>= 6.1, < 7.2)
       amatch (~> 0.4.1)
       gitlab (~> 4.19)
@@ -15,7 +15,7 @@ PATH
 GEM
   remote: https://rubygems.org/
   specs:
-    activesupport (6.1.7.3)
+    activesupport (7.0.8)
       concurrent-ruby (~> 1.0, >= 1.0.2)
       i18n (>= 1.6, < 2)
       minitest (>= 5.1)

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

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'fileutils'
+
 module GitlabQuality
   module TestTooling
     module Report

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

@@ -15,6 +15,7 @@ module GitlabQuality
             merge_request_iid: merge_request_iid)
           @files = Array(input_files)
           @merge_request_iid = merge_request_iid
+          @slow_tests = []
         end
 
         def invoke!
@@ -25,7 +26,7 @@ module GitlabQuality
 
         private
 
-        attr_reader :gitlab_merge_request, :files, :project, :merge_request_iid
+        attr_reader :gitlab_merge_request, :files, :project, :merge_request_iid, :slow_tests
 
         def run!
           puts "Reporting slow tests in MR #{merge_request_iid}"
@@ -33,21 +34,26 @@ module GitlabQuality
           TestResults::Builder.new(files).test_results_per_file do |test_results|
             puts "=> Reporting #{test_results.count} tests in #{test_results.path}"
 
-            upsert_mr_note(slow_related_tests(slow_tests(test_results))) if should_create_mr_note?(test_results)
+            @slow_tests += slow_related_tests(find_slow_tests(test_results))
           end
+
+          @slow_tests.uniq!
+          @slow_tests.reject!(&:failed?)
+
+          return unless @slow_tests.any?
+
+          upsert_mr_note(@slow_tests)
         end
 
-        def should_create_mr_note?(test_results)
-          gitlab_merge_request.merge_request_changed_files.any? &&
-            slow_tests(test_results).any? &&
-            slow_related_tests(slow_tests(test_results)).any?
+        def related_test?(slow_test)
+          base_file = slow_test.file.split('/').last.gsub('_spec.rb', '')
+
+          merge_request_changed_files.any? { |change| change =~ /#{base_file}/ }
         end
 
         def slow_related_tests(slow_test_results)
-          slow_test_results.select do |slow_test_result|
-            base_file = slow_test_result.file.split('/').last.gsub('_spec.rb', '')
-
-            merge_request_changed_files.any? { |change| change =~ /#{base_file}/ }
+          slow_test_results.select do |slow_test|
+            related_test?(slow_test)
           end
         end
 
@@ -55,7 +61,7 @@ module GitlabQuality
           @merge_request_changed_files ||= gitlab_merge_request.merge_request_changed_files
         end
 
-        def slow_tests(test_results)
+        def find_slow_tests(test_results)
           test_results.select(&:slow_test?)
         end
 
@@ -66,7 +72,8 @@ module GitlabQuality
             ":tools: #{SLOW_TEST_NOTE_SOURCE_CODE}\n",
             ":recycle: #{SLOW_TEST_NOTE_FEEDBACK}\n",
             "---\n",
-            ':snail: Slow tests detected in this merge request, might be related with changed RSpec files.',
+            ":snail: Slow tests detected in this merge request. These slow tests might be related to this merge request's changes.",
+            "<details><summary>Click to expand</summary>\n",
             '| Job | File | Name | Duration | Expected duration |',
             '| --- | --- | --- | --- | --- |'
           ]
@@ -83,7 +90,7 @@ module GitlabQuality
         end
 
         def build_note(slow_test)
-          rows = note_header + slow_test_rows(slow_test)
+          rows = note_header + slow_test_rows(slow_test) + ["\n</details>"]
 
           rows.join("\n")
         end
@@ -105,11 +112,13 @@ module GitlabQuality
         end
 
         def add_slow_test_rows(gitlab_note, slow_tests)
+          gitlab_note.gsub!("\n\n</details>", "")
+
           slow_tests.each do |slow_test|
             gitlab_note += "\n#{slow_test_table_row(slow_test)}" unless note_comment_includes_slow_test?(gitlab_note, slow_test)
           end
 
-          gitlab_note
+          "#{gitlab_note}\n\n</details>"
         end
 
         def upsert_mr_note(slow_tests)

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

@@ -126,7 +126,10 @@ module GitlabQuality
             return
           end
 
-          super
+          created_issue = super
+          test.failure_issue ||= created_issue.web_url
+
+          created_issue
         end
 
         def pipeline_issues_with_similar_stacktrace(test)

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

@@ -3,6 +3,7 @@
 require 'forwardable'
 require 'fileutils'
 require 'rainbow'
+require 'active_support'
 require 'active_support/logger'
 
 module GitlabQuality

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

@@ -33,6 +33,10 @@ module GitlabQuality
           status == 'pending'
         end
 
+        def failed?
+          status == 'failed'
+        end
+
         def ci_job_url
           report.fetch('ci_job_url', '')
         end

data/lib/gitlab_quality/test_tooling/version.rb

@@ -2,6 +2,6 @@
 
 module GitlabQuality
   module TestTooling
-    VERSION = "1.5.1"
+    VERSION = "1.5.2"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_quality-test_tooling
 version: !ruby/object:Gem::Version
-  version: 1.5.1
+  version: 1.5.2
 platform: ruby
 authors:
 - GitLab Quality
 autorequire: 
 bindir: exe
 cert_chain: []
-date: 2023-11-07 00:00:00.000000000 Z
+date: 2023-11-14 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control