test: Add inspections for null pointer and void function usage

guanguans · guanguans · commit 1aab42a16e13 · 2025-03-21T18:46:30.000+08:00
- Introduce new static analysis inspections for null pointers and void function results in multiple test files.
- Update existing annotations to improve code quality and catch potential issues early.
- Ensure all relevant tests are annotated consistently.
diff --git a/_ide_helper.php b/_ide_helper.php
@@ -1,7 +1,8 @@
 <?php
 
-/** @noinspection EmptyClassInspection */
 /** @noinspection ClassNameCollisionInspection */
+/** @noinspection EmptyClassInspection */
+/** @noinspection PhpMultipleClassDeclarationsInspection */
 
 declare(strict_types=1);
 
diff --git a/monorepo-builder.php b/monorepo-builder.php
@@ -1,5 +1,7 @@
 <?php
 
+/** @noinspection PhpUnusedAliasInspection */
+
 declare(strict_types=1);
 
 /**
diff --git a/rector.php b/rector.php
@@ -2,6 +2,8 @@
 
 /** @noinspection PhpInternalEntityUsedInspection */
 /** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpUnusedAliasInspection */
+
 declare(strict_types=1);
 
 /**
diff --git a/tests/Channels/AbstractChannelTest.php b/tests/Channels/AbstractChannelTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Channels/ChannelTest.php b/tests/Channels/ChannelTest.php
@@ -1,8 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
-/** @noinspection DebugFunctionUsageInspection */
-/** @noinspection ForgottenDebugOutputInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Channels/DumpChannelTest.php b/tests/Channels/DumpChannelTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Channels/LogChannelTest.php b/tests/Channels/LogChannelTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Channels/MailChannelTest.php b/tests/Channels/MailChannelTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Channels/NotifyChannelTest.php b/tests/Channels/NotifyChannelTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Channels/StackChannelTest.php b/tests/Channels/StackChannelTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Collectors/CollectorTest.php b/tests/Collectors/CollectorTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Commands/TestCommandTest.php b/tests/Commands/TestCommandTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/ExceptionNotifyManagerTest.php b/tests/ExceptionNotifyManagerTest.php
@@ -2,6 +2,8 @@
 
 /** @noinspection AnonymousFunctionStaticInspection */
 /** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
 /** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
diff --git a/tests/ExceptionNotifyServiceProviderTest.php b/tests/ExceptionNotifyServiceProviderTest.php
@@ -1,6 +1,10 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/FeatureTest.php b/tests/FeatureTest.php
@@ -2,6 +2,8 @@
 
 /** @noinspection AnonymousFunctionStaticInspection */
 /** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
 /** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
diff --git a/tests/Mail/ReportExceptionMailTest.php b/tests/Mail/ReportExceptionMailTest.php
@@ -1,5 +1,12 @@
 <?php
 
+/** @noinspection AnonymousFunctionStaticInspection */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
+/** @noinspection StaticClosureCanBeUsedInspection */
+
 declare(strict_types=1);
 
 /**
diff --git a/tests/Pest.php b/tests/Pest.php
@@ -1,11 +1,14 @@
 <?php
 
 /** @noinspection AnonymousFunctionStaticInspection */
-/** @noinspection PhpInconsistentReturnPointsInspection */
-/** @noinspection PhpUndefinedClassInspection */
-/** @noinspection PhpUnused */
+/** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
+/** @noinspection PhpInconsistentReturnPointsInspection */
+
 declare(strict_types=1);
 
 /**
@@ -17,9 +20,11 @@
  * @see https://github.com/guanguans/laravel-exception-notify
  */
 
+use Composer\Autoload\ClassLoader;
+use Faker\Factory;
 use Guanguans\LaravelExceptionNotifyTests\TestCase;
 use Illuminate\Support\Collection;
-use Illuminate\Support\Facades\File;
+use Pest\Expectation;
 
 uses(TestCase::class)
     ->beforeAll(function (): void {})
diff --git a/tests/Support/ExceptionContextTest.php b/tests/Support/ExceptionContextTest.php
@@ -2,6 +2,9 @@
 
 /** @noinspection AnonymousFunctionStaticInspection */
 /** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
+/** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Support/HeplersTest.php b/tests/Support/HeplersTest.php
@@ -4,6 +4,7 @@
 /** @noinspection NullPointerExceptionInspection */
 /** @noinspection PhpInternalEntityUsedInspection */
 /** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/Support/JsonFixerTest.php b/tests/Support/JsonFixerTest.php
@@ -2,7 +2,9 @@
 
 /** @noinspection AnonymousFunctionStaticInspection */
 /** @noinspection NullPointerExceptionInspection */
+/** @noinspection PhpInternalEntityUsedInspection */
 /** @noinspection PhpUnhandledExceptionInspection */
+/** @noinspection PhpVoidFunctionResultUsedInspection */
 /** @noinspection StaticClosureCanBeUsedInspection */
 
 declare(strict_types=1);
diff --git a/tests/TestCase.php b/tests/TestCase.php
@@ -1,5 +1,6 @@
 <?php
 
+/** @noinspection EfferentObjectCouplingInspection */
 /** @noinspection PhpMissingParentCallCommonInspection */
 
 declare(strict_types=1);

-Original file line number
+Diff line change
@@ @@ -1,5 +1,7 @@ @@
 <?php
 +/** @noinspection PhpUnusedAliasInspection */
++
 declare(strict_types=1);
 /**