Fixed #7621 - Add support for config_override.test.php

[Dillon-Brown]

Description

Motivation and Context

Issue reference: #7621

How To Test This

1. Check Travis
2. Add a tests/config.test.php file and then give it a custom value like so:

$testConfig = [
    'imap_test' => true,
];

3. Check that the custom config only runs during the test suite execution and overrides the config.php and config_override.php values.

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Final checklist

• My code follows the code style of this project found here.
• My change requires a change to the documentation.
• I have read the How to Contribute guidelines.

[codecov-io]

Codecov Report

❗ No coverage uploaded for pull request base (hotfix-7.10.x@38d2933). Click here to learn what that means.
The diff coverage is 0%.

@@               Coverage Diff                @@
##             hotfix-7.10.x    #7808   +/-   ##
================================================
  Coverage                 ?    7.28%           
================================================
  Files                    ?     3700           
  Lines                    ?   386196           
  Branches                 ?        0           
================================================
  Hits                     ?    28132           
  Misses                   ?   358064           
  Partials                 ?        0

[connorshea]

We should probably have a config.test.example.php or config.test.dist.php file in the repo for users to copy from. :)

Just this should be fine? Maybe a comment explaining what it's for.

<?php
$testConfig = [
    'imap_test' => true,
];

This'll be great, thank you! ❤️

[cameronblaikie]

Assessed 👍

[cameronblaikie]

Needs input from @Dillon-Brown before merging :D.

[cameronblaikie]

Assessed 👍

[connorshea]

👍 LGTM

[connorshea]

it occurs to me now, wouldn't this effect all the Robo tasks that use CliRunnerTrait?

[connorshea]

Also: when I run ./vendor/bin/robo tests:unit, at least on the StateChecker-removal branch, the config.php file is overwritten by the contents of the config.test.php.