Raise deps (#32)

* PHPUnit upgrade

* Upgrade PHP and Symfony min versions

* New PHP version upgrades

* Add github workflow CI

* Raise mockery min version

* CI badge

Author: Padam87

.coveralls.yml

@@ -0,0 +1,46 @@
+name: CI
+
+on:
+  pull_request:
+    branches:
+      - "master"
+  push:
+    branches:
+      - "master"
+
+jobs:
+  phpunit:
+    name: "PHPUnit"
+    runs-on: "ubuntu-20.04"
+
+    strategy:
+      matrix:
+        php-version:
+          - "8.0"
+          - "8.1"
+        dependencies:
+          - "highest"
+        include:
+          - php-version: "8.0"
+            dependencies: "lowest"
+
+    steps:
+      - name: "Checkout"
+        uses: "actions/checkout@v2"
+        with:
+          fetch-depth: 2
+
+      - name: "Install PHP"
+        uses: "shivammathur/setup-php@v2"
+        with:
+          php-version: "${{ matrix.php-version }}"
+          coverage: "pcov"
+          ini-values: "zend.assertions=1"
+
+      - name: "Install dependencies with Composer"
+        uses: "ramsey/composer-install@v1"
+        with:
+          dependency-versions: "${{ matrix.dependencies }}"
+
+      - name: "Run PHPUnit"
+        run: "vendor/bin/phpunit"

.github/workflows/ci.yaml

@@ -1,3 +1,6 @@
 vendor/*
 /build
 composer.lock
+.phpunit.result.cache
+docker-compose.yml
+Dockerfile

.gitignore

@@ -7,8 +7,8 @@
 
 class ConfigHelper
 {
-    private $config;
-    private $projectDir;
+    private array $config;
+    private string $projectDir;
 
     public function __construct(string $projectDir, array $config)
     {
@@ -18,7 +18,7 @@ public function __construct(string $projectDir, array $config)
 
     public function buildProcess(InputStream $input, array $arguments = [], array $env = []): Process
     {
-        $process = new Process(
+        return new Process(
             array_merge(
                 [
                     $this->config['script']['callable'],
@@ -30,7 +30,5 @@ public function buildProcess(InputStream $input, array $arguments = [], array $e
             array_merge($this->config['env_vars'], $env),
             $input
         );
-
-        return $process;
     }
 }

.travis.yml

@@ -4,28 +4,24 @@
 
 use Symfony\Component\Config\Definition\Builder\TreeBuilder;
 use Symfony\Component\Config\Definition\ConfigurationInterface;
-use Symfony\Component\HttpKernel\Kernel;
 
 class Configuration implements ConfigurationInterface
 {
     /**
-     * {@inheritdoc}
+     * {@inheritDoc}
      */
-    public function getConfigTreeBuilder()
+    public function getConfigTreeBuilder(): TreeBuilder
     {
         $treeBuilder = new TreeBuilder('padam87_rasterize');
-        $rootNode = method_exists($treeBuilder, 'getRootNode')
-            ? $treeBuilder->getRootNode()
-            : $treeBuilder->root('padam87_rasterize');
 
-        $rootNode
+        $treeBuilder->getRootNode()
             ->children()
                 ->arrayNode('script')
                     ->addDefaultsIfNotSet()
                     ->children()
                         ->scalarNode('callable')->defaultValue('node')->end()
                         ->scalarNode('path')
-                            ->defaultValue($this->getAssetsDir() . DIRECTORY_SEPARATOR . 'rasterize.js')
+                            ->defaultValue('assets' . DIRECTORY_SEPARATOR . 'rasterize.js')
                             ->info('Relative to project dir')
                         ->end()
                     ->end()
@@ -54,14 +50,4 @@ public function getConfigTreeBuilder()
 
         return $treeBuilder;
     }
-
-    private function getAssetsDir()
-    {
-        switch (Kernel::MAJOR_VERSION) {
-            case 3:
-                return 'web';
-            default:
-                return 'assets';
-        }
-    }
 }

ConfigHelper.php

@@ -1,7 +1,4 @@
-[![Build Status](https://travis-ci.org/Padam87/RasterizeBundle.svg?branch=master)](https://travis-ci.org/Padam87/RasterizeBundle)
-[![Scrutinizer Code Quality](https://scrutinizer-ci.com/g/Padam87/RasterizeBundle/badges/quality-score.png?s=dbb22e5306bff73f2b3494261ddb8d6d7c6b35d7)](https://scrutinizer-ci.com/g/Padam87/RasterizeBundle/)
-[![Coverage Status](https://coveralls.io/repos/Padam87/RasterizeBundle/badge.png)](https://coveralls.io/r/Padam87/RasterizeBundle)
-[![SensioLabsInsight](https://insight.sensiolabs.com/projects/7e2ecf7a-40d7-41f4-9103-a99593c435d2/mini.png)](https://insight.sensiolabs.com/projects/7e2ecf7a-40d7-41f4-9103-a99593c435d2)
+![CI status](https://github.com/Padam87/RasterizerBundle/actions/workflows/ci.yaml/badge.svg)
 
 [![License](https://poser.pugx.org/padam87/rasterize-bundle/license.png)](https://packagist.org/packages/padam87/rasterize-bundle)
 [![Latest Stable Version](https://poser.pugx.org/padam87/rasterize-bundle/v/stable.png)](https://packagist.org/packages/padam87/rasterize-bundle)

DependencyInjection/Configuration.php

@@ -7,8 +7,8 @@
 
 class Rasterizer
 {
-    protected $configHelper;
-    protected $stopwatch;
+    protected ConfigHelper $configHelper;
+    protected ?Stopwatch $stopwatch;
 
     public function __construct(ConfigHelper $configHelper, Stopwatch $stopwatch = null)
     {

README.md

@@ -16,12 +16,12 @@ class ConfigHelperTest extends TestCase
      */
     protected $config;
 
-    protected function tearDown()
+    protected function tearDown(): void
     {
         m::close();
     }
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->config = [
             'script' => [

Rasterizer.php

@@ -16,12 +16,12 @@ class RasterizerTest extends TestCase
     private $stopwatch;
     private $process;
 
-    protected function tearDown()
+    protected function tearDown(): void
     {
         m::close();
     }
 
-    public function setUp()
+    public function setUp(): void
     {
         $this->configHelper = m::mock(ConfigHelper::class);
         $this->stopwatch = m::mock(Stopwatch::class);