Trigger composer task also when composer.lock changes.

phpro · Jun 12, 2024 · 3d53d2c · 3d53d2c
1 parent 0106169
commit 3d53d2c
Show file tree

Hide file tree

Showing 2 changed files with 24 additions and 2 deletions.
diff --git a/src/Task/Composer.php b/src/Task/Composer.php
@@ -67,9 +67,12 @@ public function canRunInContext(ContextInterface $context): bool
     public function run(ContextInterface $context): TaskResultInterface
     {
         $config = $this->getConfig()->getOptions();
+        $composerDir = pathinfo($config['file'], PATHINFO_DIRNAME);
+        $composerFile = pathinfo($config['file'], PATHINFO_BASENAME);
+        $composerLockFile = $this->getLockFile($composerFile);
         $files = $context->getFiles()
-            ->path(pathinfo($config['file'], PATHINFO_DIRNAME))
-            ->name(pathinfo($config['file'], PATHINFO_BASENAME));
+            ->path($composerDir)
+            ->names([$composerFile, $composerLockFile]);
         if (0 === \count($files)) {
             return TaskResult::createSkipped($this, $context);
         }
@@ -116,4 +119,14 @@ private function hasLocalRepository(SplFileInfo $composerFile): bool
 
         return false;
     }
+
+    /**
+     * Verbatim copy from \Composer\Factory::getLockFile.
+     */
+    private static function getLockFile(string $composerFile): string
+    {
+        return 'json' === pathinfo($composerFile, PATHINFO_EXTENSION)
+            ? substr($composerFile, 0, -4) . 'lock'
+            : $composerFile . '.lock';
+    }
 }
diff --git a/test/Unit/Task/ComposerTest.php b/test/Unit/Task/ComposerTest.php
@@ -195,6 +195,15 @@ public function provideExternalTaskRuns(): iterable
                 './composer.json',
             ]
         ];
+        yield 'lock-only' => [
+            [],
+            $this->mockContext(RunContext::class, ['composer.lock']),
+            'composer',
+            [
+                'validate',
+                './composer.json',
+            ]
+        ];
         yield 'no-check-all' => [
             [
                 'no_check_all' => true,