Welcome to mirror list, hosted at ThFree Co, Russian Federation.

gitlab.com/gitlab-org/gitlab-foss.git - Unnamed repository; edit this file 'description' to name the repository.
summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
Diffstat (limited to 'qa/spec/tools/ci')
-rw-r--r--qa/spec/tools/ci/ff_changes_spec.rb51
-rw-r--r--qa/spec/tools/ci/non_empty_suites_spec.rb19
-rw-r--r--qa/spec/tools/ci/qa_changes_spec.rb87
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