Skip to content

Remove test environment from simulate saas consideration

Doug Stull requested to merge 349713-discussion-avoid-gitlab-dev_env_or_com3 into master

What does this MR do and why?

Currently during test, we'll be simulating SaaS and answering true to .com? when the env variable is set.

This could throw off local testing when we don't expect .com? to return true automatically when env variable is set in our terminal.

We should use :saas meta in those conditions instead, so there should be no use for test env consideration in this method.

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #349713 (closed)

Edited by Doug Stull

Merge request reports