Signed-off-by: Rémy Coutable <remy@rymai.me>

Moving this into the Change means a new keyword argument isn't required
when new merge request attributes are supported.

Move RuboCop autocorrect call into a helper method so it's easier to
reuse.

Pass existing logger instance to allow keeps to log.

This adds the test coverage for the feature added in the earlier commit.

Since migrations for EE models don't have factories in CE we need to
skip this cop in foss-only CI.

This commit also simplifies the implementation and improves the
performances be caching files operations.

If an association defines a scope with arity != 0
then the scope is not evaluated when preloading
that relation. This fixes it.

Changelog: fixed

This is to make it easier to support origin remote in the following
commit. Also we're trying to move logic out of Runner as it's become a
bit unmanageable.

In real usage if you call `ENV.fetch('NON_EXISTENT_KEY')` it will raise
a `KeyError`. But our previously implementation of of `stub_env` with
`nil` is causing it to return `nil` rather than raising an error. This
can cause us to write tests that don't match the actual production
behaviour. We might forget to rescue the exception in a certain place or
we also may not be able to write a test case for the expected behaviour.

```
irb(main):001> ENV.fetch('NON_EXISTENT_KEY')
(irb):1:in `fetch': key not found: "NON_EXISTENT_KEY" (KeyError)

irb(main):002> ENV['NON_EXISTENT_KEY'] = nil
=> nil

irb(main):003> ENV.fetch('NON_EXISTENT_KEY')
(irb):3:in `fetch': key not found: "NON_EXISTENT_KEY" (KeyError)
```

- add bad format and enforce pattern.