fix: Avoid to generate a warning caused by skipping the setup (#17)

Author: theofidry

infection.json.dist

@@ -14,6 +14,10 @@
         "@default": true,
         "MBString": false,
 
+        "global-ignore": [
+            "Fidry\\FileSystem\\Test\\FileSystemTestCase::tearDown"
+        ],
+
         "FunctionCallRemoval": false,
         "MethodCallRemoval": false,
         "PublicVisibility": false,

phpunit.xml.dist

@@ -5,6 +5,11 @@
          bootstrap="vendor/autoload.php"
          failOnRisky="true"
          failOnWarning="true"
+         displayDetailsOnTestsThatTriggerDeprecations="true"
+         displayDetailsOnTestsThatTriggerNotices="true"
+         displayDetailsOnTestsThatTriggerWarnings="true"
+         displayDetailsOnTestsThatTriggerErrors="true"
+         executionOrder="duration"
          cacheResultFile="dist/.phpunit.result.cache">
     <php>
         <ini name="error_reporting" value="-1"/>

src/Test/FileSystemTestCase.php

@@ -85,8 +85,12 @@ protected function setUp(): void
 
     protected function tearDown(): void
     {
-        chdir($this->cwd);
-        FS::remove($this->tmp);
+        $wasSetupSkipped = '' === $this->cwd && '' === $this->tmp;
+
+        if (!$wasSetupSkipped) {
+            chdir($this->cwd);
+            FS::remove($this->tmp);
+        }
     }
 
     /**

tests/Test/FilesystemTestCaseTest.php

@@ -39,6 +39,7 @@
 use Fidry\FileSystem\FS;
 use Fidry\FileSystem\Test\FileSystemTestCase;
 use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\Attributes\Small;
 use Symfony\Component\Finder\Finder;
 use function getenv;
 use function is_string;
@@ -47,6 +48,7 @@
  * @internal
  */
 #[CoversClass(FileSystemTestCase::class)]
+#[Small]
 final class FilesystemTestCaseTest extends FileSystemTestCase
 {
     public static function getTmpDirNamespace(): string

tests/Test/SkippedFilesystemTest.php

@@ -0,0 +1,72 @@
+<?php
+
+/*
+ * This code is licensed under the BSD 3-Clause License.
+ *
+ * Copyright (c) 2022, Théo FIDRY <[email protected]>
+ * All rights reserved.
+ *
+ * Redistribution and use in source and binary forms, with or without
+ * modification, are permitted provided that the following conditions are met:
+ *
+ * * Redistributions of source code must retain the above copyright notice, this
+ *   list of conditions and the following disclaimer.
+ *
+ * * Redistributions in binary form must reproduce the above copyright notice,
+ *   this list of conditions and the following disclaimer in the documentation
+ *   and/or other materials provided with the distribution.
+ *
+ * * Neither the name of the copyright holder nor the names of its
+ *   contributors may be used to endorse or promote products derived from
+ *   this software without specific prior written permission.
+ *
+ * THIS SOFTWARE IS PROVIDED BY THE COPYRIGHT HOLDERS AND CONTRIBUTORS "AS IS"
+ * AND ANY EXPRESS OR IMPLIED WARRANTIES, INCLUDING, BUT NOT LIMITED TO, THE
+ * IMPLIED WARRANTIES OF MERCHANTABILITY AND FITNESS FOR A PARTICULAR PURPOSE ARE
+ * DISCLAIMED. IN NO EVENT SHALL THE COPYRIGHT HOLDER OR CONTRIBUTORS BE LIABLE
+ * FOR ANY DIRECT, INDIRECT, INCIDENTAL, SPECIAL, EXEMPLARY, OR CONSEQUENTIAL
+ * DAMAGES (INCLUDING, BUT NOT LIMITED TO, PROCUREMENT OF SUBSTITUTE GOODS OR
+ * SERVICES; LOSS OF USE, DATA, OR PROFITS; OR BUSINESS INTERRUPTION) HOWEVER
+ * CAUSED AND ON ANY THEORY OF LIABILITY, WHETHER IN CONTRACT, STRICT LIABILITY,
+ * OR TORT (INCLUDING NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE
+ * OF THIS SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
+ */
+
+declare(strict_types=1);
+
+namespace Fidry\FileSystem\Tests\Test;
+
+use Fidry\FileSystem\Test\FileSystemTestCase;
+use PHPUnit\Framework\Attributes\CoversNothing;
+use PHPUnit\Framework\Attributes\Medium;
+use function getenv;
+use function is_string;
+
+/**
+ * @internal
+ */
+#[CoversNothing]
+#[Medium]
+final class SkippedFilesystemTest extends FileSystemTestCase
+{
+    public static function getTmpDirNamespace(): string
+    {
+        $threadId = getenv('TEST_TOKEN');
+
+        if (!is_string($threadId)) {
+            $threadId = '';
+        }
+
+        return 'FidryFilesystem'.$threadId;
+    }
+
+    protected function setUp(): void
+    {
+        self::markTestSkipped();
+    }
+
+    public function test_empty_test(): void
+    {
+        $this->addToAssertionCount(1);
+    }
+}