Init tester object in setUp

Author: tfirdaus

tests/phpunit/Commands/IncrementCommandTest.php

@@ -13,64 +13,75 @@
 class IncrementCommandTest extends TestCase
 {
 	private Commander $commander;
+	private CommandTester $tester;
 
 	public function setUp(): void
 	{
 		parent::setUp();
 
 		$this->commander = new Commander();
+		$this->tester = new CommandTester($this->commander->get('increment'));
 	}
 
 	/** @dataProvider dataInvalidVersionArgument */
 	public function testInvalidVersionArgument(string $version): void
 	{
-		$tester = new CommandTester($this->commander->get('increment'));
-		$tester->execute(['version' => $version]);
+		$this->tester->execute(['version' => $version]);
 
 		self::assertStringContainsString(
 			sprintf("[ERROR] Version string '%s' is not valid and cannot be parsed", $version),
-			$tester->getDisplay(),
+			$this->tester->getDisplay(),
 		);
 	}
 
 	public function testIncrementPatch(): void
 	{
-		$tester = new CommandTester($this->commander->get('increment'));
-		$tester->execute(['version' => '1.0.0']);
+		$this->tester->execute(['version' => '1.0.0']);
 
-		self::assertStringContainsString('1.0.1', $tester->getDisplay());
+		self::assertStringContainsString(
+			'1.0.1',
+			$this->tester->getDisplay(),
+		);
 	}
 
 	public function testIncrementMinor(): void
 	{
-		$tester = new CommandTester($this->commander->get('increment'));
-		$tester->execute(['version' => '1.0.0', '--part' => 'minor']);
+		$this->tester->execute(['version' => '1.0.0', '--part' => 'minor']);
 
-		self::assertStringContainsString('1.1.0', $tester->getDisplay());
+		self::assertStringContainsString(
+			'1.1.0',
+			$this->tester->getDisplay(),
+		);
 	}
 
 	public function testIncrementMajor(): void
 	{
-		$tester = new CommandTester($this->commander->get('increment'));
-		$tester->execute(['version' => '1.0.0', '--part' => 'major']);
+		$this->tester->execute(['version' => '1.0.0', '--part' => 'major']);
 
-		self::assertStringContainsString('2.0.0', $tester->getDisplay());
+		self::assertStringContainsString(
+			'2.0.0',
+			$this->tester->getDisplay(),
+		);
 	}
 
 	public function testIncrementWithBuildMetadata(): void
 	{
-		$tester = new CommandTester($this->commander->get('increment'));
-		$tester->execute(['version' => '1.0.0', '--build' => '123']);
+		$this->tester->execute(['version' => '1.0.0', '--build' => '123']);
 
-		self::assertStringContainsString('1.0.1+123', $tester->getDisplay());
+		self::assertStringContainsString(
+			'1.0.1+123',
+			$this->tester->getDisplay(),
+		);
 	}
 
 	public function testIncrementWithPreRelease(): void
 	{
-		$tester = new CommandTester($this->commander->get('increment'));
-		$tester->execute(['version' => '1.0.0', '--pre' => 'beta']);
+		$this->tester->execute(['version' => '1.0.0', '--pre' => 'beta']);
 
-		self::assertStringContainsString('1.0.1-beta', $tester->getDisplay());
+		self::assertStringContainsString(
+			'1.0.1-beta',
+			$this->tester->getDisplay(),
+		);
 	}
 
 	public static function dataInvalidVersionArgument(): iterable

tests/phpunit/Commands/ValidateCommandTest.php

@@ -13,35 +13,35 @@
 class ValidateCommandTest extends TestCase
 {
 	private Commander $commander;
+	private CommandTester $tester;
 
 	public function setUp(): void
 	{
 		parent::setUp();
 
 		$this->commander = new Commander();
+		$this->tester = new CommandTester($this->commander->get('validate'));
 	}
 
 	/** @dataProvider dataInvalidVersionArgument */
 	public function testInvalidVersionArgument(string $version): void
 	{
-		$tester = new CommandTester($this->commander->get('validate'));
-		$tester->execute(['version' => $version]);
+		$this->tester->execute(['version' => $version]);
 
 		self::assertStringContainsString(
 			sprintf("[ERROR] Version string '%s' is not valid and cannot be parsed", $version),
-			$tester->getDisplay(),
+			$this->tester->getDisplay(),
 		);
 	}
 
 	/** @dataProvider dataValidVersionArgument */
 	public function testValidVersionArgument(string $version): void
 	{
-		$tester = new CommandTester($this->commander->get('validate'));
-		$tester->execute(['version' => $version]);
+		$this->tester->execute(['version' => $version]);
 
 		self::assertStringContainsString(
 			sprintf("[OK] Version string '%s' is valid and can be parsed", $version),
-			$tester->getDisplay(),
+			$this->tester->getDisplay(),
 		);
 	}