Merge branch 'qa-move-testcase-cop' into 'master'

Separate out testcase cop execution

See merge request gitlab-org/gitlab!74452

.gitlab/ci/static-analysis.gitlab-ci.yml

@@ -93,6 +93,14 @@ rubocop:
   script:
     - run_timed_command "bundle exec rubocop --parallel"
 
+qa:testcases:
+  extends:
+    - .static-analysis-base
+    - .rubocop-job-cache
+    - .static-analysis:rules:ee-and-foss
+  script:
+    - run_timed_command "bundle exec rubocop qa/qa/specs/features/**/* --only QA/DuplicateTestcaseLink"
+
 feature-flags-usage:
   extends:
     - .static-analysis-base

.rubocop.yml

@@ -734,6 +734,10 @@ QA/SelectorUsage:
   Exclude:
     - 'spec/rubocop/**/*_spec.rb'
 
+QA/DuplicateTestcaseLink:
+  # this cop is executed in static-analysis.gitlab-ci.yml since it cannot be run in parallel
+  Enabled: false
+
 Performance/ActiveRecordSubtransactions:
   Exclude:
     - 'spec/**/*.rb'

qa/qa/specs/features/ee/browser_ui/10_protect/policies_list_spec.rb

@@ -23,7 +23,7 @@ module QA
           project.visit!
         end
 
-        it 'can load Policies page and view the policies list', :smoke, testcase: 'https://gitlab.com/gitlab-org/quality/testcases/-/quality/test_cases/1892' do
+        it 'can load Policies page and view the policies list', :smoke, testcase: 'https://gitlab.com/gitlab-org/quality/testcases/-/quality/test_cases/2382' do
           Page::Project::Menu.perform(&:click_on_policies)
 
           EE::Page::Project::Policies::Index.perform do |policies_page|

rubocop/cop/qa/duplicate_testcase_link.rb

 # frozen_string_literal: true
 
-require_relative '../../qa_helpers'
-
 module RuboCop
   module Cop
     module QA
@@ -17,8 +15,6 @@ module RuboCop
       #   it 'some test', testcase: '(...)/quality/test_cases/1892'
       #   it 'another test, testcase: '(...)/quality/test_cases/1894'
       class DuplicateTestcaseLink < RuboCop::Cop::Cop
-        include QAHelpers
-
         MESSAGE = "Don't reuse the same testcase link in different tests. Replace one of `%s`."
 
         @testcase_set = Set.new
@@ -34,8 +30,6 @@ module RuboCop
         PATTERN
 
         def on_block(node)
-          return unless in_qa_file?(node)
-
           duplicate_testcase_link(node) do |link|
             break unless self.class.duplicate?(link)