-
-
Notifications
You must be signed in to change notification settings - Fork 3
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Simplify the resetting of state properties in each trait
Instead of relying on `@before` fixtures to reset the [private] property that is tracking what's changed in each trait, set the default value in the property definition.
- Loading branch information
1 parent
582f8db
commit a14f08b
Showing
4 changed files
with
142 additions
and
54 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,118 @@ | ||
| <?php | ||
|
|
||
| namespace Tests; | ||
|
|
||
| use AssertWell\PHPUnitGlobalState\Exceptions\RedefineException; | ||
| use PHPUnit\Framework\SkippedTestError; | ||
|
|
||
| /** | ||
| * Tests to ensure that state may be set in PHPUnit fixtures. | ||
| * | ||
| * @ticket https://github.com/assertwell/phpunit-global-state/issues/14 | ||
| */ | ||
| class FixtureTest extends TestCase | ||
| { | ||
| protected $backupGlobalsBlacklist = [ | ||
| 'FIXTURE_BEFORE_GLOBAL', | ||
| 'FIXTURE_SETUP_GLOBAL', | ||
| ]; | ||
|
|
||
| public function setUp(): void | ||
| { | ||
| parent::setUp(); | ||
|
|
||
| $this->setConstant('FIXTURE_SETUP_CONSTANT', true); | ||
| $this->setEnvironmentVariable('FIXTURE_SETUP_ENV', 'abc'); | ||
| $this->setGlobalVariable('FIXTURE_SETUP_GLOBAL', true); | ||
| } | ||
|
|
||
| /** | ||
| * @before | ||
| */ | ||
| protected function defineInitialValues() | ||
| { | ||
| $this->setConstant('FIXTURE_BEFORE_CONSTANT', true); | ||
| $this->setEnvironmentVariable('FIXTURE_BEFORE_ENV', 'xyz'); | ||
| $this->setGlobalVariable('FIXTURE_BEFORE_GLOBAL', true); | ||
| } | ||
|
|
||
| /** | ||
| * @test | ||
| * @group Constants | ||
| */ | ||
| public function it_should_permit_constants_to_be_set_in_fixtures_method() | ||
| { | ||
| $this->assertTrue( | ||
| defined('FIXTURE_SETUP_CONSTANT'), | ||
| 'The constant should have been defined in the setUp() method.' | ||
| ); | ||
| $this->assertTrue( | ||
| defined('FIXTURE_BEFORE_CONSTANT'), | ||
| 'The constant should have been defined in the @before method.' | ||
| ); | ||
|
|
||
| $this->restoreConstants(); | ||
| $this->assertFalse( | ||
| defined('FIXTURE_SETUP_CONSTANT'), | ||
| 'The constant should have been undefined by restoreConstants().' | ||
| ); | ||
| $this->assertFalse( | ||
| defined('FIXTURE_BEFORE_CONSTANT'), | ||
| 'The constant should have been undefined by restoreConstants().' | ||
| ); | ||
| } | ||
|
|
||
| /** | ||
| * @test | ||
| * @group EnvironmentVariables | ||
| */ | ||
| public function it_should_permit_environment_variables_to_be_set_in_fixtures_method() | ||
| { | ||
| $this->assertSame( | ||
| 'abc', | ||
| getenv('FIXTURE_SETUP_ENV'), | ||
| 'The environment variable should have been defined in the setUp() method.' | ||
| ); | ||
| $this->assertSame( | ||
| 'xyz', | ||
| getenv('FIXTURE_BEFORE_ENV'), | ||
| 'The environment variable should have been defined in the @before method.' | ||
| ); | ||
|
|
||
| $this->restoreEnvironmentVariables(); | ||
| $this->assertFalse( | ||
| getenv('FIXTURE_SETUP_ENV'), | ||
| 'The environment variable should have been undefined by restoreGlobalVariables().' | ||
| ); | ||
| $this->assertFalse( | ||
| getenv('FIXTURE_BEFORE_ENV'), | ||
| 'The environment variable should have been undefined by restoreGlobalVariables().' | ||
| ); | ||
| } | ||
|
|
||
| /** | ||
| * @test | ||
| * @group GlobalVariables | ||
| */ | ||
| public function it_should_permit_global_variables_to_be_set_in_fixtures_method() | ||
| { | ||
| $this->assertTrue( | ||
| isset($GLOBALS['FIXTURE_SETUP_GLOBAL']), | ||
| 'The global variable should have been defined in the setUp() method.' | ||
| ); | ||
| $this->assertTrue( | ||
| isset($GLOBALS['FIXTURE_BEFORE_GLOBAL']), | ||
| 'The global variable should have been defined in the @before method.' | ||
| ); | ||
|
|
||
| $this->restoreGlobalVariables(); | ||
| $this->assertFalse( | ||
| isset($GLOBALS['FIXTURE_SETUP_GLOBAL']), | ||
| 'The global variable should have been undefined by restoreGlobalVariables().' | ||
| ); | ||
| $this->assertFalse( | ||
| isset($GLOBALS['FIXTURE_BEFORE_GLOBAL']), | ||
| 'The global variable should have been undefined by restoreGlobalVariables().' | ||
| ); | ||
| } | ||
| } |