Commit 7a48a06c authored by Sanad Liaquat's avatar Sanad Liaquat

Merge branch 'mc/fix/ci-variable-qa' into 'master'

Remove spaces from  variable name in CI variable QA test

Closes gitlab-org/quality/nightly#91

See merge request gitlab-org/gitlab-ce!27042
parents 3962b00b fd8f0c5c
# frozen_string_literal: true # frozen_string_literal: true
module QA module QA
# Failure issue: https://gitlab.com/gitlab-org/quality/nightly/issues/91 context 'Verify' do
context 'Verify', :quarantine do
describe 'CI variable support' do describe 'CI variable support' do
it 'user adds a CI variable' do it 'user adds a CI variable' do
Runtime::Browser.visit(:gitlab, Page::Main::Login) Runtime::Browser.visit(:gitlab, Page::Main::Login)
...@@ -10,17 +9,17 @@ module QA ...@@ -10,17 +9,17 @@ module QA
Resource::CiVariable.fabricate! do |resource| Resource::CiVariable.fabricate! do |resource|
resource.key = 'VARIABLE_KEY' resource.key = 'VARIABLE_KEY'
resource.value = 'some CI variable' resource.value = 'some_CI_variable'
end end
Page::Project::Settings::CICD.perform do |settings| Page::Project::Settings::CICD.perform do |settings|
settings.expand_ci_variables do |page| settings.expand_ci_variables do |page|
expect(page).to have_field(with: 'VARIABLE_KEY') expect(page).to have_field(with: 'VARIABLE_KEY')
expect(page).not_to have_field(with: 'some CI variable') expect(page).not_to have_field(with: 'some_CI_variable')
page.reveal_variables page.reveal_variables
expect(page).to have_field(with: 'some CI variable') expect(page).to have_field(with: 'some_CI_variable')
end end
end end
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