Skip to content

Conversation

@jrfnl
Copy link
Member

@jrfnl jrfnl commented Mar 18, 2020

TestUtils\UtilityMethodTestCase: add new $phpcsVersion property

Tests will often enough need access to the PHPCS version on which the tests are being run.

This adds a $phpcsVersion property to the PHPCSUtils\TestUtils\UtilityMethodTestCase class to make that version number easily and always available.

Note: if the PHPCS version is needed within a data provider method for a test, Helper::getVersion() still needs to be used as the data providers are run before the setUpBeforeClass() type methods.

Tests: implement use of the UtilityMethodTestCase::$phpcsVersion property

... in all relevant places in the existing tests.

jrfnl added 2 commits March 18, 2020 10:56
Tests will often enough need access to the PHPCS version on which the tests are being run. This adds a `$phpcsVersion` property to the `PHPCSUtils\TestUtils\UtilityMethodTestCase` class to make that version number easily and always available. **Note**: if the PHPCS version is needed within a data provider method for a test, `Helper::getVersion()` still needs to be used as the data providers are run before the `setUpBeforeClass()` type methods.
…erty ... in all relevant places in the existing tests.
@jrfnl jrfnl force-pushed the testutils/utilitymethodtestcase-add-phpcsversion-property branch from 87d56ce to 788a52b Compare March 18, 2020 10:00
@jrfnl jrfnl merged commit ed41b59 into develop Mar 18, 2020
@jrfnl jrfnl deleted the testutils/utilitymethodtestcase-add-phpcsversion-property branch March 18, 2020 10:20
@jrfnl jrfnl modified the milestones: 1.0.0, 1.0.0-alpha3 May 1, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment