Merge pull request #498 from asgrim/1-3-update-example-pie-ext

1.3.x - update example pie extension for tests

Author: asgrim

composer.json

@@ -28,7 +28,7 @@
     ],
     "require": {
         "php": "8.1.*||8.2.*||8.3.*||8.4.*||8.5.*",
-        "composer/composer": "^2.9.2",
+        "composer/composer": "^2.9.5",
         "composer/pcre": "^3.3.2",
         "composer/semver": "^3.4.4",
         "fidry/cpu-core-counter": "^1.3.0",

composer.lock

@@ -13,7 +13,7 @@ Feature: Extensions can be downloaded with PIE
     Examples:
       | constraint | version  |
       | 2.0.5      | 2.0.5    |
-      | ^2.0       | 2.0.5    |
+      | ^2.0       | 2.0.8    |
 
   # pie download <ext>:dev-main
   @non-windows

features/download-extensions.feature

@@ -420,26 +420,14 @@ parameters:
 			count: 1
 			path: test/unit/ComposerIntegration/VendorCleanupTest.php
 
-		-
-			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{download\-url\-method\: array\{''composer\-default''\}\} given\.$#'
-			identifier: argument.type
-			count: 1
-			path: test/unit/DependencyResolver/PackageTest.php
-
-		-
-			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{download\-url\-method\: array\{''pre\-packaged\-source'', ''composer\-default''\}\} given\.$#'
-			identifier: argument.type
-			count: 1
-			path: test/unit/DependencyResolver/PackageTest.php
-
 		-
 			message: '#^Parameter \#2 \$extractedSourcePath of static method Php\\Pie\\Downloading\\DownloadedPackage\:\:fromPackageAndExtractedPath\(\) expects string, string\|false given\.$#'
 			identifier: argument.type
 			count: 3
 			path: test/unit/Downloading/DownloadedPackageTest.php
 
 		-
-			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{extension\-name\: null\} given\.$#'
+			message: '#^Parameter \#1 \$phpExt of method Composer\\Package\\Package\:\:setPhpExt\(\) expects array\{extension\-name\?\: string, priority\?\: int, support\-zts\?\: bool, support\-nts\?\: bool, build\-path\?\: string\|null, download\-url\-method\?\: list\<string\>\|string, os\-families\?\: non\-empty\-list\<non\-empty\-string\>, os\-families\-exclude\?\: non\-empty\-list\<non\-empty\-string\>, \.\.\.\}\|null, array\{extension\-name\: null\} given\.$#'
 			identifier: argument.type
 			count: 1
 			path: test/unit/ExtensionNameTest.php

phpstan-baseline.neon

@@ -94,11 +94,11 @@ public static function fromComposerCompletePackage(CompletePackageInterface $com
             /** @var string|list<string> $method */
             $method = $phpExtOptions['download-url-method'];
             if (is_array($method)) {
-                if (count($method) !== 1) {
-                    throw new InvalidArgumentException('This extension requires a newer version of PIE. Multiple download-url-methods are not supported until PIE 1.4.0.');
+                if (count($method) === 1) {
+                    $method = $method[0];
+                } else {
+                    $method = DownloadUrlMethod::ComposerDefaultDownload->value;
                 }
-
-                $method = $method[0];
             }
 
             $package->downloadUrlMethod = DownloadUrlMethod::tryFrom($method);

src/DependencyResolver/Package.php

@@ -23,7 +23,7 @@
 #[CoversClass(DownloadCommand::class)]
 class DownloadCommandTest extends TestCase
 {
-    private const TEST_PACKAGE_LATEST = '2.0.5';
+    private const TEST_PACKAGE_LATEST = '2.0.8';
     private const TEST_PACKAGE        = 'asgrim/example-pie-extension';
 
     private CommandTester $commandTester;

test/integration/Command/DownloadCommandTest.php

@@ -26,7 +26,7 @@
 #[CoversClass(ResolveDependencyWithComposer::class)]
 final class ResolveDependencyWithComposerTest extends TestCase
 {
-    private const TEST_PACKAGE_LATEST        = '2.0.5';
+    private const TEST_PACKAGE_LATEST        = '2.0.8';
     private const DOWNLOAD_URL_ANY           = 'https://api.github.com/repos/asgrim/example-pie-extension/zipball/%s';
     private const DOWNLOAD_URL_1_0_1_ALPHA_3 = 'https://api.github.com/repos/asgrim/example-pie-extension/zipball/115f8f8e01ee098a18ec2f47af4852be51ebece7';
     private const DOWNLOAD_URL_1_0_1         = 'https://api.github.com/repos/asgrim/example-pie-extension/zipball/769f906413d6d1e12152f6d34134cbcd347ca253';

test/integration/DependencyResolver/ResolveDependencyWithComposerTest.php

@@ -173,8 +173,8 @@ public function testDownloadUrlMethodWithMultiItemListIsNotYetSupported(): void
         $composerCompletePackage = new CompletePackage('vendor/foo', '1.2.3.0', '1.2.3');
         $composerCompletePackage->setPhpExt(['download-url-method' => ['pre-packaged-source', 'composer-default']]);
 
-        $this->expectException(InvalidArgumentException::class);
-        $this->expectExceptionMessage('This extension requires a newer version of PIE. Multiple download-url-methods are not supported until PIE 1.4.0.');
-        Package::fromComposerCompletePackage($composerCompletePackage);
+        $package = Package::fromComposerCompletePackage($composerCompletePackage);
+
+        self::assertSame(DownloadUrlMethod::ComposerDefaultDownload, $package->downloadUrlMethod());
     }
 }