diff options
Diffstat (limited to 'qa/spec/tools/ci/qa_changes_spec.rb')
-rw-r--r-- | qa/spec/tools/ci/qa_changes_spec.rb | 52 |
1 files changed, 50 insertions, 2 deletions
diff --git a/qa/spec/tools/ci/qa_changes_spec.rb b/qa/spec/tools/ci/qa_changes_spec.rb index bc98ec16d7f..d93d3cd9258 100644 --- a/qa/spec/tools/ci/qa_changes_spec.rb +++ b/qa/spec/tools/ci/qa_changes_spec.rb @@ -1,9 +1,10 @@ # frozen_string_literal: true RSpec.describe QA::Tools::Ci::QaChanges do - subject(:qa_changes) { described_class.new(mr_diff, mr_labels) } + subject(:qa_changes) { described_class.new(mr_diff, mr_labels, additional_group_spec_list) } let(:mr_labels) { [] } + let(:additional_group_spec_list) { [] } before do allow(File).to receive(:directory?).and_return(false) @@ -35,7 +36,7 @@ RSpec.describe QA::Tools::Ci::QaChanges do context "with framework changes" do let(:mr_diff) { [{ path: "qa/qa.rb" }] } - it ".qa_tests do not return specifix specs" do + it ".qa_tests do not return specific specs" do expect(qa_changes.qa_tests).to be_nil end @@ -75,6 +76,43 @@ RSpec.describe QA::Tools::Ci::QaChanges do ) end end + + context "when configured to run tests from other stages" do + let(:additional_group_spec_list) do + { + 'foo' => %w[create], + 'bar' => %w[monitor verify] + } + end + + context "with a single extra stage configured for the group name" do + let(:mr_labels) { %w[devops::manage group::foo] } + + it ".qa_tests return specs for both devops stage and create stage" do + expect(qa_changes.qa_tests.split(" ")).to include( + "qa/specs/features/browser_ui/1_manage/", + "qa/specs/features/api/1_manage/", + "qa/specs/features/browser_ui/3_create/", + "qa/specs/features/api/3_create/" + ) + end + end + + context "with a multiple extra stages configured for the group name" do + let(:mr_labels) { %w[devops::manage group::bar] } + + it ".qa_tests return specs for both devops stage and multiple other stages" do + expect(qa_changes.qa_tests.split(" ")).to include( + "qa/specs/features/browser_ui/1_manage/", + "qa/specs/features/api/1_manage/", + "qa/specs/features/browser_ui/8_monitor/", + "qa/specs/features/api/8_monitor/", + "qa/specs/features/browser_ui/4_verify/", + "qa/specs/features/api/4_verify/" + ) + end + end + end end context "with quarantine changes" do @@ -84,4 +122,14 @@ RSpec.describe QA::Tools::Ci::QaChanges do expect(qa_changes.quarantine_changes?).to eq(true) end end + + %w[GITALY_SERVER_VERSION Gemfile.lock yarn.lock Dockerfile.assets].each do |dependency_file| + context "when #{dependency_file} change" do + let(:mr_diff) { [{ path: dependency_file }] } + + it ".qa_tests do not return specific specs" do + expect(qa_changes.qa_tests).to be_nil + end + end + end end |