diff options
Diffstat (limited to 'qa/spec/tools/ci')
-rw-r--r-- | qa/spec/tools/ci/ff_changes_spec.rb | 51 | ||||
-rw-r--r-- | qa/spec/tools/ci/non_empty_suites_spec.rb | 19 | ||||
-rw-r--r-- | qa/spec/tools/ci/qa_changes_spec.rb | 87 |
3 files changed, 157 insertions, 0 deletions
diff --git a/qa/spec/tools/ci/ff_changes_spec.rb b/qa/spec/tools/ci/ff_changes_spec.rb new file mode 100644 index 00000000000..71ca26867e0 --- /dev/null +++ b/qa/spec/tools/ci/ff_changes_spec.rb @@ -0,0 +1,51 @@ +# frozen_string_literal: true + +RSpec.describe QA::Tools::Ci::FfChanges do + subject(:ff_changes) { described_class.new(mr_diff) } + + before do + allow(Gitlab::QA::TestLogger).to receive(:logger).and_return(Logger.new(StringIO.new)) + end + + context "with merge request pipeline" do + let(:deleted_file) { false } + let(:mr_diff) do + [ + { + path: "config/feature_flags/development/bulk_import_projects.yml", + deleted_file: deleted_file + } + ] + end + + before do + allow(File).to receive(:read) + .with(File.expand_path("../#{mr_diff.first[:path]}", QA::Runtime::Path.qa_root)) + .and_return(File.read("spec/fixtures/ff/bulk_import_projects.yml")) + end + + context "with changed feature flag" do + it "returns inverse ff state option" do + expect(ff_changes.fetch).to eq("bulk_import_projects=enabled") + end + end + + context "with deleted feature flag" do + let(:deleted_file) { true } + + it "returns deleted ff state option" do + expect(ff_changes.fetch).to eq("bulk_import_projects=deleted") + end + end + end + + context "without merge request pipeline" do + let(:mr_diff) { [] } + + context "with empty mr diff" do + it "doesn't return any ff options" do + expect(ff_changes.fetch).to be_nil + end + end + end +end diff --git a/qa/spec/tools/ci/non_empty_suites_spec.rb b/qa/spec/tools/ci/non_empty_suites_spec.rb new file mode 100644 index 00000000000..d9bfe1eebe7 --- /dev/null +++ b/qa/spec/tools/ci/non_empty_suites_spec.rb @@ -0,0 +1,19 @@ +# frozen_string_literal: true + +RSpec.describe QA::Tools::Ci::NonEmptySuites do + let(:non_empty_suites) { described_class.new(nil) } + + let(:status) { instance_double(Process::Status, success?: true) } + + before do + allow(Gitlab::QA::TestLogger).to receive(:logger).and_return(Logger.new(StringIO.new)) + allow(Open3).to receive(:capture3).and_return(["output\n0", "", status]) + allow(Open3).to receive(:capture3) + .with("bundle exec bin/qa Test::Instance::All --count-examples-only --address http://dummy1.test") + .and_return(["output\n1", "", status]) + end + + it "returns runnable test suites" do + expect(non_empty_suites.fetch).to eq("Test::Instance::All") + end +end diff --git a/qa/spec/tools/ci/qa_changes_spec.rb b/qa/spec/tools/ci/qa_changes_spec.rb new file mode 100644 index 00000000000..bc98ec16d7f --- /dev/null +++ b/qa/spec/tools/ci/qa_changes_spec.rb @@ -0,0 +1,87 @@ +# frozen_string_literal: true + +RSpec.describe QA::Tools::Ci::QaChanges do + subject(:qa_changes) { described_class.new(mr_diff, mr_labels) } + + let(:mr_labels) { [] } + + before do + allow(File).to receive(:directory?).and_return(false) + end + + context "with spec only changes" do + let(:mr_diff) do + [ + { path: "qa/qa/specs/features/test_spec.rb", diff: "" }, + { path: "qa/qa/specs/features/another_test_spec.rb", diff: "" } + ] + end + + it ".qa_tests return changed specs" do + expect(qa_changes.qa_tests).to eq( + "qa/specs/features/test_spec.rb qa/specs/features/another_test_spec.rb" + ) + end + + it ".framework_changes? return false" do + expect(qa_changes.framework_changes?).to eq(false) + end + + it ".quarantine_changes? return false" do + expect(qa_changes.quarantine_changes?).to eq(false) + end + end + + context "with framework changes" do + let(:mr_diff) { [{ path: "qa/qa.rb" }] } + + it ".qa_tests do not return specifix specs" do + expect(qa_changes.qa_tests).to be_nil + end + + it ".framework_changes? return true" do + expect(qa_changes.framework_changes?).to eq(true) + end + + it ".quarantine_changes? return false" do + expect(qa_changes.quarantine_changes?).to eq(false) + end + end + + context "with non qa changes" do + let(:mr_diff) { [{ path: "Gemfile" }] } + + it ".framework_changes? return false" do + expect(qa_changes.framework_changes?).to eq(false) + end + + it ".quarantine_changes? return false" do + expect(qa_changes.quarantine_changes?).to eq(false) + end + + context "without mr labels" do + it ".qa_tests do not return any specific specs" do + expect(qa_changes.qa_tests).to be_nil + end + end + + context "with mr label" do + let(:mr_labels) { ["devops::manage"] } + + it ".qa_tests return specs for devops stage" do + expect(qa_changes.qa_tests.split(" ")).to include( + "qa/specs/features/browser_ui/1_manage/", + "qa/specs/features/api/1_manage/" + ) + end + end + end + + context "with quarantine changes" do + let(:mr_diff) { [{ path: "qa/qa/specs/features/test_spec.rb", diff: "+ , quarantine: true" }] } + + it ".quarantine_changes? return true" do + expect(qa_changes.quarantine_changes?).to eq(true) + end + end +end |