gitlab_quality-test_tooling 1.26.0 → 1.27.0

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: 6895c7a00b6414c417fdadfe15fc030d316c1a6ecac135d5a3b281748d321926
-  data.tar.gz: d9c8dcf80524df5f22c3cc32439f2abaf5f148d061c46a1b8f74a6c40aaa2a21
+  metadata.gz: a5b3317b1df5c3f923259fb0dbb71e9cfdf9b4e32f9338b2189758538aaacb30
+  data.tar.gz: 60fda58d2a830e1849c8175347050cb81f62658a20e1ee4c9bdde2cb3049d965
 SHA512:
-  metadata.gz: c6305ab3fcdf665926a8a1180ffd1308611e39ef53442d584e1797e8caf0115527bac36376b668ee89473d94f86ee160eeeeeaf2a72dca8348a96b44c375f233
-  data.tar.gz: bf77f8ce83ebebb939925479593cb7c21b9f2f4a8ac564da009605d509ec4e56c4eb68b73fbc156b05567c9c0d5603b7ac2552f77c615b1b19fa2e5e19ad568a
+  metadata.gz: 0c70f1860b051ccd633aae6cb58298a90edecdd8a93a598e76f90ac8b2a337f755d77e30fa8b1dd6bcd3df7473c9af31c8a6e46509b59a5047d1be03a4bb2910
+  data.tar.gz: 1ae2fbf63375f49d43ef28cb5a5ec44af6940a5ffd46b43a07c07855085449994858bc49b4ca61c0cd73737c7d4960de9f5f092019681668ad339326f069c741

data/Gemfile.lock

@@ -1,11 +1,12 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (1.26.0)
-      activesupport (>= 6.1, < 7.2)
+    gitlab_quality-test_tooling (1.27.0)
+      activesupport (~> 7.1)
       amatch (~> 0.4.1)
       gitlab (~> 4.19)
       http (~> 5.0)
+      influxdb-client (~> 3.1)
       nokogiri (~> 1.10)
       parallel (>= 1, < 2)
       rainbow (>= 3, < 4)
@@ -35,7 +36,7 @@ GEM
     backport (1.2.0)
     base64 (0.2.0)
     benchmark (0.3.0)
-    bigdecimal (3.1.6)
+    bigdecimal (3.1.8)
     binding_of_caller (1.0.0)
       debug_inspector (>= 0.0.1)
     byebug (11.1.3)
@@ -128,7 +129,7 @@ GEM
     httparty (0.21.0)
       mini_mime (>= 1.0.0)
       multi_xml (>= 0.5.2)
-    i18n (1.14.4)
+    i18n (1.14.5)
       concurrent-ruby (~> 1.0)
     influxdb-client (3.1.0)
     jaro_winkler (1.5.6)
@@ -148,7 +149,7 @@ GEM
     method_source (1.0.0)
     mini_mime (1.1.5)
     mini_portile2 (2.8.5)
-    minitest (5.22.2)
+    minitest (5.23.0)
     mize (0.4.1)
       protocol (~> 2.0)
     multi_xml (0.6.0)
@@ -313,13 +314,11 @@ PLATFORMS
   ruby
 
 DEPENDENCIES
-  activesupport (>= 6.1, < 7.2)
   climate_control (~> 1.2)
   gitlab-dangerfiles (~> 3.8)
   gitlab-styles (~> 10.0)
   gitlab_quality-test_tooling!
   guard-rspec (~> 4.7)
-  influxdb-client (~> 3.1)
   lefthook (~> 1.3)
   pry-byebug (= 3.10.1)
   rake (~> 13.0)

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/config.rb

@@ -5,19 +5,19 @@ require "singleton"
 module GitlabQuality
   module TestTooling
     module TestMetricsExporter
-      class << self
-        def configuration
-          Config.instance
-        end
-
-        def configure
-          yield(configuration)
-        end
-      end
-
       class Config
         include Singleton
 
+        class << self
+          def configuration
+            Config.instance
+          end
+
+          def configure
+            yield(configuration)
+          end
+        end
+
         attr_accessor :influxdb_url,
           :influxdb_token,
           :influxdb_bucket,

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

@@ -6,8 +6,8 @@ module GitlabQuality
       class Formatter < RSpec::Core::Formatters::BaseFormatter
         RSpec::Core::Formatters.register(self, :stop)
 
-        def stop(_notification)
-          setup_test_metrics_exporter
+        def stop(notification)
+          setup_test_metrics_exporter(notification.examples)
 
           log_test_metrics.push_test_metrics(
             custom_keys_tags: config.custom_keys_tags,
@@ -29,9 +29,9 @@ module GitlabQuality
           Config.configuration
         end
 
-        def setup_test_metrics_exporter
+        def setup_test_metrics_exporter(examples)
           @log_test_metrics = LogTestMetrics.new(
-            examples: notification.examples,
+            examples: examples,
             influxdb_url: config.influxdb_url,
             influxdb_token: config.influxdb_token,
             influxdb_bucket: config.influxdb_bucket,

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.0"
+    VERSION = "1.27.0"
   end
 end

metadata

@@ -1,35 +1,15 @@
 --- !ruby/object:Gem::Specification
 name: gitlab_quality-test_tooling
 version: !ruby/object:Gem::Version
-  version: 1.26.0
+  version: 1.27.0
 platform: ruby
 authors:
 - GitLab Quality
 autorequire:
 bindir: exe
 cert_chain: []
-date: 2024-05-16 00:00:00.000000000 Z
+date: 2024-05-23 00:00:00.000000000 Z
 dependencies:
-- !ruby/object:Gem::Dependency
-  name: activesupport
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '6.1'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '7.2'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '6.1'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '7.2'
 - !ruby/object:Gem::Dependency
   name: climate_control
   requirement: !ruby/object:Gem::Requirement
@@ -86,20 +66,6 @@ dependencies:
     - - "~>"
       - !ruby/object:Gem::Version
         version: '4.7'
-- !ruby/object:Gem::Dependency
-  name: influxdb-client
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.1'
 - !ruby/object:Gem::Dependency
   name: lefthook
   requirement: !ruby/object:Gem::Requirement
@@ -230,22 +196,16 @@ dependencies:
   name: activesupport
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '6.1'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '7.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - ">="
-      - !ruby/object:Gem::Version
-        version: '6.1'
-    - - "<"
+    - - "~>"
       - !ruby/object:Gem::Version
-        version: '7.2'
+        version: '7.1'
 - !ruby/object:Gem::Dependency
   name: amatch
   requirement: !ruby/object:Gem::Requirement
@@ -288,6 +248,20 @@ 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