Remove test environment from simulate saas consideration
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.
-
I have evaluated the MR acceptance checklist for this MR.
Related to #349713 (closed)
Edited by Doug Stull