minor symfony#52351 [AssetMapper] Fixing merge (weaverryan)

This PR was merged into the 6.4 branch.

Discussion
----------

[AssetMapper] Fixing merge

| Q             | A
| ------------- | ---
| Branch?       | 6.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | None
| License       | MIT

Fixing the merge of symfony#52323, symfony#52331 and symfony#52349

Also tested on a real project locally to verify the moving pieces :).

Thanks!

Commits
-------

99d5cbb [AssetMapper] Fixing merge from multiple PR's

Author: xabbuh

src/Symfony/Component/AssetMapper/Compiler/JavaScriptImportPathCompiler.php

@@ -172,7 +172,12 @@ private function findAssetForRelativeImport(string $importedModule, MappedAsset
             return null;
         }
 
-        $dependentAsset = $assetMapper->getAsset($resolvedSourcePath);
+        try {
+            $dependentAsset = $assetMapper->getAssetFromSourcePath($resolvedSourcePath);
+        } catch (CircularAssetsException $exception) {
+            $dependentAsset = $exception->getIncompleteMappedAsset();
+        }
+
         if ($dependentAsset) {
             return $dependentAsset;
         }

src/Symfony/Component/AssetMapper/Tests/Compiler/JavaScriptImportPathCompilerTest.php

@@ -173,7 +173,6 @@ public static function provideCompileTests(): iterable
         ];
 
         yield 'importing_non_existent_file_without_strict_mode_is_ignored_and_no_import_added' => [
-            'sourceLogicalName' => 'app.js',
             'input' => "import './non-existent.js';",
             'expectedJavaScriptImports' => [],
         ];
@@ -277,7 +276,6 @@ public static function provideCompileTests(): iterable
         ];
 
         yield 'absolute_import_ignored_and_no_dependency_added' => [
-            'sourceLogicalName' => 'app.js',
             'input' => 'import "https://example.com/module.js";',
             'expectedJavaScriptImports' => [],
         ];
@@ -415,14 +413,14 @@ public static function providePathsCanUpdateTests(): iterable
 
     public function testCompileHandlesCircularRelativeAssets()
     {
-        $appAsset = new MappedAsset('app.js', 'anythingapp', '/assets/app.js');
-        $otherAsset = new MappedAsset('other.js', 'anythingother', '/assets/other.js');
+        $appAsset = new MappedAsset('app.js', '/project/assets/app.js', '/assets/app.js');
+        $otherAsset = new MappedAsset('other.js', '/project/assets/other.js', '/assets/other.js');
 
         $importMapConfigReader = $this->createMock(ImportMapConfigReader::class);
         $assetMapper = $this->createMock(AssetMapperInterface::class);
         $assetMapper->expects($this->once())
-            ->method('getAsset')
-            ->with('other.js')
+            ->method('getAssetFromSourcePath')
+            ->with('/project/assets/other.js')
             ->willThrowException(new CircularAssetsException($otherAsset));
 
         $compiler = new JavaScriptImportPathCompiler($importMapConfigReader);