Support packagist 2 and use it by default

Author: kylekatarnls

src/MultiTester/MultiTester.php

@@ -40,13 +40,10 @@ public function exec($command, $quiet = false)
  : $this->execCommand($command, $quiet);
  }
 
- public function getComposerSettings($package)
+ public function getComposerSettings($package, $platforms = null)
  {
  if (!isset($this->composerSettings[$package])) {
- $file = new File("https://repo.packagist.org/p/$package.json");
- $this->composerSettings[$package] = $file->isValid()
- ? (($file['packages'] ?? [])[$package] ?? null)
- : $this->getFromLibrariesIo($package);
+ $this->composerSettings[$package] = $this->getFromFirstValidPlatform($package, $platforms);
  }
 
  return $this->composerSettings[$package];
@@ -285,10 +282,88 @@ protected function error($message)
  new MultiTesterException($message);
  }
 
- private function getFromLibrariesIo(string $package): ?File
+ private function getSourceFromLibrariesIo($package): ?array
  {
  $file = new File('https://libraries.io/api/Packagist/' . urlencode($package), 'json');
 
- return $file->isValid() ? $file : null;
+ return $file->isValid() ? $file->toArray() : null;
+ }
+
+ private function getSourceFromPackagist($package, $namespace = 'p2'): ?array
+ {
+ $file = new File("https://repo.packagist.org/$namespace/$package.json");
+
+ return $file->isValid()
+ ? (($file['packages'] ?? [])[$package] ?? null)
+ : null;
+ }
+
+ private function getSourceFromPackagist2($package): ?array
+ {
+ $list = $this->getSourceFromPackagist($package);
+
+ if (!is_array($list)) {
+ return $list;
+ }
+
+ $listByVersion = [];
+ $previousItem = [];
+
+ foreach ($list as $item) {
+ if (isset($item['version'])) {
+ $previousItem = array_merge($previousItem, $item);
+ $listByVersion[$item['version']] = $previousItem;
+ }
+ }
+
+ return $listByVersion;
+ }
+
+ private function getSourceFromPlatform($package, $platform): ?array
+ {
+ switch ($platform) {
+ case 'composer1':
+ case 'packagist1':
+ return $this->getSourceFromPackagist($package, 'p');
+
+ case 'composer':
+ case 'packagist':
+ case 'composer2':
+ case 'packagist2':
+ case 'packagist.org':
+ return $this->getSourceFromPackagist2($package);
+
+ case 'libraries':
+ case 'libraries.io':
+ return $this->getSourceFromLibrariesIo($package);
+
+ default:
+ $this->error("Unknown platform '$platform'");
+ }
+ }
+
+ /**
+ * @param mixed $platforms
+ */
+ private function parsePlatformList($platforms): array
+ {
+ if (is_string($platforms)) {
+ return array_values(array_filter(preg_split('/[,\s]+/', $platforms)));
+ }
+
+ return (array) ($platforms ?? ['packagist.org', 'libraries.io']);
+ }
+
+ private function getFromFirstValidPlatform($package, $platforms): ?array
+ {
+ foreach ($this->parsePlatformList($platforms) as $platform) {
+ $result = $this->getSourceFromPlatform($package, $platform);
+
+ if ($result !== null) {
+ return $result;
+ }
+ }
+
+ return null;
  }
 }

src/MultiTester/Project.php

@@ -174,7 +174,7 @@ protected function seedSourceSetting(&$settings)
 
  $package = $this->getPackage();
  $tester = $this->getConfig()->getTester();
- $composerSettings = $tester->getComposerSettings($package);
+ $composerSettings = $tester->getComposerSettings($package, $settings['platforms'] ?? null);
  $version = is_array($composerSettings)
  ? $this->filterVersion($settings['version'], array_keys((array) ($composerSettings ?: [])))
  : '';

tests/MultiTesterTest.php

@@ -262,7 +262,7 @@ public function testLibrariesIoFallback()
  $method = new ReflectionMethod($tester, 'getComposerSettings');
  $method->setAccessible(true);
 
- $package = $method->invoke($tester, 'pug/pug');
+ $package = $method->invoke($tester, 'pug/pug', 'packagist1, libraries.io');
 
  $cwd = @getcwd() ?: '.';
  chdir(__DIR__ . '/project');
@@ -280,6 +280,21 @@ public function testLibrariesIoFallback()
  ], $source);
  }
 
+ /**
+ * @throws \ReflectionException
+ */
+ public function testInvalidPlatform()
+ {
+ $this->expectException(MultiTesterException::class);
+ $this->expectExceptionMessage("Unknown platform 'foobar'");
+
+ $tester = new MultiTester();
+ $method = new ReflectionMethod($tester, 'getComposerSettings');
+ $method->setAccessible(true);
+
+ $method->invoke($tester, 'pug/pug', 'packagist1, foobar');
+ }
+
  /**
  * @throws \ReflectionException
  */