add prefer lowest

Author: IonBazan

.github/workflows/test.yml

@@ -8,17 +8,17 @@ jobs:
       fail-fast: false
       matrix:
         php-versions:
-          - '5.4'
-          - '5.5'
-          - '5.6'
-          - '7.0'
-          - '7.1'
-          - '7.2'
-          - '7.3'
+#          - '5.4'
+#          - '5.5'
+#          - '5.6'
+#          - '7.0'
+#          - '7.1'
+#          - '7.2'
+#          - '7.3'
           - '7.4'
         include:
           - php-versions: '5.3'
-              composer-flags: '--prefer-lowest'
+            composer-flags: '--prefer-lowest'
           - php-versions: '8.0'
             composer-flags: '--ignore-platform-reqs'
     steps:

.gitignore

@@ -4,3 +4,4 @@
 /.php_cs
 /tests/fixtures/**/vendor/
 /tests/test-git/
+/coverage/

src/Command/DiffCommand.php

@@ -49,13 +49,18 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
     protected function displayTable(array $operations, $header, OutputInterface $output)
     {
+        if (!\count($operations)) {
+            return;
+        }
+
         $table = new MarkdownTable($output);
+        $table->setHeaders(array($header, 'Base', 'Target'));
 
         foreach ($operations as $operation) {
             $table->addRow($this->getTableRow($operation));
         }
 
-        $table->setHeaders(array($header, 'Base', 'Target'))->render();
+        $table->render();
         $output->writeln('');
     }
 

src/MarkdownTable.php

@@ -2,7 +2,6 @@
 
 namespace IonBazan\ComposerDiff;
 
-use Symfony\Component\Console\Helper\TableSeparator;
 use Symfony\Component\Console\Output\OutputInterface;
 
 class MarkdownTable
@@ -106,11 +105,7 @@ private function getColumnWidth($column)
             return $this->columnWidths[$column];
         }
 
-        foreach (array_merge($this->headers, $this->rows) as $row) {
-            if ($row instanceof TableSeparator) {
-                continue;
-            }
-
+        foreach (array_merge(array($this->headers), $this->rows) as $row) {
             $lengths[] = strlen($row[$column]);
         }
 

src/Plugin.php

@@ -23,13 +23,13 @@ public function activate(Composer $composer, IOInterface $io)
 
     public function getCommands()
     {
-        return array(new DiffCommand($this->composer));
+        return array(new DiffCommand());
     }
 
     public function getCapabilities()
     {
         return array(
-            CommandProvider::class => static::class,
+            'Composer\Plugin\Capability\CommandProvider' => 'IonBazan\ComposerDiff\Plugin',
         );
     }
 }

tests/PackageDiffTest.php

@@ -53,21 +53,21 @@ public function testSameBaseAndTarget()
      */
     public function testGitUsage(array $expected, $dev, $withPlatform)
     {
-        $gitDir = __DIR__.'/test-git';
         $diff = new PackageDiff();
-        @mkdir($gitDir);
-        chdir($gitDir);
-        exec('git init');
-        exec('git config user.name test');
-        exec('git config user.email test@example.com');
-        file_put_contents($gitDir.'/composer.lock', file_get_contents(__DIR__.'/fixtures/base/composer.lock'));
-        exec('git add composer.lock && git commit -m "init"');
-        file_put_contents($gitDir.'/composer.lock', file_get_contents(__DIR__.'/fixtures/target/composer.lock'));
-        $operations = $diff->getPackageDiff('HEAD:composer.lock', 'composer.lock', $dev, $withPlatform);
+        $this->prepareGit();
+        $operations = $diff->getPackageDiff('HEAD', '', $dev, $withPlatform);
 
         $this->assertSame($expected, array_map(array($this, 'operationToString'), $operations));
     }
 
+    public function testInvalidGitRef()
+    {
+        $diff = new PackageDiff();
+        $this->prepareGit();
+        $this->expectException('RuntimeException');
+        $diff->getPackageDiff('invalid-ref', '');
+    }
+
     public function operationsProvider()
     {
         return array(
@@ -123,6 +123,19 @@ public function operationsProvider()
         );
     }
 
+    private function prepareGit()
+    {
+        $gitDir = __DIR__.'/test-git';
+        @mkdir($gitDir);
+        chdir($gitDir);
+        exec('git init');
+        exec('git config user.name test');
+        exec('git config user.email test@example.com');
+        file_put_contents($gitDir.'/composer.lock', file_get_contents(__DIR__.'/fixtures/base/composer.lock'));
+        exec('git add composer.lock && git commit -m "init"');
+        file_put_contents($gitDir.'/composer.lock', file_get_contents(__DIR__.'/fixtures/target/composer.lock'));
+    }
+
     private function operationToString(OperationInterface $operation)
     {
         if ($operation instanceof InstallOperation) {

tests/PluginTest.php

@@ -0,0 +1,25 @@
+<?php
+
+namespace IonBazan\ComposerDiff\Tests;
+
+use IonBazan\ComposerDiff\Command\DiffCommand;
+use IonBazan\ComposerDiff\Plugin;
+use PHPUnit\Framework\TestCase;
+
+class PluginTest extends TestCase
+{
+    public function testPlugin()
+    {
+        $composer = $this->createMock('Composer\Composer');
+        $io = $this->createMock('Composer\IO\IOInterface');
+
+        $plugin = new Plugin();
+        $plugin->activate($composer, $io);
+
+        $this->assertSame(
+            array('Composer\Plugin\Capability\CommandProvider' => 'IonBazan\ComposerDiff\Plugin'),
+            $plugin->getCapabilities()
+        );
+        $this->assertEquals(array(new DiffCommand()), $plugin->getCommands());
+    }
+}