gitlab_quality-test_tooling 0.6.0 → 0.6.2

checksums.yaml

@@ -1,7 +1,7 @@
 ---
 SHA256:
-  metadata.gz: dbf155d73f05905e7650ae5660238042126c7eb4db61207882eca9dde123bc3c
-  data.tar.gz: 9f09da32ac58114050a29e6d1394cea9b9e10dec024b2d6ec99137826e2927cb
+  metadata.gz: 80c8edb6daa8bf924d294ef291d82fd60c8545898aaa0eac7e71b98dad4dc1c2
+  data.tar.gz: 9e414922f797ce645e951f3fc26adb394a7a7f48f2c01bf7ddd7d218a1229486
 SHA512:
-  metadata.gz: 6f2957b97fe89cfa6d18339f7b6b9ab00aa74d2677c0a957ec241c874b8ad69d5ee99470c3468b4ae23f239fbad3f2618cdba8499aa3548e0726884a6c688e66
-  data.tar.gz: 4abb3047a022e643314ae67a51bddfdb7c6c2d96034b2a1197cd8419e00142218d729ffad5d2506b4efd9c7a2d8617bbf2c828521733db1c9c3af0a813baea76
+  metadata.gz: 573774ab0d7be7d23d2c262f0246377b824604240930ba451b088af914e7b166b8844e63425f3583a16f45bdc73be3779e5e5eb081218ff934fcb2e87698a8bf
+  data.tar.gz: f0ad9eb31e28e2c9f21c317b6d84e78645c36f5fff30baa8ada905d17f202710e087125a373f8fc38dd06612c08764839d2a35aaa835d133211219ee2778c37a

data/Gemfile.lock

@@ -1,7 +1,7 @@
 PATH
   remote: .
   specs:
-    gitlab_quality-test_tooling (0.6.0)
+    gitlab_quality-test_tooling (0.6.2)
       activesupport (~> 6.1)
       gitlab (~> 4.19)
       http (~> 5.0)

data/README.md

@@ -80,7 +80,7 @@ Purpose: Relate test failures to failure issues from RSpec report files (JSON or
 Usage: exe/relate-failure-issue [options]
     -i, --input-files INPUT_FILES    RSpec report files (JSON or JUnit XML)
         --max-diff-ratio MAX_DIFF_RATO
-                                     Max stacktrace diff ratio for QA failure issues detection
+                                     Max stacktrace diff ratio for failure issues detection
     -p, --project PROJECT            Can be an integer or a group/project string
     -t, --token TOKEN                A valid access token with `api` scope and Maintainer permission in PROJECT
         --system-log-files SYSTEM_LOG_FILES

data/exe/relate-failure-issue

@@ -15,7 +15,7 @@ options = OptionParser.new do |opts|
     params[:input_files] = input_files
   end
 
-  opts.on('--max-diff-ratio MAX_DIFF_RATO', Float, 'Max stacktrace diff ratio for QA failure issues detection') do |max_diff_ratio|
+  opts.on('--max-diff-ratio MAX_DIFF_RATO', Float, 'Max stacktrace diff ratio for failure issues detection') do |max_diff_ratio|
     params[:max_diff_ratio] = max_diff_ratio
   end
 

data/lib/gitlab_quality/test_tooling/gitlab_issue_client.rb

@@ -153,7 +153,7 @@ module GitlabQuality
         slack_options = {
           slack_webhook_url: ENV.fetch('CI_SLACK_WEBHOOK_URL', nil),
           channel: channel,
-          username: "GitLab QA Bot",
+          username: "GitLab Quality Test Tooling",
           icon_emoji: ':ci_failing:',
           message: <<~MSG
             An unexpected error occurred while reporting test results in issues.

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

@@ -25,7 +25,7 @@ module GitlabQuality
         FAILED_JOB_DESCRIPTION_REGEX = %r{First happened in https?://\S+\.}m
         FAILED_JOB_NOTE_REGEX = %r{Failed most recently in \D+ pipeline: https?://\S+}
         NEW_ISSUE_LABELS = Set.new(%w[test failure::new priority::2]).freeze
-        IGNORE_EXCEPTIONS = ['Net::ReadTimeout'].freeze
+        IGNORE_EXCEPTIONS = ['Net::ReadTimeout', '403 Forbidden - Your account has been blocked'].freeze
         SCREENSHOT_IGNORED_ERRORS = ['500 Internal Server Error', 'fabricate_via_api!', 'Error Code 500'].freeze
 
         MultipleIssuesFound = Class.new(StandardError)
@@ -118,10 +118,10 @@ module GitlabQuality
         end
 
         def pipeline_issues_with_similar_stacktrace(test)
-          gitlab.find_issues(options: { state: 'opened', labels: (base_issue_labels + %w[failure::new]).join(','),
+          gitlab.find_issues(options: { state: 'opened', labels: (base_issue_labels + %w[test failure::new]).join(','),
                                         created_after: past_timestamp(2) }).select do |issue|
             job_url_from_issue = failed_issue_job_url(issue)
-            next unless pipeline == pipeline_env_from_job_url(job_url_from_issue)
+            next if pipeline != pipeline_env_from_job_url(job_url_from_issue)
 
             stack_trace_from_issue = cleaned_stack_trace_from_issue(issue)
             stack_trace_from_test = cleaned_stacktrace_from_test(test)
@@ -153,7 +153,7 @@ module GitlabQuality
           if job_url.include?('/quality/')
             job_url.partition('/quality/').last.partition('/').first
           else
-            'master'
+            Runtime::Env.default_branch
           end
         end
 
@@ -163,7 +163,7 @@ module GitlabQuality
         end
 
         def failure_issues(test)
-          gitlab.find_issues(options: { state: 'opened', labels: 'QA' }).select do |issue|
+          gitlab.find_issues(options: { state: 'opened', labels: base_issue_labels + %w[test] }).select do |issue|
             issue_title = issue.title.strip
             issue_title.include?(test.name) || issue_title.include?(partial_file_path(test.file))
           end
@@ -202,7 +202,7 @@ module GitlabQuality
 
         def find_relevant_failure_issues(test) # rubocop:disable Metrics/AbcSize
           clean_first_test_failure_stacktrace = cleaned_stacktrace_from_test(test)
-          # Search with the `search` param returns 500 errors, so we filter by ~QA and then filter further in Ruby
+          # Search with the `search` param returns 500 errors, so we filter by `base_issue_labels` and then filter further in Ruby
           failure_issues(test).each_with_object({}) do |issue, memo|
             clean_relevant_issue_stacktrace = cleaned_stack_trace_from_issue(issue)
             next if clean_relevant_issue_stacktrace.nil?
@@ -370,13 +370,13 @@ module GitlabQuality
         # @param [Array<Hash>] exceptions the exceptions associated with the failure.
         # @return [String] the reason to ignore the exceptions, or `nil` if any exceptions should not be ignored.
         def ignore_failure_reason(exceptions)
-          exception_classes = exceptions
-            .filter_map { |exception| exception['class'] if IGNORE_EXCEPTIONS.include?(exception['class']) }
+          exception_messages = exceptions
+            .filter_map { |exception| exception['message'] if IGNORE_EXCEPTIONS.any? { |e| exception['message'].include?(e) } }
             .compact
-          return if exception_classes.empty? || exception_classes.size < exceptions.size
+          return if exception_messages.empty? || exception_messages.size < exceptions.size
 
-          msg = exception_classes.many? ? 'the errors were' : 'the error was'
-          "#{msg} #{exception_classes.join(', ')}"
+          msg = exception_messages.many? ? 'the errors were' : 'the error was'
+          "#{msg} #{exception_messages.join(', ')}"
         end
       end
     end

data/lib/gitlab_quality/test_tooling/version.rb

@@ -2,6 +2,6 @@
 
 module GitlabQuality
   module TestTooling
-    VERSION = "0.6.0"
+    VERSION = "0.6.2"
   end
 end

metadata

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