bug #1335 Ignore use statement generator duplicates (mdoutreluingne)

This PR was squashed before being merged into the 1.0-dev branch.

Discussion
----------

Ignore use statement generator duplicates

Follow #1328 (comment)

Following weaverryan's idea I've improved `UseStatementGenerator` in this PR to ignore duplicates.

Commits
-------

1bfaf30 Ignore use statement generator duplicates

Author: weaverryan

src/Util/UseStatementGenerator.php

@@ -45,7 +45,11 @@ public function __toString(): string
                 $class = $aliasClass;
             }
 
-            $transformed[$key] = str_replace('\\', ' ', $class);
+            $transformedClass = str_replace('\\', ' ', $class);
+            // Let's not add the class again if it already exists.
+            if (!\in_array($transformedClass, $transformed, true)) {
+                $transformed[$key] = $transformedClass;
+            }
         }
 
         asort($transformed);

tests/Util/UseStatementGeneratorTest.php

@@ -90,4 +90,20 @@ public function testUseStatementsWithAliases(): void
             EOT;
         self::assertSame($expected, (string) $unsorted);
     }
+
+    public function testUseStatementsWithDuplicates(): void
+    {
+        $unsorted = new UseStatementGenerator([
+            \Symfony\UX\Turbo\Attribute\Broadcast::class,
+            \ApiPlatform\Core\Annotation\ApiResource::class,
+            \ApiPlatform\Core\Annotation\ApiResource::class,
+        ]);
+
+        $expected = <<< 'EOT'
+            use ApiPlatform\Core\Annotation\ApiResource;
+            use Symfony\UX\Turbo\Attribute\Broadcast;
+
+            EOT;
+        self::assertSame($expected, (string) $unsorted);
+    }
 }