Return package home URL when no generator supports it (#37)

IonBazan · Dec 9, 2024 · ee8037c · ee8037c
1 parent 4f10027
commit ee8037c
Show file tree

Hide file tree

Showing 2 changed files with 21 additions and 3 deletions.
diff --git a/src/Url/GeneratorContainer.php b/src/Url/GeneratorContainer.php
@@ -2,6 +2,7 @@
 
 namespace IonBazan\ComposerDiff\Url;
 
+use Composer\Package\CompletePackage;
 use Composer\Package\PackageInterface;
 
 class GeneratorContainer implements UrlGenerator
@@ -69,10 +70,14 @@ public function getReleaseUrl(PackageInterface $package)
 
     public function getProjectUrl(PackageInterface $package)
     {
-        if (!$generator = $this->get($package)) {
-            return null;
+        if ($generator = $this->get($package)) {
+            return $generator->getProjectUrl($package);
         }
 
-        return $generator->getProjectUrl($package);
+        if ($package instanceof CompletePackage) {
+            return $package->getHomepage();
+        }
+
+        return null;
     }
 }
diff --git a/tests/Url/GeneratorContainerTest.php b/tests/Url/GeneratorContainerTest.php
@@ -2,6 +2,7 @@
 
 namespace IonBazan\ComposerDiff\Tests\Url;
 
+use Composer\Package\CompletePackage;
 use IonBazan\ComposerDiff\Tests\TestCase;
 use IonBazan\ComposerDiff\Url\GeneratorContainer;
 
@@ -30,4 +31,16 @@ public function testItSupportsPackageSupportedByOneOfTheGenerators()
         self::assertTrue($generator->supportsPackage($this->getPackageWithSource('acme/package', '3.12.0', 'https://bitbucket.org/acme/package.git')));
         self::assertFalse($generator->supportsPackage($this->getPackageWithSource('acme/package', '3.12.0', 'https://non-existent.org/acme/package.git')));
     }
+
+    public function testItReturnsHomepageForProjectUrlWhenNoGeneratorSupportsPackage()
+    {
+        $generator = new GeneratorContainer();
+        $package = new CompletePackage('acme/package', '3.12.0', '3.12.0');
+        $package->setHomepage('https://acme.com');
+
+        self::assertSame('https://acme.com', $generator->getProjectUrl($package));
+
+        $package->setHomepage(null);
+        self::assertNull($generator->getProjectUrl($package));
+    }
 }