Merge pull request #52 from nojimage/php-56-test

Add test and CI for PHP 5.6

Author: ADmad

.travis.yml

@@ -5,16 +5,24 @@ php:
   - 7.4
   - 'nightly'
 
+env:
+  global:
+    - DEFAULT=1
+
 matrix:
   include:
     - php: 7.2
-      env: PHPCS=1
+      env: PHPCS=1 DEFAULT=0
+
+    - php: 5.6
+      env: PHP56=1 DEFAULT=0
 
 install:
   - composer install --no-interaction
 
 script:
-  - if [[ $PHPCS != 1 ]]; then vendor/bin/phpunit; fi
+  - if [[ $DEFAULT == 1 ]]; then vendor/bin/phpunit; fi
+  - if [[ $PHP56 == 1 ]]; then vendor/bin/phpunit tests/php56/; fi
   - if [[ $PHPCS == 1 ]]; then vendor/bin/phpcs -p -n --extensions=php --standard=vendor/cakephp/cakephp-codesniffer/CakePHP ./src ./tests; fi
 
 notifications:

composer.json

@@ -16,7 +16,7 @@
     "require-dev": {
         "cakephp/cakephp-codesniffer": "^3.3",
         "composer/composer": "^2.0",
-        "phpunit/phpunit": "^8.5 || ^9.3"
+        "phpunit/phpunit": "^5.7 || ^6.5 || ^8.5 || ^9.3"
     },
     "autoload": {
         "psr-4": {
@@ -25,7 +25,8 @@
     },
     "autoload-dev": {
         "psr-4": {
-            "Cake\\Test\\TestCase\\Composer\\": "tests/"
+            "Cake\\Test\\TestCase\\Composer\\": "tests/TestCase/",
+            "Cake\\Test\\TestCase\\Composer\\Php56\\": "tests/php56/TestCase/"
         }
     },
     "extra": {

src/Installer/PluginInstaller.php

@@ -1,6 +1,4 @@
 <?php
-declare(strict_types=1);
-
 namespace Cake\Composer\Installer;
 
 use Composer\Installer\LibraryInstaller;

tests/php56/TestCase/Installer/PluginInstallerTest.php

@@ -0,0 +1,42 @@
+<?php
+namespace Cake\Test\TestCase\Composer\Php56\Installer;
+
+use Cake\Composer\Installer\PluginInstaller;
+use Composer\Composer;
+use Composer\IO\IOInterface;
+use Composer\Package\RootPackage;
+use Composer\Script\Event;
+use PHPUnit\Framework\TestCase;
+
+class PluginInstallerTest extends TestCase
+{
+    /**
+     * setUp
+     *
+     * @return void
+     */
+    public function setUp()
+    {
+        parent::setUp();
+
+        $this->composer = new Composer();
+        $this->io = $this->getMockBuilder(IOInterface::class)->getMock();
+    }
+
+    public function testPostAutoloadDump()
+    {
+        $rootPackage = new RootPackage('cakephp/app', '1.0', '1.0');
+        $rootPackage->setType('project');
+        $rootPackage->setScripts([
+            'post-autoload-dump' => 'Cake\Composer\Installer\PluginInstaller::postAutoloadDump',
+        ]);
+
+        $this->composer->setPackage($rootPackage);
+        $this->io->expects($this->once())
+                ->method('write');
+
+        $event = new Event('post-autoload-dump', $this->composer, $this->io);
+
+        PluginInstaller::postAutoloadDump($event);
+    }
+}

tests/php56/TestCase/PHPUnitAssertionCompatTrait.php

@@ -0,0 +1,11 @@
+<?php
+
+namespace Cake\Test\TestCase\Composer\Php56;
+
+trait PHPUnitAssertionCompatTrait
+{
+    public function assertStringContainsString($needle, $haystack, $message = '')
+    {
+        $this->assertContains($needle, $haystack, $message);
+    }
+}