Cleanup unused test skipping

Author: gapple

tests/ParsingRulesetTrait.php

@@ -22,12 +22,6 @@ public static function parseRulesetDataProvider(): array
     #[DataProvider('parseRulesetDataProvider')]
     public function testParsing(Rule $record): void
     {
-        if (array_key_exists($record->name, $this->skipParsingRules)) {
-            $this->markTestSkipped(
-                'Skipped "' . $record->name . '": ' . $this->skipParsingRules[$record->name]
-            );
-        }
-
         try {
             $raw = implode(', ', $record->raw);
             $parsedValue = Parser::{'parse' . ucfirst($record->header_type)}($raw);

tests/RulesetTestBase.php

@@ -6,26 +6,6 @@
 
 abstract class RulesetTestBase extends TestCase
 {
-    /**
-     * An array of rules which should skip the parsing test.
-     *
-     * The element key should be the name of the rule, and the value should be
-     * the message to provide for skipping the rule.
-     *
-     * @var array<string, string>
-     */
-    protected array $skipParsingRules = [];
-
-    /**
-     * An array of rules which should skip the serializing test.
-     *
-     * The element key should be the name of the rule, and the value should be
-     * the message to provide for skipping the rule.
-     *
-     * @var array<string, string>
-     */
-    protected array $skipSerializingRules = [];
-
     /**
      * @return array<string, array{Rule}>
      */

tests/SerializingRulesetTrait.php

@@ -22,12 +22,6 @@ public static function serializeRulesetDataProvider(): array
     #[DataProvider('serializeRulesetDataProvider')]
     public function testSerializing(Rule $record): void
     {
-        if (array_key_exists($record->name, $this->skipSerializingRules)) {
-            $this->markTestSkipped(
-                'Skipped "' . $record->name . '": ' . $this->skipSerializingRules[$record->name]
-            );
-        }
-
         try {
             $serializedValue = Serializer::{'serialize' . ucfirst($record->header_type)}($record->expected);