436: bit of test and more useful exception message

Author: asgrim

src/Building/UnixBuild.php

@@ -36,7 +36,8 @@ public function __invoke(
         IOInterface $io,
         PhpizePath|null $phpizePath,
     ): BinaryFile {
-        switch (DownloadUrlMethod::fromDownloadedPackage($downloadedPackage)) {
+        $selectedDownloadMethod = DownloadUrlMethod::fromDownloadedPackage($downloadedPackage);
+        switch ($selectedDownloadMethod) {
             case DownloadUrlMethod::PrePackagedBinary:
                 return $this->prePackagedBinary($downloadedPackage, $io);
 
@@ -45,7 +46,7 @@ public function __invoke(
                 return $this->buildFromSource($downloadedPackage, $targetPlatform, $configureOptions, $io, $phpizePath);
 
             default:
-                throw new LogicException('Unknown download method');
+                throw new LogicException('Unsupported download method: ' . $selectedDownloadMethod->value);
         }
     }
 

test/unit/Building/ExtensionBinaryNotFoundTest.php

@@ -0,0 +1,29 @@
+<?php
+
+declare(strict_types=1);
+
+namespace Php\PieUnitTest\Building;
+
+use Php\Pie\Building\ExtensionBinaryNotFound;
+use PHPUnit\Framework\Attributes\CoversClass;
+use PHPUnit\Framework\TestCase;
+
+#[CoversClass(ExtensionBinaryNotFound::class)]
+final class ExtensionBinaryNotFoundTest extends TestCase
+{
+    public function testFromPrePackagedBinary(): void
+    {
+        self::assertSame(
+            'Expected pre-packaged binary does not exist: /foo/bar',
+            ExtensionBinaryNotFound::fromPrePackagedBinary('/foo/bar')->getMessage(),
+        );
+    }
+
+    public function testFromExpectedBinary(): void
+    {
+        self::assertSame(
+            'Build complete, but expected /foo/bar does not exist.',
+            ExtensionBinaryNotFound::fromExpectedBinary('/foo/bar')->getMessage(),
+        );
+    }
+}