Skip to content

Mock skip_pull in specs to prevent failures when ENV is set

George Koltsov requested to merge jmd-handle-qa-skip-pull-spec into master

What does this MR do and why?

Some specs fail if a user has QA_SKIP_PULL set on their own machine.
We should avoid having our specs depend on the environment variables set (or not set) on a specific machine, and instead configure the tests to have the expected state.

How to set up and validate locally

# both of these should pass with this change, confirming that external environmental state doesn't impact tests
QA_SKIP_PULL=true bundle exec rspec
QA_SKIP_PULL=false bundle exec rspec

MR acceptance checklist

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

Edited by George Koltsov

Merge request reports

Loading