Commit 14527f00 authored by treagitlab's avatar treagitlab

Lift restrictions from tests

Remove FF and let include_multiple_files_from_a_project run in prod
Remove FF and let pass_dotenv_variables_to_downstream run in prod
Let run_pipeline_via_web_only run in prod
Remove FF and let trigger_child_pipeline_with_manual run in prod
parent 098b6eb0
...@@ -3,9 +3,8 @@ ...@@ -3,9 +3,8 @@
require 'faker' require 'faker'
module QA module QA
RSpec.describe 'Verify', :runner, :requires_admin, :skip_live_env do RSpec.describe 'Verify', :runner do
describe "Include multiple files from a project" do describe 'Include multiple files from a project' do
let(:feature_flag) { :ci_include_multiple_files_from_project }
let(:executor) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" } let(:executor) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" }
let(:expected_text) { Faker::Lorem.sentence } let(:expected_text) { Faker::Lorem.sentence }
let(:unexpected_text) { Faker::Lorem.sentence } let(:unexpected_text) { Faker::Lorem.sentence }
...@@ -31,7 +30,6 @@ module QA ...@@ -31,7 +30,6 @@ module QA
end end
before do before do
Runtime::Feature.enable(feature_flag)
Flow::Login.sign_in Flow::Login.sign_in
add_included_files add_included_files
add_main_ci_file add_main_ci_file
...@@ -40,7 +38,6 @@ module QA ...@@ -40,7 +38,6 @@ module QA
end end
after do after do
Runtime::Feature.disable(feature_flag)
runner.remove_via_api! runner.remove_via_api!
end end
......
...@@ -3,9 +3,8 @@ ...@@ -3,9 +3,8 @@
require 'faker' require 'faker'
module QA module QA
RSpec.describe 'Verify', :runner, :requires_admin do RSpec.describe 'Verify', :runner do
describe "Pass dotenv variables to downstream via bridge" do describe 'Pass dotenv variables to downstream via bridge' do
let(:feature_flag) { :ci_bridge_dependency_variables }
let(:executor_1) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" } let(:executor_1) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" }
let(:executor_2) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" } let(:executor_2) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" }
...@@ -38,7 +37,6 @@ module QA ...@@ -38,7 +37,6 @@ module QA
end end
before do before do
Runtime::Feature.enable(feature_flag)
Flow::Login.sign_in Flow::Login.sign_in
add_ci_file(downstream_project, downstream_ci_file) add_ci_file(downstream_project, downstream_ci_file)
add_ci_file(upstream_project, upstream_ci_file) add_ci_file(upstream_project, upstream_ci_file)
...@@ -47,7 +45,6 @@ module QA ...@@ -47,7 +45,6 @@ module QA
end end
after do after do
Runtime::Feature.disable(feature_flag)
runner_1.remove_via_api! runner_1.remove_via_api!
runner_2.remove_via_api! runner_2.remove_via_api!
end end
......
...@@ -2,7 +2,7 @@ ...@@ -2,7 +2,7 @@
module QA module QA
RSpec.describe 'Verify' do RSpec.describe 'Verify' do
describe 'Run pipeline', only: { subdomain: :staging } do describe 'Run pipeline' do
context 'with web only rule' do context 'with web only rule' do
let(:job_name) { 'test_job' } let(:job_name) { 'test_job' }
let(:project) do let(:project) do
......
...@@ -3,11 +3,8 @@ ...@@ -3,11 +3,8 @@
require 'faker' require 'faker'
module QA module QA
RSpec.describe 'Verify', :runner, :requires_admin do RSpec.describe 'Verify', :runner do
# [TODO]: Developer to remove :requires_admin once FF is removed in follow up issue
describe "Trigger child pipeline with 'when:manual'" do describe "Trigger child pipeline with 'when:manual'" do
let(:feature_flag) { :ci_manual_bridges } # [TODO]: Developer to remove when feature flag is removed
let(:executor) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" } let(:executor) { "qa-runner-#{Faker::Alphanumeric.alphanumeric(8)}" }
let(:project) do let(:project) do
...@@ -25,7 +22,6 @@ module QA ...@@ -25,7 +22,6 @@ module QA
end end
before do before do
Runtime::Feature.enable(feature_flag) # [TODO]: Developer to remove when feature flag is removed
Flow::Login.sign_in Flow::Login.sign_in
add_ci_files add_ci_files
project.visit! project.visit!
...@@ -33,7 +29,6 @@ module QA ...@@ -33,7 +29,6 @@ module QA
end end
after do after do
Runtime::Feature.disable(feature_flag) # [TODO]: Developer to remove when feature flag is removed
runner.remove_via_api! runner.remove_via_api!
end end
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment