gitlab_quality-test_tooling 1.26.1 → 1.27.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: e3ee730d9233005716cead9875a8a94b79db857b68ef7dcc16715e8f1ac3f281
-  data.tar.gz: 74d20185fa73cb46cdc998786267d79946c5f96e29966b4c5d57eac40b26c14e
+  metadata.gz: a5b3317b1df5c3f923259fb0dbb71e9cfdf9b4e32f9338b2189758538aaacb30
+  data.tar.gz: 60fda58d2a830e1849c8175347050cb81f62658a20e1ee4c9bdde2cb3049d965
 SHA512:
-  metadata.gz: 7909fce13b1b085bef53aa1843029cda34199bd11757ec50a3175a933b3e2805d432a489e9fe157926c14c7f6819466b6a45970cd8b0d62f483df0b0ea422856
-  data.tar.gz: 2e28b30086187f268911b15c359bd5635958c6061dfddc0213e3176e686cbf27fdc96c5fd72594a608cd0ebf21ac3870ac9f0330a148a5112e252cb97d32a8fc
+  metadata.gz: 0c70f1860b051ccd633aae6cb58298a90edecdd8a93a598e76f90ac8b2a337f755d77e30fa8b1dd6bcd3df7473c9af31c8a6e46509b59a5047d1be03a4bb2910
+  data.tar.gz: 1ae2fbf63375f49d43ef28cb5a5ec44af6940a5ffd46b43a07c07855085449994858bc49b4ca61c0cd73737c7d4960de9f5f092019681668ad339326f069c741

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (1.26.1)
+    gitlab_quality-test_tooling (1.27.0)
       activesupport (~> 7.1)
       amatch (~> 0.4.1)
       gitlab (~> 4.19)
@@ -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/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.0"
   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.0
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-05-22 00:00:00.000000000 Z
+date: 2024-05-23 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: climate_control