diff options
author | GitLab Bot <gitlab-bot@gitlab.com> | 2020-04-14 18:09:54 +0000 |
---|---|---|
committer | GitLab Bot <gitlab-bot@gitlab.com> | 2020-04-14 18:09:54 +0000 |
commit | f697dc5e76dfc5894df006d53b2b7e751653cf05 (patch) | |
tree | 1387cd225039e611f3683f96b318bb17d4c422cb /qa/qa/specs | |
parent | 874ead9c3a50de4c4ca4551eaf5b7eb976d26b50 (diff) | |
download | gitlab-ce-f697dc5e76dfc5894df006d53b2b7e751653cf05.tar.gz |
Add latest changes from gitlab-org/gitlab@master
Diffstat (limited to 'qa/qa/specs')
-rw-r--r-- | qa/qa/specs/features/browser_ui/4_verify/ci_variable/add_remove_ci_variable_spec.rb | 23 |
1 files changed, 16 insertions, 7 deletions
diff --git a/qa/qa/specs/features/browser_ui/4_verify/ci_variable/add_remove_ci_variable_spec.rb b/qa/qa/specs/features/browser_ui/4_verify/ci_variable/add_remove_ci_variable_spec.rb index cff415dcf97..f7a6c8411db 100644 --- a/qa/qa/specs/features/browser_ui/4_verify/ci_variable/add_remove_ci_variable_spec.rb +++ b/qa/qa/specs/features/browser_ui/4_verify/ci_variable/add_remove_ci_variable_spec.rb @@ -2,7 +2,7 @@ module QA context 'Verify' do - describe 'Add or Remove CI variable via UI', :smoke, quarantine: { issue: 'https://gitlab.com/gitlab-org/gitlab/issues/207915', type: :stale } do + describe 'Add or Remove CI variable via UI', :smoke do let!(:project) do Resource::Project.fabricate_via_api! do |project| project.name = 'project-with-ci-variables' @@ -10,6 +10,14 @@ module QA end end + before(:all) do + Runtime::Feature.enable_and_verify('new_variables_ui') + end + + after(:all) do + Runtime::Feature.remove('new_variables_ui') + end + before do Flow::Login.sign_in add_ci_variable @@ -19,12 +27,12 @@ module QA it 'user adds a CI variable' do Page::Project::Settings::CICD.perform do |settings| settings.expand_ci_variables do |page| - expect(page).to have_field(with: 'VARIABLE_KEY') - expect(page).not_to have_field(with: 'some_CI_variable') + expect(page).to have_text('VARIABLE_KEY') + expect(page).not_to have_text('some_CI_variable') - page.reveal_variables + page.click_reveal_ci_variable_value_button - expect(page).to have_field(with: 'some_CI_variable') + expect(page).to have_text('some_CI_variable') end end end @@ -32,9 +40,10 @@ module QA it 'user removes a CI variable' do Page::Project::Settings::CICD.perform do |settings| settings.expand_ci_variables do |page| - page.remove_variable + page.click_edit_ci_variable + page.click_ci_variable_delete_button - expect(page).not_to have_field(with: 'VARIABLE_KEY') + expect(page).not_to have_text('VARIABLE_KEY') end end end |