gitlab_quality-test_tooling 0.1.0 → 0.2.0

data/lib/gitlab_quality/test_tooling/report/concerns/results_reporter.rb

@@ -0,0 +1,75 @@
+# frozen_string_literal: true
+
+require 'active_support/core_ext/enumerable'
+
+module GitlabQuality
+  module TestTooling
+    module Report
+      module Concerns
+        module ResultsReporter
+          include Concerns::Utils
+
+          TEST_CASE_RESULTS_SECTION_TEMPLATE = "\n\n### DO NOT EDIT BELOW THIS LINE\n\nActive and historical test results:"
+
+          def find_issue(test)
+            issues = search_for_issues(test)
+
+            warn(%(Too many #{issue_type}s found with the file path "#{test.file}" and name "#{test.name}")) if issues.many?
+
+            puts "Found existing #{issue_type}: #{issues.first.web_url}" unless issues.empty?
+
+            issues.first
+          end
+
+          def find_issue_by_iid(iid)
+            issues = gitlab.find_issues(iid: iid) do |issue|
+              issue.state == 'opened' && issue.issue_type == issue_type
+            end
+
+            warn(%(#{issue_type} iid "#{iid}" not valid)) if issues.empty?
+
+            issues.first
+          end
+
+          def issue_title_needs_updating?(issue, test)
+            issue.title.strip != title_from_test(test) && !%w[canary production preprod release].include?(pipeline)
+          end
+
+          def new_issue_labels(_test)
+            %w[Quality status::automated]
+          end
+
+          def up_to_date_labels(test:, issue: nil, new_labels: Set.new)
+            labels = super
+            labels |= new_issue_labels(test).to_set
+            labels.delete_if { |label| label.start_with?("#{pipeline}::") }
+            labels << (test.failures.empty? ? "#{pipeline}::passed" : "#{pipeline}::failed")
+          end
+
+          def update_issue_title(issue, test)
+            old_title = issue.title.strip
+            new_title = title_from_test(test)
+
+            warn(%(#{issue_type} title needs to be updated from '#{old_title}' to '#{new_title}'))
+
+            new_description = updated_description(issue, test)
+
+            gitlab.edit_issue(iid: issue.iid, options: { title: new_title, description: new_description })
+          end
+
+          private
+
+          def search_term(test)
+            %("#{partial_file_path(test.file)}" "#{search_safe(test.name)}")
+          end
+
+          def search_for_issues(test)
+            gitlab.find_issues(options: { search: search_term(test) }) do |issue|
+              issue.state == 'opened' && issue.issue_type == issue_type && issue.title.strip == title_from_test(test)
+            end
+          end
+        end
+      end
+    end
+  end
+end

data/lib/gitlab_quality/test_tooling/report/concerns/utils.rb

@@ -0,0 +1,49 @@
+# frozen_string_literal: true
+
+module GitlabQuality
+  module TestTooling
+    module Report
+      module Concerns
+        module Utils
+          MAX_TITLE_LENGTH = 255
+
+          def partial_file_path(path)
+            path.match(/((ee|api|browser_ui).*)/i)[1]
+          end
+
+          def new_issue_title(test)
+            "#{partial_file_path(test.file)} | #{search_safe(test.name)}".strip
+          end
+
+          def title_from_test(test)
+            title = new_issue_title(test)
+
+            return title unless title.length > MAX_TITLE_LENGTH
+
+            "#{title[0...MAX_TITLE_LENGTH - 3]}..."
+          end
+
+          def search_safe(value)
+            value.delete('"')
+          end
+
+          def pipeline
+            # Gets the name of the pipeline the test was run in, to be used as the key of a scoped label
+            #
+            # Tests can be run in several pipelines:
+            #   gitlab, nightly, staging, canary, production, preprod, MRs, and the default branch (master/main)
+            #
+            # Some of those run in their own project, so CI_PROJECT_NAME is the name we need. Those are:
+            #   nightly, staging, canary, production, and preprod
+            #
+            # MR, master/main, and gitlab tests run in gitlab-qa, but we only want to report tests run on
+            # master/main because the other pipelines will be monitored by the author of the MR that triggered them.
+            # So we assume that we're reporting a master/main pipeline if the project name is 'gitlab'.
+
+            @pipeline ||= Runtime::Env.pipeline_from_project_name
+          end
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,275 @@
+# frozen_string_literal: true
+
+require 'erb'
+require 'date'
+
+module GitlabQuality
+  module TestTooling
+    module Report
+      class GenerateTestSession < ReportAsIssue
+        def initialize(**kwargs)
+          super
+          @issue_type = 'issue'
+        end
+
+        private
+
+        # rubocop:disable Metrics/AbcSize
+        def run!
+          puts "Generating test results in `#{files.join(',')}` as issues in project `#{project}` via the API at `#{Runtime::Env.gitlab_api_base}`."
+
+          tests = Dir.glob(files).flat_map do |path|
+            puts "Loading tests in #{path}"
+
+            TestResults::JsonTestResults.new(path).to_a
+          end
+
+          issue = gitlab.create_issue(
+            title: "#{Time.now.strftime('%Y-%m-%d')} Test session report | #{Runtime::Env.qa_run_type}",
+            description: generate_description(tests),
+            labels: ['Quality', 'QA', 'triage report', pipeline_name_label]
+          )
+
+          # Workaround for https://gitlab.com/gitlab-org/gitlab/-/issues/295493
+          unless Runtime::Env.qa_issue_url.to_s.empty?
+            gitlab.create_issue_note(
+              iid: issue.iid,
+              note: "/relate #{Runtime::Env.qa_issue_url}")
+          end
+
+          File.write('REPORT_ISSUE_URL', issue.web_url)
+        end
+        # rubocop:enable Metrics/AbcSize
+
+        def generate_description(tests)
+          <<~MARKDOWN.rstrip
+          ## Session summary
+
+          * Deploy version: #{Runtime::Env.deploy_version}
+          * Deploy environment: #{Runtime::Env.deploy_environment}
+          * Pipeline: #{Runtime::Env.pipeline_from_project_name} [#{Runtime::Env.ci_pipeline_id}](#{Runtime::Env.ci_pipeline_url})
+          #{generate_summary(tests: tests)}
+
+          #{generate_failed_jobs_listing}
+
+          #{generate_stages_listing(tests)}
+
+          #{generate_qa_issue_relation}
+
+          #{generate_link_to_dashboard}
+          MARKDOWN
+        end
+
+        def generate_summary(tests:, tests_by_status: nil)
+          tests_by_status ||= tests.group_by(&:status)
+          total = tests.size
+          passed = tests_by_status['passed']&.size || 0
+          failed = tests_by_status['failed']&.size || 0
+          others = total - passed - failed
+
+          <<~MARKDOWN.chomp
+          * Total #{total} tests
+          * Passed #{passed} tests
+          * Failed #{failed} tests
+          * #{others} other tests (usually skipped)
+          MARKDOWN
+        end
+
+        def generate_failed_jobs_listing
+          failed_jobs = []
+
+          client = Gitlab.client(
+            endpoint: Runtime::Env.ci_api_v4_url,
+            private_token: Runtime::Env.gitlab_ci_api_token)
+
+          gitlab.handle_gitlab_client_exceptions do
+            failed_jobs = client.pipeline_jobs(
+              Runtime::Env.ci_project_id,
+              Runtime::Env.ci_pipeline_id,
+              scope: 'failed')
+          end
+
+          listings = failed_jobs.map do |job|
+            allowed_to_fail = ' (allowed to fail)' if job.allow_failure
+
+            "* [#{job.name}](#{job.web_url})#{allowed_to_fail}"
+          end.join("\n")
+
+          <<~MARKDOWN.chomp if failed_jobs.any?
+          ## Failed jobs
+
+          #{listings}
+          MARKDOWN
+        end
+
+        def generate_stages_listing(tests)
+          generate_tests_by_stage(tests).map do |stage, tests_for_stage|
+            tests_by_status = tests_for_stage.group_by(&:status)
+
+            <<~MARKDOWN.chomp
+            ### #{stage&.capitalize || 'Unknown'}
+
+            #{generate_summary(
+              tests: tests_for_stage, tests_by_status: tests_by_status)}
+
+            #{generate_testcase_listing_by_status(
+              tests: tests_for_stage, tests_by_status: tests_by_status)}
+            MARKDOWN
+          end.join("\n\n")
+        end
+
+        def generate_tests_by_stage(tests)
+          # https://about.gitlab.com/handbook/product/product-categories/#devops-stages
+          ordering = %w[
+            manage
+            plan
+            create
+            verify
+            package
+            release
+            configure
+            monitor
+            secure
+            defend
+            growth
+            fulfillment
+            enablement
+          ]
+
+          tests.sort_by do |test|
+            ordering.index(test.stage) || ordering.size
+          end.group_by(&:stage)
+        end
+
+        def generate_testcase_listing_by_status(tests:, tests_by_status:)
+          failed_tests = tests_by_status['failed']
+          passed_tests = tests_by_status['passed']
+          other_tests = tests.reject do |test|
+            test.status == 'failed' || test.status == 'passed'
+          end
+
+          [
+            (failed_listings(failed_tests) if failed_tests),
+            (passed_listings(passed_tests) if passed_tests),
+            (other_listings(other_tests) if other_tests.any?)
+          ].compact.join("\n\n")
+        end
+
+        def failed_listings(failed_tests)
+          generate_testcase_listing(failed_tests)
+        end
+
+        def passed_listings(passed_tests)
+          <<~MARKDOWN.chomp
+            <details><summary>Passed tests:</summary>
+
+            #{generate_testcase_listing(passed_tests, passed: true)}
+
+            </details>
+          MARKDOWN
+        end
+
+        def other_listings(other_tests)
+          <<~MARKDOWN.chomp
+            <details><summary>Other tests:</summary>
+
+            #{generate_testcase_listing(other_tests)}
+
+            </details>
+          MARKDOWN
+        end
+
+        def generate_testcase_listing(tests, passed: false)
+          body = tests.group_by(&:testcase).map do |testcase, tests_with_same_testcase|
+            tests_with_same_testcase.sort_by!(&:name)
+            [
+              generate_test_text(testcase, tests_with_same_testcase, passed),
+              generate_test_job(tests_with_same_testcase),
+              generate_test_status(tests_with_same_testcase),
+              generate_test_actions(tests_with_same_testcase)
+            ].join(' | ')
+          end.join("\n")
+
+          <<~MARKDOWN.chomp
+          | Test | Job | Status | Action |
+          | - | - | - | - |
+          #{body}
+          MARKDOWN
+        end
+
+        def generate_test_text(testcase, tests_with_same_testcase, passed)
+          text = tests_with_same_testcase.map(&:name).uniq.join(', ')
+          encoded_text = ERB::Util.url_encode(text)
+
+          if testcase && !passed
+            # Workaround for reducing system notes on testcase issues
+            # The first regex extracts the link to the issues list page from a link to a single issue show page by removing the issue id.
+            "[#{text}](#{testcase.match(%r{[\s\S]+/[^/\d]+})}?state=opened&search=#{encoded_text})"
+          else
+            text
+          end
+        end
+
+        def generate_test_job(tests_with_same_testcase)
+          tests_with_same_testcase.map do |test|
+            ci_job_id = test.ci_job_url[/\d+\z/]
+
+            "[#{ci_job_id}](#{test.ci_job_url})#{' ~"quarantine"' if test.quarantine?}"
+          end.uniq.join(', ')
+        end
+
+        def generate_test_status(tests_with_same_testcase)
+          tests_with_same_testcase.map(&:status).uniq.map do |status|
+            %(~"#{status}")
+          end.join(', ')
+        end
+
+        def generate_test_actions(tests_with_same_testcase)
+          # All failed tests would be grouped together, meaning that
+          # if one failed, all the tests here would be failed too.
+          # So this check is safe. Same applies to 'passed'.
+          # But all other status might be mixing together,
+          # we cannot assume other statuses.
+          if tests_with_same_testcase.first.status == 'failed'
+            tests_having_failure_issue =
+              tests_with_same_testcase.select(&:failure_issue)
+
+            if tests_having_failure_issue.any?
+              items = tests_having_failure_issue.uniq(&:failure_issue).map do |test|
+                "<li>[ ] [failure issue](#{test.failure_issue})</li>"
+              end.join(' ')
+
+              "<ul>#{items}</ul>"
+            else
+              '<ul><li>[ ] failure issue exists or was created</li></ul>'
+            end
+          else
+            '-'
+          end
+        end
+
+        def generate_qa_issue_relation
+          return unless Runtime::Env.qa_issue_url
+
+          <<~MARKDOWN.chomp
+          ## Release QA issue
+
+          * #{Runtime::Env.qa_issue_url}
+
+          /relate #{Runtime::Env.qa_issue_url}
+          MARKDOWN
+        end
+
+        def generate_link_to_dashboard
+          return unless Runtime::Env.qa_run_type
+
+          <<~MARKDOWN.chomp
+          ## Link to Grafana dashboard for run-type of #{Runtime::Env.qa_run_type}
+
+          * https://dashboards.quality.gitlab.net/d/kuNYMgDnz/test-run-metrics?orgId=1&refresh=1m&var-run_type=#{Runtime::Env.qa_run_type}
+          MARKDOWN
+        end
+      end
+    end
+  end
+end

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

@@ -0,0 +1,79 @@
+# frozen_string_literal: true
+
+require 'nokogiri'
+
+module GitlabQuality
+  module TestTooling
+    module Report
+      # Create a new JUnit report file for each Stage, containing tests from that Stage alone
+      class PrepareStageReports
+        EXTRACT_STAGE_FROM_TEST_FILE_REGEX = %r{(?:api|browser_ui)/(?:[0-9]+_)?(?<stage>[_\w]+)/}i
+
+        def initialize(junit_files:)
+          @junit_files = junit_files
+        end
+
+        def invoke!
+          collate_test_cases.each do |stage, tests|
+            filename = "#{stage}.xml"
+
+            File.write(filename, junit_report(tests).to_s)
+
+            puts "Saved #{filename}"
+          end
+        end
+
+        private
+
+        attr_reader :junit_files
+
+        # Collect the test cases from the original reports and group them by Stage
+        def collate_test_cases
+          Dir.glob(junit_files)
+            .each_with_object(Hash.new { |h, k| h[k] = [] }) do |junit_file, test_cases|
+            report = Nokogiri::XML(File.open(junit_file))
+            report.xpath('//testcase').each do |test_case|
+              # The test file paths could start with any of
+              #  /qa/specs/features/api/<stage>
+              #  /qa/specs/features/browser_ui/<stage>
+              #  /qa/specs/features/ee/api/<stage>
+              #  /qa/specs/features/ee/browser_ui/<stage>
+              # For now we assume the Stage is whatever follows api/ or browser_ui/
+              test_file_match = test_case['file'].match(EXTRACT_STAGE_FROM_TEST_FILE_REGEX)
+              next unless test_file_match
+
+              stage = test_file_match[:stage]
+              test_cases[stage] << test_case
+            end
+          end
+        end
+
+        def junit_report(test_cases)
+          Nokogiri::XML::Document.new.tap do |report|
+            test_suite_node = report.create_element('testsuite', name: 'rspec', **collect_stats(test_cases))
+            report.root = test_suite_node
+
+            test_cases.each do |test_case|
+              test_suite_node.add_child(test_case)
+            end
+          end
+        end
+
+        def collect_stats(test_cases)
+          stats = {
+            tests: test_cases.size,
+            failures: 0,
+            errors: 0,
+            skipped: 0
+          }
+
+          test_cases.each_with_object(stats) do |test_case, memo|
+            memo[:failures] += 1 unless test_case.search('failure').empty?
+            memo[:errors] += 1 unless test_case.search('error').empty?
+            memo[:skipped] += 1 unless test_case.search('skipped').empty?
+          end
+        end
+      end
+    end
+  end
+end