rector: update test return types

Author: ashleyhindle

tests/Feature/Console/InstallCommandWslTest.php

@@ -4,60 +4,55 @@
 
 use Laravel\Boost\Console\InstallCommand;
 
-test('isRunningInWsl returns true when WSL_DISTRO_NAME is set', function () {
+test('isRunningInWsl returns true when WSL_DISTRO_NAME is set', function (): void {
     putenv('WSL_DISTRO_NAME=Ubuntu');
 
-    $command = new InstallCommand();
+    $command = new InstallCommand;
     $reflection = new ReflectionClass($command);
     $method = $reflection->getMethod('isRunningInWsl');
-    $method->setAccessible(true);
 
     expect($method->invoke($command))->toBeTrue();
 });
 
-test('isRunningInWsl returns true when IS_WSL is set', function () {
+test('isRunningInWsl returns true when IS_WSL is set', function (): void {
     putenv('IS_WSL=1');
 
-    $command = new InstallCommand();
+    $command = new InstallCommand;
     $reflection = new ReflectionClass($command);
     $method = $reflection->getMethod('isRunningInWsl');
-    $method->setAccessible(true);
 
     expect($method->invoke($command))->toBeTrue();
 });
 
-test('isRunningInWsl returns true when both WSL env vars are set', function () {
+test('isRunningInWsl returns true when both WSL env vars are set', function (): void {
     putenv('WSL_DISTRO_NAME=Ubuntu');
     putenv('IS_WSL=true');
 
-    $command = new InstallCommand();
+    $command = new InstallCommand;
     $reflection = new ReflectionClass($command);
     $method = $reflection->getMethod('isRunningInWsl');
-    $method->setAccessible(true);
 
     expect($method->invoke($command))->toBeTrue();
 });
 
-test('isRunningInWsl returns false when no WSL env vars are set', function () {
+test('isRunningInWsl returns false when no WSL env vars are set', function (): void {
     putenv('WSL_DISTRO_NAME');
     putenv('IS_WSL');
 
-    $command = new InstallCommand();
+    $command = new InstallCommand;
     $reflection = new ReflectionClass($command);
     $method = $reflection->getMethod('isRunningInWsl');
-    $method->setAccessible(true);
 
     expect($method->invoke($command))->toBeFalse();
 });
 
-test('isRunningInWsl returns false when WSL env vars are empty strings', function () {
+test('isRunningInWsl returns false when WSL env vars are empty strings', function (): void {
     putenv('WSL_DISTRO_NAME=');
     putenv('IS_WSL=');
 
-    $command = new InstallCommand();
+    $command = new InstallCommand;
     $reflection = new ReflectionClass($command);
     $method = $reflection->getMethod('isRunningInWsl');
-    $method->setAccessible(true);
 
     expect($method->invoke($command))->toBeFalse();
 });

tests/Feature/Install/CodeEnvironment/CodeEnvironmentPathResolutionTest.php

@@ -38,7 +38,7 @@
     expect($cursor->getArtisanPath())->toBe('artisan');
 });
 
-test('CodeEnvironment returns absolute paths when forceAbsolutePath is true', function () {
+test('CodeEnvironment returns absolute paths when forceAbsolutePath is true', function (): void {
     $strategyFactory = Mockery::mock(DetectionStrategyFactory::class);
     $cursor = new Cursor($strategyFactory);
 
@@ -47,15 +47,15 @@
         ->and($cursor->getArtisanPath(true))->not()->toBe('artisan');
 });
 
-test('CodeEnvironment maintains relative paths when forceAbsolutePath is false', function () {
+test('CodeEnvironment maintains relative paths when forceAbsolutePath is false', function (): void {
     $strategyFactory = Mockery::mock(DetectionStrategyFactory::class);
     $cursor = new Cursor($strategyFactory);
 
     expect($cursor->getPhpPath(false))->toBe('php');
     expect($cursor->getArtisanPath(false))->toBe('artisan');
 });
 
-test('PhpStorm paths remain absolute regardless of forceAbsolutePath parameter', function () {
+test('PhpStorm paths remain absolute regardless of forceAbsolutePath parameter', function (): void {
     $strategyFactory = Mockery::mock(DetectionStrategyFactory::class);
     $phpStorm = new PhpStorm($strategyFactory);
 

tests/Unit/Install/CodeEnvironment/CodeEnvironmentTest.php

@@ -410,12 +410,12 @@ public function mcpConfigPath(): string
         ->and($capturedContent)->toBe(fixture('mcp-expected.json5'));
 });
 
-test('getPhpPath uses absolute paths when forceAbsolutePath is true', function () {
+test('getPhpPath uses absolute paths when forceAbsolutePath is true', function (): void {
     $environment = new TestCodeEnvironment($this->strategyFactory);
     expect($environment->getPhpPath(true))->toBe(PHP_BINARY);
 });
 
-test('getPhpPath maintains default behavior when forceAbsolutePath is false', function () {
+test('getPhpPath maintains default behavior when forceAbsolutePath is false', function (): void {
     $environment = new TestCodeEnvironment($this->strategyFactory);
     expect($environment->getPhpPath(false))->toBe('php');
 });