Skip to content

Fixes `TEST` value on gitlab-ci.yml

Mayra Cabrera requested to merge fixes-test-variable-on-gitlab-ci-yml into master

What does this MR do?

On dry_run?, we explicitly check for TEST value to be false, so using an empty string for this variable makes dry_run? to return true, to fix that this merge request sets TEST to false

❯ export TEST=''
❯ irb
2.7.2 :001 > !ENV.fetch('TEST', 'false').casecmp?('false')
 => true


❯ export TEST='false'
❯ irb
2.7.2 :001 > !ENV.fetch('TEST', 'false').casecmp?('false')
 => false

Related to gitlab-com/gl-infra/delivery#1578 (closed)

Merge request reports