Skip Pedentry method sort for generated files

Author: GromNaN

src/Builder/Variable.php

@@ -18,18 +18,6 @@
  */
 final class Variable
 {
-    /**
-     * A variable that returns the current datetime value.
-     * NOW returns the same value for all members of the deployment and remains the same throughout all stages of the
-     * aggregation pipeline.
-     *
-     * New in MongoDB 4.2.
-     */
-    public static function now(): ResolvesToDate
-    {
-        return new Expression\Variable('NOW');
-    }
-
     /**
      * A variable that returns the current timestamp value.
      * CLUSTER_TIME is only available on replica sets and sharded clusters.
@@ -43,15 +31,6 @@ public static function clusterTime(): ResolvesToTimestamp
         return new Expression\Variable('CLUSTER_TIME');
     }
 
-    /**
-     * References the root document, i.e. the top-level document, currently being processed in the aggregation pipeline
-     * stage.
-     */
-    public static function root(): ResolvesToObject
-    {
-        return new Expression\Variable('ROOT');
-    }
-
     /**
      * References the start of the field path being processed in the aggregation pipeline stage.
      * Unless documented otherwise, all stages start with CURRENT the same as ROOT.
@@ -64,29 +43,43 @@ public static function current(string $fieldPath = ''): ResolvesToAny
     }
 
     /**
-     * A variable which evaluates to the missing value. Allows for the conditional exclusion of fields. In a $project,
-     * a field set to the variable REMOVE is excluded from the output.
-     * Can be used with $cond operator for conditionally exclude fields.
+     * One of the allowed results of a $redact expression.
      *
-     * @see https://www.mongodb.com/docs/manual/reference/operator/aggregation/project/#std-label-remove-example
+     * $redact returns the fields at the current document level, excluding embedded documents. To include embedded
+     * documents and embedded documents within arrays, apply the $cond expression to the embedded documents to determine
+     * access for these embedded documents.
+     *
+     * @see https://www.mongodb.com/docs/manual/reference/operator/aggregation/redact/#mongodb-pipeline-pipe.-redact
      */
-    public static function remove(): ResolvesToAny
+    public static function descend(): ExpressionInterface
     {
-        return new Expression\Variable('REMOVE');
+        return new Expression\Variable('DESCEND');
     }
 
     /**
      * One of the allowed results of a $redact expression.
      *
-     * $redact returns the fields at the current document level, excluding embedded documents. To include embedded
-     * documents and embedded documents within arrays, apply the $cond expression to the embedded documents to determine
-     * access for these embedded documents.
+     * $redact returns or keeps all fields at this current document/embedded document level, without further inspection
+     * of the fields at this level. This applies even if the included field contains embedded documents that may have
+     * different access levels.
      *
      * @see https://www.mongodb.com/docs/manual/reference/operator/aggregation/redact/#mongodb-pipeline-pipe.-redact
      */
-    public static function descend(): ExpressionInterface
+    public static function keep(): ExpressionInterface
     {
-        return new Expression\Variable('DESCEND');
+        return new Expression\Variable('KEEP');
+    }
+
+    /**
+     * A variable that returns the current datetime value.
+     * NOW returns the same value for all members of the deployment and remains the same throughout all stages of the
+     * aggregation pipeline.
+     *
+     * New in MongoDB 4.2.
+     */
+    public static function now(): ResolvesToDate
+    {
+        return new Expression\Variable('NOW');
     }
 
     /**
@@ -104,17 +97,24 @@ public static function prune(): ExpressionInterface
     }
 
     /**
-     * One of the allowed results of a $redact expression.
-     *
-     * $redact returns or keeps all fields at this current document/embedded document level, without further inspection
-     * of the fields at this level. This applies even if the included field contains embedded documents that may have
-     * different access levels.
+     * A variable which evaluates to the missing value. Allows for the conditional exclusion of fields. In a $project,
+     * a field set to the variable REMOVE is excluded from the output.
+     * Can be used with $cond operator for conditionally exclude fields.
      *
-     * @see https://www.mongodb.com/docs/manual/reference/operator/aggregation/redact/#mongodb-pipeline-pipe.-redact
+     * @see https://www.mongodb.com/docs/manual/reference/operator/aggregation/project/#std-label-remove-example
      */
-    public static function keep(): ExpressionInterface
+    public static function remove(): ResolvesToAny
     {
-        return new Expression\Variable('KEEP');
+        return new Expression\Variable('REMOVE');
+    }
+
+    /**
+     * References the root document, i.e. the top-level document, currently being processed in the aggregation pipeline
+     * stage.
+     */
+    public static function root(): ResolvesToObject
+    {
+        return new Expression\Variable('ROOT');
     }
 
     /**

tests/PedantryTest.php

@@ -2,6 +2,7 @@
 
 namespace MongoDB\Tests;
 
+use MongoDB;
 use RecursiveDirectoryIterator;
 use RecursiveIteratorIterator;
 use ReflectionClass;
@@ -10,6 +11,7 @@
 
 use function array_filter;
 use function array_map;
+use function in_array;
 use function realpath;
 use function str_contains;
 use function str_replace;
@@ -25,6 +27,11 @@
  */
 class PedantryTest extends TestCase
 {
+    private const SKIPPED_CLASSES = [
+        // Generated
+        MongoDB\Builder\Stage\FluentFactoryTrait::class,
+    ];
+
     /** @dataProvider provideProjectClassNames */
     public function testMethodsAreOrderedAlphabeticallyByVisibility($className): void
     {
@@ -74,6 +81,10 @@ public function provideProjectClassNames()
             }
 
             $className = 'MongoDB\\' . str_replace(DIRECTORY_SEPARATOR, '\\', substr($file->getRealPath(), strlen($srcDir) + 1, -4));
+            if (in_array($className, self::SKIPPED_CLASSES)) {
+                continue;
+            }
+
             $classNames[$className][] = $className;
         }