Skip to content

TestUtils\UtilityMethodTestCase: add new $phpcsVersion property#107

Merged
jrfnl merged 2 commits intodevelopfrom
testutils/utilitymethodtestcase-add-phpcsversion-property
Mar 18, 2020
Merged

TestUtils\UtilityMethodTestCase: add new $phpcsVersion property#107
jrfnl merged 2 commits intodevelopfrom
testutils/utilitymethodtestcase-add-phpcsversion-property

Conversation

@jrfnl
Copy link
Copy Markdown
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

Development

Successfully merging this pull request may close these issues.

1 participant