diff options
author | Sean McGivern <sean@mcgivern.me.uk> | 2016-12-20 10:14:20 +0000 |
---|---|---|
committer | Sean McGivern <sean@mcgivern.me.uk> | 2016-12-20 10:14:20 +0000 |
commit | deb74f73d9432c90649142cf8333c5cd5d0984ea (patch) | |
tree | e4928ef04afee97dd500b5e609f2290ad2d14bb2 /spec | |
parent | 52278412c7350b8087ef4ffc688c79e4593369cc (diff) | |
parent | 212967aefb55e0792a36e67a881b66c8bd871e9f (diff) | |
download | gitlab-ce-deb74f73d9432c90649142cf8333c5cd5d0984ea.tar.gz |
Merge branch '25848-fix-git-rev-list-env-parsing' into 'master'
Reject blank environment variables in Gitlab::Git::RevList
Closes #25848
See merge request !8189
Diffstat (limited to 'spec')
-rw-r--r-- | spec/lib/gitlab/git/rev_list_spec.rb | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/spec/lib/gitlab/git/rev_list_spec.rb b/spec/lib/gitlab/git/rev_list_spec.rb index 444639acbaa..1f9c987be0b 100644 --- a/spec/lib/gitlab/git/rev_list_spec.rb +++ b/spec/lib/gitlab/git/rev_list_spec.rb @@ -26,6 +26,13 @@ describe Gitlab::Git::RevList, lib: true do expect(rev_list).not_to be_valid end + + it "ignores nil values" do + env = { var => nil } + rev_list = described_class.new('oldrev', 'newrev', project: project, env: env) + + expect(rev_list).to be_valid + end end end end |