Error last argument is considered chainable

consistence · Sep 22, 2017 · 9bc8789 · 9bc8789
1 parent 5b2879d
commit 9bc8789
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 0 deletions.
diff --git a/Consistence/Sniffs/Exceptions/ExceptionDeclarationSniff.php b/Consistence/Sniffs/Exceptions/ExceptionDeclarationSniff.php
@@ -116,6 +116,7 @@ private function checkThatExceptionIsChainable(PhpCsFile $phpcsFile, int $classP
 		if (
 			$lastArgument->getTypeHint() !== '\Throwable'
 			&& !StringHelper::endsWith($lastArgument->getTypeHint(), 'Exception')
+			&& !StringHelper::endsWith($lastArgument->getTypeHint(), 'Error')
 		) {
 			$phpcsFile->addError(sprintf(
 				'Exception is not chainable. It must have optional \Throwable as last constructor argument and has "%s".',

diff --git a/tests/Sniffs/Exceptions/ExceptionDeclarationSniffTest.php b/tests/Sniffs/Exceptions/ExceptionDeclarationSniffTest.php
@@ -153,6 +153,15 @@ public function testExceptionWithCustomExceptionArgumentIsChainable()
 		$this->assertNoSniffError($resultFile, 10);
 	}
 
+	public function testExceptionWithErrorArgumentIsChainable()
+	{
+		$resultFile = $this->checkFile(__DIR__ . '/data/ErrorArgumentChainableConstructorException.php', [
+			'exceptionsDirectoryName' => 'data',
+		]);
+
+		$this->assertNoSniffError($resultFile, 10);
+	}
+
 	public function testExceptionWithNonchainableConstructorIsNotChainable()
 	{
 		$resultFile = $this->checkFile(__DIR__ . '/data/NonChainableConstructorException.php', [