Do not sort

sebastianbergmann · Dec 14, 2024 · 2e084f3 · 2e084f3
1 parent b36a500
commit 2e084f3
Show file tree

Hide file tree

Showing 4 changed files with 2 additions and 8 deletions.
diff --git a/src/Target/MapBuilder.php b/src/Target/MapBuilder.php
@@ -13,7 +13,6 @@
 use function array_merge;
 use function array_unique;
 use function range;
-use function sort;
 use SebastianBergmann\CodeCoverage\Filter;
 use SebastianBergmann\CodeCoverage\StaticAnalysis\FileAnalyser;
 
@@ -104,8 +103,6 @@ public function build(Filter $filter, FileAnalyser $analyser): array
         foreach (array_keys($namespaces) as $namespace) {
             foreach (array_keys($namespaces[$namespace]) as $file) {
                 $namespaces[$namespace][$file] = array_unique($namespaces[$namespace][$file]);
-
-                sort($namespaces[$namespace][$file]);
             }
         }
 

diff --git a/src/Target/Mapper.php b/src/Target/Mapper.php
@@ -11,7 +11,6 @@
 
 use function array_merge;
 use function array_unique;
-use function sort;
 
 /**
  * @phpstan-type TargetMap array{namespaces: TargetMapPart, classes: TargetMapPart, classesThatExtendClass: TargetMapPart, classesThatImplementInterface: TargetMapPart, traits: TargetMapPart, methods: TargetMapPart, functions: TargetMapPart, reverseLookup: ReverseLookup}
@@ -55,8 +54,6 @@ public function mapTargets(TargetCollection $targets): array
                 }
 
                 $result[$file] = array_unique(array_merge($result[$file], $lines));
-
-                sort($result[$file]);
             }
         }
 

diff --git a/tests/tests/Target/MapBuilderTest.php b/tests/tests/Target/MapBuilderTest.php
@@ -34,9 +34,9 @@ public function testBuildsMap(): void
                 'namespaces' => [
                     'SebastianBergmann\\CodeCoverage\\StaticAnalysis' => [
                         $file => array_merge(
-                            range(19, 24),
                             range(26, 31),
                             range(33, 52),
+                            range(19, 24),
                             range(54, 56),
                         ),
                     ],

diff --git a/tests/tests/Target/MapperTest.php b/tests/tests/Target/MapperTest.php
@@ -100,9 +100,9 @@ public static function provider(): array
             'namespace' => [
                 [
                     $file => array_merge(
-                        range(19, 24),
                         range(26, 31),
                         range(33, 52),
+                        range(19, 24),
                         range(54, 56),
                     ),
                 ],