diff --git a/src/StaticPHP/Config/ArtifactConfig.php b/src/StaticPHP/Config/ArtifactConfig.php index d25c6dd1..8244acd6 100644 --- a/src/StaticPHP/Config/ArtifactConfig.php +++ b/src/StaticPHP/Config/ArtifactConfig.php @@ -31,7 +31,7 @@ class ArtifactConfig */ public static function loadFromFile(string $file): void { - $content = file_get_contents($file); + $content = @file_get_contents($file); if ($content === false) { throw new WrongUsageException("Failed to read artifact config file: {$file}"); } diff --git a/src/StaticPHP/Config/PackageConfig.php b/src/StaticPHP/Config/PackageConfig.php index dc0b3d54..dda9761b 100644 --- a/src/StaticPHP/Config/PackageConfig.php +++ b/src/StaticPHP/Config/PackageConfig.php @@ -38,7 +38,7 @@ class PackageConfig */ public static function loadFromFile(string $file): void { - $content = file_get_contents($file); + $content = @file_get_contents($file); if ($content === false) { throw new WrongUsageException("Failed to read package config file: {$file}"); } diff --git a/tests/StaticPHP/Config/ArtifactConfigTest.php b/tests/StaticPHP/Config/ArtifactConfigTest.php index dc396488..28312dd7 100644 --- a/tests/StaticPHP/Config/ArtifactConfigTest.php +++ b/tests/StaticPHP/Config/ArtifactConfigTest.php @@ -26,7 +26,7 @@ class ArtifactConfigTest extends TestCase $reflection = new \ReflectionClass(ArtifactConfig::class); $property = $reflection->getProperty('artifact_configs'); $property->setAccessible(true); - $property->setValue([]); + $property->setValue(null, []); } /** @noinspection PhpExpressionResultUnusedInspection */ @@ -42,7 +42,7 @@ class ArtifactConfigTest extends TestCase $reflection = new \ReflectionClass(ArtifactConfig::class); $property = $reflection->getProperty('artifact_configs'); $property->setAccessible(true); - $property->setValue([]); + $property->setValue(null, []); } public function testLoadFromDirThrowsExceptionWhenDirectoryDoesNotExist(): void diff --git a/tests/StaticPHP/Config/PackageConfigTest.php b/tests/StaticPHP/Config/PackageConfigTest.php index 4072e39e..a705cfdb 100644 --- a/tests/StaticPHP/Config/PackageConfigTest.php +++ b/tests/StaticPHP/Config/PackageConfigTest.php @@ -26,7 +26,7 @@ class PackageConfigTest extends TestCase $reflection = new \ReflectionClass(PackageConfig::class); $property = $reflection->getProperty('package_configs'); $property->setAccessible(true); - $property->setValue([]); + $property->setValue(null, []); } protected function tearDown(): void @@ -41,7 +41,7 @@ class PackageConfigTest extends TestCase $reflection = new \ReflectionClass(PackageConfig::class); $property = $reflection->getProperty('package_configs'); $property->setAccessible(true); - $property->setValue([]); + $property->setValue(null, []); } public function testLoadFromDirThrowsExceptionWhenDirectoryDoesNotExist(): void diff --git a/tests/StaticPHP/DI/CallbackInvokerTest.php b/tests/StaticPHP/DI/CallbackInvokerTest.php index 751a70eb..0a757b63 100644 --- a/tests/StaticPHP/DI/CallbackInvokerTest.php +++ b/tests/StaticPHP/DI/CallbackInvokerTest.php @@ -7,15 +7,15 @@ namespace Tests\StaticPHP\DI; use DI\Container; use PHPUnit\Framework\TestCase; use StaticPHP\DI\CallbackInvoker; +use StaticPHP\Exception\SPCInternalException; /** * Helper class that requires constructor parameters for testing */ -class UnresolvableTestClass +readonly class UnresolvableTestClass { - public function __construct( - private string $requiredParam - ) {} + /** @noinspection PhpPropertyOnlyWrittenInspection */ + public function __construct(private string $requiredParam) {} } /** @@ -92,7 +92,7 @@ class CallbackInvokerTest extends TestCase // Should not resolve from container as 'test.service' is not a type // Will try default value or null - $this->expectException(\RuntimeException::class); + $this->expectException(SPCInternalException::class); $this->invoker->invoke($callback); } @@ -139,7 +139,7 @@ class CallbackInvokerTest extends TestCase return $required; }; - $this->expectException(\RuntimeException::class); + $this->expectException(SPCInternalException::class); $this->expectExceptionMessage("Cannot resolve parameter 'required' of type 'string'"); $this->invoker->invoke($callback); } @@ -527,7 +527,7 @@ class CallbackInvokerTest extends TestCase $callback = eval('return function (string|int $param) { return $param; };'); // Union types are not ReflectionNamedType, should not be resolved from container - $this->expectException(\RuntimeException::class); + $this->expectException(SPCInternalException::class); $this->invoker->invoke($callback); } @@ -594,7 +594,7 @@ class CallbackInvokerTest extends TestCase return $obj; }; - $this->expectException(\RuntimeException::class); + $this->expectException(SPCInternalException::class); $this->expectExceptionMessage("Cannot resolve parameter 'obj'"); $this->invoker->invoke($callback);