gitlab_quality-test_tooling 1.26.1 → 1.27.1

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e3ee730d9233005716cead9875a8a94b79db857b68ef7dcc16715e8f1ac3f281
-  data.tar.gz: 74d20185fa73cb46cdc998786267d79946c5f96e29966b4c5d57eac40b26c14e
+  metadata.gz: 34edb6377d3304f4b061a5704e9ba865cd550209c7fefda3c66b6734f2b95ce5
+  data.tar.gz: e00f10eaf9690f803168a86da44417b5bdc02f074773819f6deb6fa466bef23e
 SHA512:
-  metadata.gz: 7909fce13b1b085bef53aa1843029cda34199bd11757ec50a3175a933b3e2805d432a489e9fe157926c14c7f6819466b6a45970cd8b0d62f483df0b0ea422856
-  data.tar.gz: 2e28b30086187f268911b15c359bd5635958c6061dfddc0213e3176e686cbf27fdc96c5fd72594a608cd0ebf21ac3870ac9f0330a148a5112e252cb97d32a8fc
+  metadata.gz: 3ced20afebe7dbdcc5e47fcc3c864db514bd2d2b4920d6085683c201444243e5ec023cde39c7d333d28acd6a2539f35dfe6af21ee881d2442f48e5c4335b164c
+  data.tar.gz: 9d279b982fb6438e064db0336b532fbd0ba7e803e16b65e678576fca037120cc7a327bba21d6ab3400122e6aee18060e900eb14d1063aed77f61d57df73f8cab

data/Gemfile.lock

@@ -1,8 +1,8 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (1.26.1)
-      activesupport (~> 7.1)
+    gitlab_quality-test_tooling (1.27.1)
+      activesupport (>= 7.0, < 7.2)
       amatch (~> 0.4.1)
       gitlab (~> 4.19)
       http (~> 5.0)
@@ -330,4 +330,4 @@ DEPENDENCIES
   webmock (= 3.7.0)
 
 BUNDLED WITH
-   2.5.6
+   2.5.4

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

@@ -29,15 +29,10 @@ module GitlabQuality
         MultipleNotesFound = Class.new(StandardError)
 
         def initialize(
-          token:,
-          input_files:,
           base_issue_labels: nil,
-          dry_run: false,
-          related_issues_file: nil,
-          project: nil,
           max_diff_ratio: DEFAULT_MAX_DIFF_RATIO_FOR_DETECTION,
-          **_kwargs)
-          super(token: token, input_files: input_files, related_issues_file: related_issues_file, project: project, dry_run: dry_run)
+          **kwargs)
+          super(**kwargs)
 
           @base_issue_labels = Set.new(base_issue_labels)
           @max_diff_ratio = max_diff_ratio.to_f
@@ -64,7 +59,13 @@ module GitlabQuality
             puts " => Reporting failure for test '#{test.name}'..."
 
             issues = find_issues_by_hash(test_hash(test), state: 'opened', labels: SEARCH_LABELS)
-            issues << create_issue(test) if issues.empty?
+
+            if issues.empty?
+              issues << create_issue(test)
+            else
+              # Keep issues description up-to-date
+              update_issues(issues, test)
+            end
 
             update_reports(issues, test)
             collect_issues(test, issues)

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

@@ -27,14 +27,9 @@ module GitlabQuality
         DOC
 
         def initialize(
-          token:,
-          input_files:,
           base_issue_labels: nil,
-          confidential: false,
-          dry_run: false,
-          project: nil,
-          **_kwargs)
-          super(token: token, input_files: input_files, project: project, confidential: confidential, dry_run: dry_run)
+          **kwargs)
+          super(**kwargs)
 
           @base_issue_labels = Set.new(base_issue_labels)
         end
@@ -60,7 +55,13 @@ module GitlabQuality
             puts " => Reporting flakiness for test '#{test.name}'..."
 
             issues = find_issues_by_hash(test_hash(test), state: 'opened', labels: SEARCH_LABELS)
-            issues << create_issue(test) if issues.empty?
+
+            if issues.empty?
+              issues << create_issue(test)
+            else
+              # Keep issues description up-to-date
+              update_issues(issues, test)
+            end
 
             update_reports(issues, test)
             collect_issues(test, issues)

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

@@ -51,17 +51,19 @@ module GitlabQuality
 
         def new_issue_description(test)
           <<~DESCRIPTION
-          ### Test metadata (don't modify)
+          ### Test metadata
 
+          <!-- Don't modify this section as it's auto-generated -->
           | Field | Value |
           | ------ | ------ |
           | File URL | #{test.test_file_link} |
           | Filename | `#{test.file}` |
           | Description | `#{test.name}` |
-          | Test level | #{test.level} |
+          | Test level | `#{test.level}` |
           | Hash | `#{test_hash(test)}` |
-          | Expected duration | < #{test.max_duration_for_test} seconds |
+          | 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
 
@@ -111,6 +113,21 @@ module GitlabQuality
           end
         end
 
+        def update_issues(issues, test)
+          issues.each do |issue|
+            update_issue(issue, test)
+          end
+        end
+
+        def update_issue(issue, test)
+          issue_attrs = {
+            description: new_issue_description(test)
+          }
+          return if issue.description == issue_attrs[:description]
+
+          gitlab.edit_issue(iid: issue.iid, options: issue_attrs)
+        end
+
         def issue_labels(issue)
           issue&.labels&.to_set || Set.new
         end

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

@@ -45,7 +45,13 @@ module GitlabQuality
             puts " => Reporting slowness for test '#{test.name}'..."
 
             issues = find_issues_by_hash(test_hash(test), state: 'opened', labels: SEARCH_LABELS)
-            issues << create_issue(test) if issues.empty?
+
+            if issues.empty?
+              issues << create_issue(test)
+            else
+              # Keep issues description up-to-date
+              update_issues(issues, test)
+            end
 
             update_reports(issues, test)
             collect_issues(test, issues)

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

@@ -81,9 +81,7 @@ module GitlabQuality
           live_envs = %w[staging staging-canary staging-ref canary preprod production]
           return unless live_envs.include?(ci_project_name)
 
-          test_subset = if env_var_value_if_defined('NO_ADMIN') == 'true'
-                          'sanity-no-admin'
-                        elsif env_var_value_if_defined('SMOKE_ONLY') == 'true'
+          test_subset = if env_var_value_if_defined('SMOKE_ONLY') == 'true'
                           'sanity'
                         else
                           'full'

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

@@ -38,7 +38,11 @@ module GitlabQuality
               console_log = console_logger(source: source, level: Env.log_level)
               file_log = file_logger(source: source, path: log_path)
 
-              ActiveSupport::BroadcastLogger.new(console_log, file_log, file_log)
+              if ActiveSupport.const_defined?(:BroadcastLogger) # only available in ActiveSupport >= 7.1.0
+                ActiveSupport::BroadcastLogger.new(console_log, file_log, file_log)
+              else
+                console_log.extend(ActiveSupport::Logger.broadcast(file_log))
+              end
             end
           end
 

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

@@ -1,5 +1,7 @@
 # frozen_string_literal: true
 
+require 'influxdb-client'
+
 module GitlabQuality
   module TestTooling
     module TestMetricsExporter

data/lib/gitlab_quality/test_tooling/version.rb

@@ -2,6 +2,6 @@
 
 module GitlabQuality
   module TestTooling
-    VERSION = "1.26.1"
+    VERSION = "1.27.1"
   end
 end

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_quality-test_tooling
 version: !ruby/object:Gem::Version
-  version: 1.26.1
+  version: 1.27.1
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-05-22 00:00:00.000000000 Z
+date: 2024-05-28 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control
@@ -196,16 +196,22 @@ dependencies:
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '7.0'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
+      - !ruby/object:Gem::Version
+        version: '7.0'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '7.1'
+        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: amatch
   requirement: !ruby/object:Gem::Requirement