remove support for PHP 7.0 and adopt PHP 7.1

VasekPurchart · VasekPurchart · commit 30b8d84f03e1 · 2018-12-31T11:33:32.000+01:00
diff --git a/.travis.yml b/.travis.yml
@@ -3,7 +3,6 @@ language: php
 sudo: false
 
 php:
-  - 7.0
   - 7.1
   - 7.2
   - nightly
diff --git a/build/cs-ruleset.xml b/build/cs-ruleset.xml
@@ -1,9 +1,6 @@
 <?xml version="1.0"?>
 <ruleset name="ConsistenceCodingStandard">
-	<rule ref="../vendor/consistence/coding-standard/Consistence/ruleset.xml">
-		<exclude name="SlevomatCodingStandard.Classes.ClassConstantVisibility"/><!-- requires 7.1+ -->
-		<exclude name="SlevomatCodingStandard.TypeHints.NullableTypeForNullDefaultValue"/><!-- requires PHP 7.1+ -->
-	</rule>
+	<rule ref="../vendor/consistence/coding-standard/Consistence/ruleset.xml"/>
 	<rule ref="SlevomatCodingStandard.Files.TypeNameMatchesFileName">
 		<properties>
 			<property name="rootNamespaces" type="array" value="
diff --git a/composer.json b/composer.json
@@ -11,7 +11,7 @@
 		}
 	],
 	"require": {
-		"php": "~7.0",
+		"php": "~7.1",
 		"phing/phing": "~2.16"
 	},
 	"require-dev": {
diff --git a/src/CopyFileElement.php b/src/CopyFileElement.php
@@ -16,46 +16,37 @@ class CopyFileElement
 	/** @var string|null */
 	private $existsMode;
 
-	/**
-	 * @return string|null
-	 */
-	public function getSource()
+	public function getSource(): ?string
 	{
 		return $this->source;
 	}
 
-	public function setSource(string $source)
+	public function setSource(string $source): void
 	{
 		$this->source = $source;
 	}
 
-	/**
-	 * @return string|null
-	 */
-	public function getTarget()
+	public function getTarget(): ?string
 	{
 		return $this->target;
 	}
 
-	public function setTarget(string $target)
+	public function setTarget(string $target): void
 	{
 		$this->target = $target;
 	}
 
-	/**
-	 * @return string|null
-	 */
-	public function getExistsMode()
+	public function getExistsMode(): ?string
 	{
 		return $this->existsMode;
 	}
 
-	public function setExistsMode(string $mode)
+	public function setExistsMode(string $mode): void
 	{
 		$this->existsMode = $mode;
 	}
 
-	public function validateAttributes()
+	public function validateAttributes(): void
 	{
 		if ($this->getSource() === null) {
 			throw new \BuildException('<file> must have the `source` attribute');
diff --git a/src/CopyFilesTask.php b/src/CopyFilesTask.php
@@ -9,22 +9,22 @@
 class CopyFilesTask extends \Task
 {
 
-	const EXISTS_MODE_FAIL = 'fail';
-	const EXISTS_MODE_REPLACE = 'replace';
-	const EXISTS_MODE_SKIP = 'skip';
+	public const EXISTS_MODE_FAIL = 'fail';
+	public const EXISTS_MODE_REPLACE = 'replace';
+	public const EXISTS_MODE_SKIP = 'skip';
 
 	/** @var \VasekPurchart\Phing\CopyFiles\CopyFileElement[] */
 	private $copyFiles = [];
 
 	/** @var string|null */
 	private $existsMode;
 
-	public function setExistsMode(string $mode)
+	public function setExistsMode(string $mode): void
 	{
 		$this->existsMode = $mode;
 	}
 
-	public function main()
+	public function main(): void
 	{
 		$this->validateAttributes();
 
@@ -74,7 +74,7 @@ public function createFile(): CopyFileElement
 		return $copyFile;
 	}
 
-	private function validateAttributes()
+	private function validateAttributes(): void
 	{
 		if (count($this->copyFiles) === 0) {
 			throw new \BuildException('At least one <file> element expected');
@@ -87,7 +87,7 @@ private function validateAttributes()
 		}
 	}
 
-	public static function checkAllowedExistsMode(string $mode)
+	public static function checkAllowedExistsMode(string $mode): void
 	{
 		$allowed = static::getAllowedExistsModes();
 		if (!in_array($mode, $allowed, true)) {
@@ -122,7 +122,7 @@ private function getCopyFileFileExistsMode(CopyFileElement $copyFile): string
 	/**
 	 * @param string[] $errors
 	 */
-	private function handleErrors(array $errors)
+	private function handleErrors(array $errors): void
 	{
 		if (count($errors) > 0) {
 			foreach ($errors as $error) {
diff --git a/tests/CopyFilesTaskIntegrationTest.php b/tests/CopyFilesTaskIntegrationTest.php
@@ -10,9 +10,9 @@
 class CopyFilesTaskIntegrationTest extends \PHPUnit\Framework\TestCase
 {
 
-	const TEMP_DIRECTORY_PATH = __DIR__ . '/temp';
+	private const TEMP_DIRECTORY_PATH = __DIR__ . '/temp';
 
-	public function testCopyFile()
+	public function testCopyFile(): void
 	{
 		$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/foo';
 		file_put_contents($sourceFilePath, 'FOO');
@@ -29,7 +29,7 @@ public function testCopyFile()
 		$tester->assertLogMessageRegExp('~Copying.+/foo.+->.+/foo-copy~', $target, Project::MSG_INFO);
 	}
 
-	public function testCopyFileWithAbsolutePath()
+	public function testCopyFileWithAbsolutePath(): void
 	{
 		$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/foo';
 		file_put_contents($sourceFilePath, 'FOO');
@@ -46,7 +46,7 @@ public function testCopyFileWithAbsolutePath()
 		$tester->assertLogMessageRegExp('~Copying.+/foo.+->.+/foo-copy~', $target, Project::MSG_INFO);
 	}
 
-	public function testTargetFileExists()
+	public function testTargetFileExists(): void
 	{
 		$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';
 		file_put_contents($sourceFilePath, 'NEW');
@@ -62,7 +62,7 @@ public function testTargetFileExists()
 		$tester->assertLogMessageRegExp('~/existing.+already exists.+SKIPPING~', $target, Project::MSG_INFO);
 	}
 
-	public function testTargetFileExistsSkip()
+	public function testTargetFileExistsSkip(): void
 	{
 		$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';
 		file_put_contents($sourceFilePath, 'NEW');
@@ -78,7 +78,7 @@ public function testTargetFileExistsSkip()
 		$tester->assertLogMessageRegExp('~/existing.+already exists.+SKIPPING~', $target, Project::MSG_INFO);
 	}
 
-	public function testTargetFileExistsReplace()
+	public function testTargetFileExistsReplace(): void
 	{
 		$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';
 		file_put_contents($sourceFilePath, 'NEW');
@@ -93,7 +93,7 @@ public function testTargetFileExistsReplace()
 		$tester->assertLogMessageRegExp('~/existing.+already exists~', $target, Project::MSG_VERBOSE);
 	}
 
-	public function testTargetFileExistsFail()
+	public function testTargetFileExistsFail(): void
 	{
 		$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';
 		file_put_contents($sourceFilePath, 'NEW');
@@ -110,7 +110,7 @@ public function testTargetFileExistsFail()
 		$tester->assertLogMessageRegExp('~/existing.+already exists~', $target, Project::MSG_ERR);
 	}
 
-	public function testCopyMultipleFiles()
+	public function testCopyMultipleFiles(): void
 	{
 		$sourceFooFilePath = self::TEMP_DIRECTORY_PATH . '/foo';
 		file_put_contents($sourceFooFilePath, 'FOO');
@@ -135,7 +135,7 @@ public function testCopyMultipleFiles()
 		$tester->assertLogMessageRegExp('~Copying.+/bar.+->.+/bar-copy~', $target, Project::MSG_INFO);
 	}
 
-	public function testReplaceMultipleFilesWithExistingTargets()
+	public function testReplaceMultipleFilesWithExistingTargets(): void
 	{
 		$sourceFooFilePath = self::TEMP_DIRECTORY_PATH . '/foo-new';
 		file_put_contents($sourceFooFilePath, 'FOO-NEW');
@@ -160,7 +160,7 @@ public function testReplaceMultipleFilesWithExistingTargets()
 		$tester->assertLogMessageRegExp('~/bar-existing.+already exists~', $target, Project::MSG_VERBOSE);
 	}
 
-	public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModes()
+	public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModes(): void
 	{
 		$sourceSkipFilePath = self::TEMP_DIRECTORY_PATH . '/skip-new';
 		file_put_contents($sourceSkipFilePath, 'SKIP-NEW');
@@ -192,7 +192,7 @@ public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModes()
 		$this->assertSame('DEFAULT-EXISTING', file_get_contents($targetDefaultFilePath));
 	}
 
-	public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModesWithReplaceFallback()
+	public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModesWithReplaceFallback(): void
 	{
 		$sourceSkipFilePath = self::TEMP_DIRECTORY_PATH . '/skip-new';
 		file_put_contents($sourceSkipFilePath, 'SKIP-NEW');
@@ -225,7 +225,7 @@ public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModesWithR
 		$tester->assertLogMessageRegExp('~/default-existing.+already exists~', $target, Project::MSG_VERBOSE);
 	}
 
-	public function testCopyNonExistentFile()
+	public function testCopyNonExistentFile(): void
 	{
 		$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');
 		$target = __FUNCTION__;
@@ -235,7 +235,7 @@ public function testCopyNonExistentFile()
 		$tester->assertLogMessageRegExp('~XXX.+does not exist~', $target, Project::MSG_ERR);
 	}
 
-	public function testCopyMultipleNonExistentFiles()
+	public function testCopyMultipleNonExistentFiles(): void
 	{
 		$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');
 		$target = __FUNCTION__;
@@ -246,7 +246,7 @@ public function testCopyMultipleNonExistentFiles()
 		$tester->assertLogMessageRegExp('~BAR.+does not exist~', $target, Project::MSG_ERR);
 	}
 
-	public function testCopyFileToNonExistingDirectory()
+	public function testCopyFileToNonExistingDirectory(): void
 	{
 		$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/foo';
 		file_put_contents($sourceFilePath, 'FOO');
@@ -263,7 +263,7 @@ public function testCopyFileToNonExistingDirectory()
 		$tester->assertLogMessageRegExp('~/foo.+cannot be copied.+/foo-copy~', $target, Project::MSG_ERR);
 	}
 
-	public function testMissingCopyFileElement()
+	public function testMissingCopyFileElement(): void
 	{
 		$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');
 		$target = __FUNCTION__;
@@ -274,7 +274,7 @@ public function testMissingCopyFileElement()
 		$tester->executeTarget($target);
 	}
 
-	public function testMissingSource()
+	public function testMissingSource(): void
 	{
 		$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');
 		$target = __FUNCTION__;
@@ -285,7 +285,7 @@ public function testMissingSource()
 		$tester->executeTarget($target);
 	}
 
-	public function testMissingTarget()
+	public function testMissingTarget(): void
 	{
 		$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');
 		$target = __FUNCTION__;
@@ -296,7 +296,7 @@ public function testMissingTarget()
 		$tester->executeTarget($target);
 	}
 
-	public function testInvalidFileExistsMode()
+	public function testInvalidFileExistsMode(): void
 	{
 		$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');
 		$target = __FUNCTION__;

-Original file line number
+Diff line change
 sudo: false
 php:
 -  - 7.0
   - 7.1
   - 7.2
   - nightly
Original file line number	Diff line number	Diff line change
`@@ -11,7 +11,7 @@`
`11`	`11`	`}`
`12`	`12`	`],`
`13`	`13`	`"require": {`
`14`		`- "php": "~7.0",`
	`14`	`+ "php": "~7.1",`
`15`	`15`	`"phing/phing": "~2.16"`
`16`	`16`	`},`
`17`	`17`	`"require-dev": {`
Original file line number	Diff line number	Diff line change
`@@ -16,46 +16,37 @@ class CopyFileElement`
`16`	`16`	`/** @var string\|null */`
`17`	`17`	`private $existsMode;`
`18`	`18`
`19`		`- /**`
`20`		`- * @return string\|null`
`21`		`- */`
`22`		`- public function getSource()`
	`19`	`+ public function getSource(): ?string`
`23`	`20`	`{`
`24`	`21`	`return $this->source;`
`25`	`22`	`}`
`26`	`23`
`27`		`- public function setSource(string $source)`
	`24`	`+ public function setSource(string $source): void`
`28`	`25`	`{`
`29`	`26`	`$this->source = $source;`
`30`	`27`	`}`
`31`	`28`
`32`		`- /**`
`33`		`- * @return string\|null`
`34`		`- */`
`35`		`- public function getTarget()`
	`29`	`+ public function getTarget(): ?string`
`36`	`30`	`{`
`37`	`31`	`return $this->target;`
`38`	`32`	`}`
`39`	`33`
`40`		`- public function setTarget(string $target)`
	`34`	`+ public function setTarget(string $target): void`
`41`	`35`	`{`
`42`	`36`	`$this->target = $target;`
`43`	`37`	`}`
`44`	`38`
`45`		`- /**`
`46`		`- * @return string\|null`
`47`		`- */`
`48`		`- public function getExistsMode()`
	`39`	`+ public function getExistsMode(): ?string`
`49`	`40`	`{`
`50`	`41`	`return $this->existsMode;`
`51`	`42`	`}`
`52`	`43`
`53`		`- public function setExistsMode(string $mode)`
	`44`	`+ public function setExistsMode(string $mode): void`
`54`	`45`	`{`
`55`	`46`	`$this->existsMode = $mode;`
`56`	`47`	`}`
`57`	`48`
`58`		`- public function validateAttributes()`
	`49`	`+ public function validateAttributes(): void`
`59`	`50`	`{`
`60`	`51`	`if ($this->getSource() === null) {`
`61`	`52`	throw new \BuildException('<file> must have the `source` attribute');
Original file line number	Diff line number	Diff line change
`@@ -9,22 +9,22 @@`
`9`	`9`	`class CopyFilesTask extends \Task`
`10`	`10`	`{`
`11`	`11`
`12`		`- const EXISTS_MODE_FAIL = 'fail';`
`13`		`- const EXISTS_MODE_REPLACE = 'replace';`
`14`		`- const EXISTS_MODE_SKIP = 'skip';`
	`12`	`+ public const EXISTS_MODE_FAIL = 'fail';`
	`13`	`+ public const EXISTS_MODE_REPLACE = 'replace';`
	`14`	`+ public const EXISTS_MODE_SKIP = 'skip';`
`15`	`15`
`16`	`16`	`/** @var \VasekPurchart\Phing\CopyFiles\CopyFileElement[] */`
`17`	`17`	`private $copyFiles = [];`
`18`	`18`
`19`	`19`	`/** @var string\|null */`
`20`	`20`	`private $existsMode;`
`21`	`21`
`22`		`- public function setExistsMode(string $mode)`
	`22`	`+ public function setExistsMode(string $mode): void`
`23`	`23`	`{`
`24`	`24`	`$this->existsMode = $mode;`
`25`	`25`	`}`
`26`	`26`
`27`		`- public function main()`
	`27`	`+ public function main(): void`
`28`	`28`	`{`
`29`	`29`	`$this->validateAttributes();`
`30`	`30`
`@@ -74,7 +74,7 @@ public function createFile(): CopyFileElement`
`74`	`74`	`return $copyFile;`
`75`	`75`	`}`
`76`	`76`
`77`		`- private function validateAttributes()`
	`77`	`+ private function validateAttributes(): void`
`78`	`78`	`{`
`79`	`79`	`if (count($this->copyFiles) === 0) {`
`80`	`80`	`throw new \BuildException('At least one <file> element expected');`
`@@ -87,7 +87,7 @@ private function validateAttributes()`
`87`	`87`	`}`
`88`	`88`	`}`
`89`	`89`
`90`		`- public static function checkAllowedExistsMode(string $mode)`
	`90`	`+ public static function checkAllowedExistsMode(string $mode): void`
`91`	`91`	`{`
`92`	`92`	`$allowed = static::getAllowedExistsModes();`
`93`	`93`	`if (!in_array($mode, $allowed, true)) {`
`@@ -122,7 +122,7 @@ private function getCopyFileFileExistsMode(CopyFileElement $copyFile): string`
`122`	`122`	`/**`
`123`	`123`	`* @param string[] $errors`
`124`	`124`	`*/`
`125`		`- private function handleErrors(array $errors)`
	`125`	`+ private function handleErrors(array $errors): void`
`126`	`126`	`{`
`127`	`127`	`if (count($errors) > 0) {`
`128`	`128`	`foreach ($errors as $error) {`
Original file line number	Diff line number	Diff line change
`@@ -10,9 +10,9 @@`
`10`	`10`	`class CopyFilesTaskIntegrationTest extends \PHPUnit\Framework\TestCase`
`11`	`11`	`{`
`12`	`12`
`13`		`- const TEMP_DIRECTORY_PATH = __DIR__ . '/temp';`
	`13`	`+ private const TEMP_DIRECTORY_PATH = __DIR__ . '/temp';`
`14`	`14`
`15`		`- public function testCopyFile()`
	`15`	`+ public function testCopyFile(): void`
`16`	`16`	`{`
`17`	`17`	`$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/foo';`
`18`	`18`	`file_put_contents($sourceFilePath, 'FOO');`
`@@ -29,7 +29,7 @@ public function testCopyFile()`
`29`	`29`	`$tester->assertLogMessageRegExp('~Copying.+/foo.+->.+/foo-copy~', $target, Project::MSG_INFO);`
`30`	`30`	`}`
`31`	`31`
`32`		`- public function testCopyFileWithAbsolutePath()`
	`32`	`+ public function testCopyFileWithAbsolutePath(): void`
`33`	`33`	`{`
`34`	`34`	`$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/foo';`
`35`	`35`	`file_put_contents($sourceFilePath, 'FOO');`
`@@ -46,7 +46,7 @@ public function testCopyFileWithAbsolutePath()`
`46`	`46`	`$tester->assertLogMessageRegExp('~Copying.+/foo.+->.+/foo-copy~', $target, Project::MSG_INFO);`
`47`	`47`	`}`
`48`	`48`
`49`		`- public function testTargetFileExists()`
	`49`	`+ public function testTargetFileExists(): void`
`50`	`50`	`{`
`51`	`51`	`$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';`
`52`	`52`	`file_put_contents($sourceFilePath, 'NEW');`
`@@ -62,7 +62,7 @@ public function testTargetFileExists()`
`62`	`62`	`$tester->assertLogMessageRegExp('~/existing.+already exists.+SKIPPING~', $target, Project::MSG_INFO);`
`63`	`63`	`}`
`64`	`64`
`65`		`- public function testTargetFileExistsSkip()`
	`65`	`+ public function testTargetFileExistsSkip(): void`
`66`	`66`	`{`
`67`	`67`	`$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';`
`68`	`68`	`file_put_contents($sourceFilePath, 'NEW');`
`@@ -78,7 +78,7 @@ public function testTargetFileExistsSkip()`
`78`	`78`	`$tester->assertLogMessageRegExp('~/existing.+already exists.+SKIPPING~', $target, Project::MSG_INFO);`
`79`	`79`	`}`
`80`	`80`
`81`		`- public function testTargetFileExistsReplace()`
	`81`	`+ public function testTargetFileExistsReplace(): void`
`82`	`82`	`{`
`83`	`83`	`$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';`
`84`	`84`	`file_put_contents($sourceFilePath, 'NEW');`
`@@ -93,7 +93,7 @@ public function testTargetFileExistsReplace()`
`93`	`93`	`$tester->assertLogMessageRegExp('~/existing.+already exists~', $target, Project::MSG_VERBOSE);`
`94`	`94`	`}`
`95`	`95`
`96`		`- public function testTargetFileExistsFail()`
	`96`	`+ public function testTargetFileExistsFail(): void`
`97`	`97`	`{`
`98`	`98`	`$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/new';`
`99`	`99`	`file_put_contents($sourceFilePath, 'NEW');`
`@@ -110,7 +110,7 @@ public function testTargetFileExistsFail()`
`110`	`110`	`$tester->assertLogMessageRegExp('~/existing.+already exists~', $target, Project::MSG_ERR);`
`111`	`111`	`}`
`112`	`112`
`113`		`- public function testCopyMultipleFiles()`
	`113`	`+ public function testCopyMultipleFiles(): void`
`114`	`114`	`{`
`115`	`115`	`$sourceFooFilePath = self::TEMP_DIRECTORY_PATH . '/foo';`
`116`	`116`	`file_put_contents($sourceFooFilePath, 'FOO');`
`@@ -135,7 +135,7 @@ public function testCopyMultipleFiles()`
`135`	`135`	`$tester->assertLogMessageRegExp('~Copying.+/bar.+->.+/bar-copy~', $target, Project::MSG_INFO);`
`136`	`136`	`}`
`137`	`137`
`138`		`- public function testReplaceMultipleFilesWithExistingTargets()`
	`138`	`+ public function testReplaceMultipleFilesWithExistingTargets(): void`
`139`	`139`	`{`
`140`	`140`	`$sourceFooFilePath = self::TEMP_DIRECTORY_PATH . '/foo-new';`
`141`	`141`	`file_put_contents($sourceFooFilePath, 'FOO-NEW');`
`@@ -160,7 +160,7 @@ public function testReplaceMultipleFilesWithExistingTargets()`
`160`	`160`	`$tester->assertLogMessageRegExp('~/bar-existing.+already exists~', $target, Project::MSG_VERBOSE);`
`161`	`161`	`}`
`162`	`162`
`163`		`- public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModes()`
	`163`	`+ public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModes(): void`
`164`	`164`	`{`
`165`	`165`	`$sourceSkipFilePath = self::TEMP_DIRECTORY_PATH . '/skip-new';`
`166`	`166`	`file_put_contents($sourceSkipFilePath, 'SKIP-NEW');`
`@@ -192,7 +192,7 @@ public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModes()`
`192`	`192`	`$this->assertSame('DEFAULT-EXISTING', file_get_contents($targetDefaultFilePath));`
`193`	`193`	`}`
`194`	`194`
`195`		`- public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModesWithReplaceFallback()`
	`195`	`+ public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModesWithReplaceFallback(): void`
`196`	`196`	`{`
`197`	`197`	`$sourceSkipFilePath = self::TEMP_DIRECTORY_PATH . '/skip-new';`
`198`	`198`	`file_put_contents($sourceSkipFilePath, 'SKIP-NEW');`
`@@ -225,7 +225,7 @@ public function testCopyMultipleFilesWithExistingTargetsUsingDifferentModesWithR`
`225`	`225`	`$tester->assertLogMessageRegExp('~/default-existing.+already exists~', $target, Project::MSG_VERBOSE);`
`226`	`226`	`}`
`227`	`227`
`228`		`- public function testCopyNonExistentFile()`
	`228`	`+ public function testCopyNonExistentFile(): void`
`229`	`229`	`{`
`230`	`230`	`$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');`
`231`	`231`	`$target = __FUNCTION__;`
`@@ -235,7 +235,7 @@ public function testCopyNonExistentFile()`
`235`	`235`	`$tester->assertLogMessageRegExp('~XXX.+does not exist~', $target, Project::MSG_ERR);`
`236`	`236`	`}`
`237`	`237`
`238`		`- public function testCopyMultipleNonExistentFiles()`
	`238`	`+ public function testCopyMultipleNonExistentFiles(): void`
`239`	`239`	`{`
`240`	`240`	`$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');`
`241`	`241`	`$target = __FUNCTION__;`
`@@ -246,7 +246,7 @@ public function testCopyMultipleNonExistentFiles()`
`246`	`246`	`$tester->assertLogMessageRegExp('~BAR.+does not exist~', $target, Project::MSG_ERR);`
`247`	`247`	`}`
`248`	`248`
`249`		`- public function testCopyFileToNonExistingDirectory()`
	`249`	`+ public function testCopyFileToNonExistingDirectory(): void`
`250`	`250`	`{`
`251`	`251`	`$sourceFilePath = self::TEMP_DIRECTORY_PATH . '/foo';`
`252`	`252`	`file_put_contents($sourceFilePath, 'FOO');`
`@@ -263,7 +263,7 @@ public function testCopyFileToNonExistingDirectory()`
`263`	`263`	`$tester->assertLogMessageRegExp('~/foo.+cannot be copied.+/foo-copy~', $target, Project::MSG_ERR);`
`264`	`264`	`}`
`265`	`265`
`266`		`- public function testMissingCopyFileElement()`
	`266`	`+ public function testMissingCopyFileElement(): void`
`267`	`267`	`{`
`268`	`268`	`$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');`
`269`	`269`	`$target = __FUNCTION__;`
`@@ -274,7 +274,7 @@ public function testMissingCopyFileElement()`
`274`	`274`	`$tester->executeTarget($target);`
`275`	`275`	`}`
`276`	`276`
`277`		`- public function testMissingSource()`
	`277`	`+ public function testMissingSource(): void`
`278`	`278`	`{`
`279`	`279`	`$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');`
`280`	`280`	`$target = __FUNCTION__;`
`@@ -285,7 +285,7 @@ public function testMissingSource()`
`285`	`285`	`$tester->executeTarget($target);`
`286`	`286`	`}`
`287`	`287`
`288`		`- public function testMissingTarget()`
	`288`	`+ public function testMissingTarget(): void`
`289`	`289`	`{`
`290`	`290`	`$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');`
`291`	`291`	`$target = __FUNCTION__;`
`@@ -296,7 +296,7 @@ public function testMissingTarget()`
`296`	`296`	`$tester->executeTarget($target);`
`297`	`297`	`}`
`298`	`298`
`299`		`- public function testInvalidFileExistsMode()`
	`299`	`+ public function testInvalidFileExistsMode(): void`
`300`	`300`	`{`
`301`	`301`	`$tester = new PhingTester(__DIR__ . '/copy-files-task-integration-test.xml');`
`302`	`302`	`$target = __FUNCTION__;`