fix: streamline temporary file path generation in tests

Author: jackd248

tests/src/Generators/DocBlockHeaderTest.php

@@ -307,7 +307,7 @@ public function testToArrayWithAddStructureName(): void
 
     public function testFromComposerWithSingleAuthor(): void
     {
-        $testComposerPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $testComposerPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
         $composerData = [
             'name' => 'test/package',
             'authors' => [
@@ -332,7 +332,7 @@ public function testFromComposerWithSingleAuthor(): void
 
     public function testFromComposerWithMultipleAuthors(): void
     {
-        $testComposerPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $testComposerPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
         $composerData = [
             'name' => 'test/package',
             'authors' => [
@@ -359,7 +359,7 @@ public function testFromComposerWithMultipleAuthors(): void
 
     public function testFromComposerWithAuthorWithoutEmail(): void
     {
-        $testComposerPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $testComposerPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
         $composerData = [
             'name' => 'test/package',
             'authors' => [
@@ -382,7 +382,7 @@ public function testFromComposerWithAuthorWithoutEmail(): void
 
     public function testFromComposerWithNoAuthors(): void
     {
-        $testComposerPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $testComposerPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
         $composerData = [
             'name' => 'test/package',
             'license' => 'MIT',
@@ -402,7 +402,7 @@ public function testFromComposerWithNoAuthors(): void
 
     public function testFromComposerWithNoLicense(): void
     {
-        $testComposerPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $testComposerPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
         $composerData = [
             'name' => 'test/package',
             'authors' => [
@@ -424,7 +424,7 @@ public function testFromComposerWithNoLicense(): void
 
     public function testFromComposerWithAdditionalAnnotations(): void
     {
-        $testComposerPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $testComposerPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
         $composerData = [
             'name' => 'test/package',
             'authors' => [
@@ -452,7 +452,7 @@ public function testFromComposerWithAdditionalAnnotations(): void
 
     public function testFromComposerWithCustomParameters(): void
     {
-        $testComposerPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $testComposerPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
         $composerData = [
             'name' => 'test/package',
             'authors' => [

tests/src/Rules/DocBlockHeaderFixerTest.php

@@ -952,6 +952,7 @@ public function testIsAnonymousClassWithAttribute(): void
         self::assertTrue($result);
     }
 
+    #[\PHPUnit\Framework\Attributes\RequiresPhp('>= 8.2')]
     public function testIsAnonymousClassWithReadonlyModifier(): void
     {
         $code = '<?php $obj = new readonly class {};';
@@ -974,6 +975,7 @@ public function testIsAnonymousClassWithReadonlyModifier(): void
         self::assertTrue($result);
     }
 
+    #[\PHPUnit\Framework\Attributes\RequiresPhp('>= 8.2')]
     public function testSkipsAnonymousClassWithReadonlyModifier(): void
     {
         $code = '<?php $obj = new readonly class {};';

tests/src/Service/ComposerServiceTest.php

@@ -33,7 +33,7 @@ final class ComposerServiceTest extends TestCase
 
     protected function setUp(): void
     {
-        $this->testComposerJsonPath = sys_get_temp_dir() . '/test-composer-' . uniqid() . '.json';
+        $this->testComposerJsonPath = sys_get_temp_dir().'/test-composer-'.uniqid().'.json';
     }
 
     protected function tearDown(): void
@@ -71,7 +71,7 @@ public function testReadComposerJsonThrowsExceptionWhenFileDoesNotExist(): void
     public function testReadComposerJsonThrowsExceptionWhenFileCannotBeRead(): void
     {
         // Create an empty directory to simulate unreadable file
-        $dirPath = sys_get_temp_dir() . '/test-dir-' . uniqid();
+        $dirPath = sys_get_temp_dir().'/test-dir-'.uniqid();
         mkdir($dirPath);
 
         $this->expectException(RuntimeException::class);
@@ -88,7 +88,7 @@ public function testReadComposerJsonThrowsExceptionWhenFileCannotBeRead(): void
     public function testReadComposerJsonThrowsExceptionOnReadFailure(): void
     {
         // Create a file with no read permissions to force file_get_contents to return false
-        $invalidPath = sys_get_temp_dir() . '/test-unreadable-' . uniqid() . '.json';
+        $invalidPath = sys_get_temp_dir().'/test-unreadable-'.uniqid().'.json';
         file_put_contents($invalidPath, '{}');
 
         // Make file unreadable (this may not work on all systems, especially Windows)
@@ -171,9 +171,12 @@ public function testExtractAuthorsWithValidAuthors(): void
 
         self::assertCount(2, $result);
         self::assertSame('John Doe', $result[0]['name']);
+        /* @phpstan-ignore-next-line offsetAccess.notFound */
         self::assertSame('john@example.com', $result[0]['email']);
+        /* @phpstan-ignore-next-line offsetAccess.notFound */
         self::assertSame('Developer', $result[0]['role']);
         self::assertSame('Jane Smith', $result[1]['name']);
+        /* @phpstan-ignore-next-line offsetAccess.notFound */
         self::assertSame('jane@example.com', $result[1]['email']);
     }
 
@@ -241,6 +244,7 @@ public function testGetPrimaryAuthor(): void
 
         self::assertNotNull($result);
         self::assertSame('John Doe', $result['name']);
+        /* @phpstan-ignore-next-line offsetAccess.notFound */
         self::assertSame('john@example.com', $result['email']);
     }