Don't rely on code scanning order for test

cspray · Jul 20, 2022 · 5d59760 · 5d59760
1 parent bcae9ac
commit 5d59760
Show file tree

Hide file tree

Showing 2 changed files with 7 additions and 3 deletions.
diff --git a/phpunit.xml b/phpunit.xml
@@ -28,7 +28,7 @@
         </exclude>
         <report>
             <text outputFile="php://stdout" showOnlySummary="true" />
-            <html outputDirectory="build" />
+            <!-- <html outputDirectory="build" /> -->
         </report>
     </coverage>
 </phpunit>
diff --git a/tests/SourceArchitecturalDecisionAttributeRegistryTest.php b/tests/SourceArchitecturalDecisionAttributeRegistryTest.php
@@ -6,6 +6,7 @@
 use Cspray\ArchitecturalDecision\Stub\Adr\StubDocBlockArchitecturalDecision;
 use Cspray\ArchitecturalDecision\Stub\Adr\StubMetaDataArchitecturalDecision;
 use Cspray\ArchitecturalDecision\Stub\BadAdr\MissingDocBlockArchitecturalDecision;
+use Cspray\Typiphy\ObjectType;
 use PHPUnit\Framework\TestCase;
 use function Cspray\Typiphy\objectType;
 
@@ -20,11 +21,14 @@ public function testSourceDirectoryContainsAttributes() : void {
         $attributes = $subject->getArchitecturalDecisionAttributes();
 
         self::assertCount(4, $attributes);
+
+        usort($attributes, fn(ObjectType $a, ObjectType $b) => $a->getName() <=> $b->getName());
+
         self::assertSame([
-            objectType(MissingDocBlockArchitecturalDecision::class),
-            objectType(StubDocBlockArchitecturalDecision::class),
             objectType(AnotherDocBlockArchitecturalDecision::class),
+            objectType(StubDocBlockArchitecturalDecision::class),
             objectType(StubMetaDataArchitecturalDecision::class),
+            objectType(MissingDocBlockArchitecturalDecision::class),
         ], $attributes);
     }