Simplify ECS skip option

composer.json

@@ -28,7 +28,7 @@
         "orchestra/testbench": "^4.0 || ^5.0 || ^6.0",
         "phpstan/phpstan-mockery": "^1.0",
         "phpunit/phpunit": "^8.0,<8.5.12 || ^9.3.3",
-        "zing/coding-standard": "^5.0"
+        "zing/coding-standard": "^5.1"
     },
     "autoload": {
         "psr-4": {

ecs.php

@@ -2,9 +2,6 @@
 
 declare(strict_types=1);
 
-use PhpCsFixer\Fixer\ControlStructure\YodaStyleFixer;
-use PhpCsFixer\Fixer\PhpUnit\PhpUnitInternalClassFixer;
-use PhpCsFixer\Fixer\PhpUnit\PhpUnitTestClassRequiresCoversFixer;
 use Symfony\Component\DependencyInjection\Loader\Configurator\ContainerConfigurator;
 use Symplify\EasyCodingStandard\ValueObject\Option;
 use Zing\CodingStandard\Set\ECSSetList;
@@ -16,11 +13,6 @@
 
     $parameters = $containerConfigurator->parameters();
     $parameters->set(Option::PARALLEL, true);
-    $parameters->set(Option::SKIP, [
-        YodaStyleFixer::class => null,
-        PhpUnitInternalClassFixer::class,
-        PhpUnitTestClassRequiresCoversFixer::class,
-    ]);
     $parameters->set(
         Option::PATHS,
         [__DIR__ . '/config', __DIR__ . '/src', __DIR__ . '/tests', __DIR__ . '/ecs.php', __DIR__ . '/rector.php']

tests/CommandTest.php

@@ -7,7 +7,10 @@
 use Zing\LaravelSms\Commands\SmsSwitchConnectionCommand;
 use function config;
 
-class CommandTest extends TestCase
+/**
+ * @internal
+ */
+final class CommandTest extends TestCase
 {
     public function testCommand(): void
     {

tests/Drivers/DriverTest.php

@@ -12,7 +12,10 @@
 use Zing\LaravelSms\Exceptions\CouldNotSendNotification;
 use Zing\LaravelSms\Tests\TestCase;
 
-class DriverTest extends TestCase
+/**
+ * @internal
+ */
+final class DriverTest extends TestCase
 {
     public function testException(): void
     {

tests/Drivers/MeiLianDriverTest.php

@@ -12,7 +12,10 @@
 use Zing\LaravelSms\SmsMessage;
 use Zing\LaravelSms\Tests\TestCase;
 
-class MeiLianDriverTest extends TestCase
+/**
+ * @internal
+ */
+final class MeiLianDriverTest extends TestCase
 {
     /**
      * @var string

tests/Drivers/NullDriverTest.php

@@ -10,7 +10,10 @@
 use Zing\LaravelSms\SmsMessage;
 use Zing\LaravelSms\Tests\TestCase;
 
-class NullDriverTest extends TestCase
+/**
+ * @internal
+ */
+final class NullDriverTest extends TestCase
 {
     public function testSend(): void
     {

tests/Drivers/YunpianDriverTest.php

@@ -12,7 +12,10 @@
 use Zing\LaravelSms\SmsMessage;
 use Zing\LaravelSms\Tests\TestCase;
 
-class YunpianDriverTest extends TestCase
+/**
+ * @internal
+ */
+final class YunpianDriverTest extends TestCase
 {
     public function testSend(): void
     {

tests/IntegrationTest.php

@@ -15,7 +15,10 @@
 use Zing\LaravelSms\SmsMessage;
 use Zing\LaravelSms\SmsNumber;
 
-class IntegrationTest extends TestCase
+/**
+ * @internal
+ */
+final class IntegrationTest extends TestCase
 {
     protected function getEnvironmentSetUp($app): void
     {

tests/MessageTest.php

@@ -7,7 +7,10 @@
 use Overtrue\EasySms\Contracts\MessageInterface;
 use Zing\LaravelSms\SmsMessage;
 
-class MessageTest extends TestCase
+/**
+ * @internal
+ */
+final class MessageTest extends TestCase
 {
     public function testStaticCreate(): void
     {

tests/ServiceProviderTest.php

@@ -10,7 +10,10 @@
 use Zing\LaravelSms\Facades\Sms;
 use Zing\LaravelSms\SmsMessage;
 
-class ServiceProviderTest extends TestCase
+/**
+ * @internal
+ */
+final class ServiceProviderTest extends TestCase
 {
     public function testSms(): void
     {

tests/SmsManagerTest.php

@@ -20,7 +20,10 @@
 use Zing\LaravelSms\SmsManager;
 use Zing\LaravelSms\SmsMessage;
 
-class SmsManagerTest extends TestCase
+/**
+ * @internal
+ */
+final class SmsManagerTest extends TestCase
 {
     /**
      * @var string

tests/SmsNumberTest.php

@@ -9,7 +9,10 @@
 use Overtrue\EasySms\Message;
 use Zing\LaravelSms\SmsNumber;
 
-class SmsNumberTest extends TestCase
+/**
+ * @internal
+ */
+final class SmsNumberTest extends TestCase
 {
     /**
      * @param \Overtrue\EasySms\Contracts\PhoneNumberInterface|string $number

tests/TestCase.php

@@ -13,7 +13,7 @@
 use Zing\LaravelSms\Gateways\YunpianGateway;
 use Zing\LaravelSms\SmsServiceProvider;
 
-class TestCase extends BaseTestCase
+abstract class TestCase extends BaseTestCase
 {
     /**
      * @var string
@@ -26,7 +26,7 @@ class TestCase extends BaseTestCase
      */
     public static function assertSameMessage($expected, $actual, string $message = ''): void
     {
-        static::assertThat($actual, new IsEqual($expected), $message);
+        self::assertThat($actual, new IsEqual($expected), $message);
     }
 
     /**

tests/VerificationCodeManagerTest.php

@@ -9,7 +9,10 @@
 use Zing\LaravelSms\SmsNumber;
 use Zing\LaravelSms\VerificationCodeManager;
 
-class VerificationCodeManagerTest extends TestCase
+/**
+ * @internal
+ */
+final class VerificationCodeManagerTest extends TestCase
 {
     use WithFaker;