diff options
author | Stan Hu <stanhu@gmail.com> | 2019-03-05 22:27:22 +0000 |
---|---|---|
committer | Stan Hu <stanhu@gmail.com> | 2019-03-05 22:27:22 +0000 |
commit | d11b112664db578327d028cb4552f408e89222d0 (patch) | |
tree | 117609c9a4872df2173094f3487dbb671222c9b9 /spec/models/ci | |
parent | a592a78072bb44fed1a25c25f2cabdc4cf4bc0bd (diff) | |
parent | a5ac7e7a0c9aa5e6c9a429444d1ae2a90718eb94 (diff) | |
download | gitlab-ce-d11b112664db578327d028cb4552f408e89222d0.tar.gz |
Merge branch 'fix/gb/fix-bridge-jobs-variables-policy' into 'master'
Fix bridge jobs only/except variables policy
Closes #58199
See merge request gitlab-org/gitlab-ce!25710
Diffstat (limited to 'spec/models/ci')
-rw-r--r-- | spec/models/ci/bridge_spec.rb | 15 |
1 files changed, 15 insertions, 0 deletions
diff --git a/spec/models/ci/bridge_spec.rb b/spec/models/ci/bridge_spec.rb index 741cdfef1a5..b5ec8991720 100644 --- a/spec/models/ci/bridge_spec.rb +++ b/spec/models/ci/bridge_spec.rb @@ -22,4 +22,19 @@ describe Ci::Bridge do expect(status).to be_a Gitlab::Ci::Status::Success end end + + describe '#scoped_variables_hash' do + it 'returns a hash representing variables' do + variables = %w[ + CI_JOB_NAME CI_JOB_STAGE CI_COMMIT_SHA CI_COMMIT_SHORT_SHA + CI_COMMIT_BEFORE_SHA CI_COMMIT_REF_NAME CI_COMMIT_REF_SLUG + CI_PROJECT_ID CI_PROJECT_NAME CI_PROJECT_PATH + CI_PROJECT_PATH_SLUG CI_PROJECT_NAMESPACE CI_PIPELINE_IID + CI_CONFIG_PATH CI_PIPELINE_SOURCE CI_COMMIT_MESSAGE + CI_COMMIT_TITLE CI_COMMIT_DESCRIPTION + ] + + expect(bridge.scoped_variables_hash.keys).to include(*variables) + end + end end |