Merge pull request #204 from mmoreram/5

fixes #197

Author: mihaileu

.github/workflows/test.yaml

@@ -0,0 +1,69 @@
+name: Test
+
+on: [push, pull_request]
+
+jobs:
+    test:
+        name: PHP ${{ matrix.php-version }} + Symfony ${{ matrix.symfony-version }}
+        runs-on: ubuntu-18.04
+        strategy:
+            matrix:
+                php-version: ['7.4','8.0','8.1']
+                symfony-version: ['4.4','5.4']
+                coverage: ['none']
+                include:
+                    - php-version: '7.4'
+                      symfony-version: '4.4'
+                      coverage: xdebug
+                    - php: "8.0"
+                      php-version: "8.0"
+                      symfony-version: "6.0"
+                    - php: "8.1"
+                      php-version: "8.1"
+                      symfony-version: "6.0"
+        steps:
+            - name: Checkout
+              uses: actions/checkout@v2
+
+            - name: Setup PHP
+              uses: shivammathur/setup-php@v2
+              with:
+                  coverage: ${{ matrix.coverage }}
+                  ini-values: "memory_limit=-1"
+                  php-version: ${{ matrix.php-version }}
+                  extensions: gearman
+                  tools: composer,flex
+
+            - name: Validate composer.json
+              run: composer validate --no-check-lock
+
+            - name: Install Composer dependencies
+              uses: ramsey/composer-install@v1
+              with:
+                  composer-options: "--prefer-dist"
+              env:
+                  SYMFONY_REQUIRE: "${{ matrix.symfony-version }}.*"
+
+            - name: Setup problem matchers for PHP
+              run: echo "::add-matcher::${{ runner.tool_cache }}/php.json"
+
+            - name: Setup problem matchers for PHPUnit
+              run: echo "::add-matcher::${{ runner.tool_cache }}/phpunit.json"
+
+            - name: Run PHPUnit
+              if: matrix.coverage == 'none'
+              run: vendor/bin/phpunit
+
+            - name: Run PHPUnit with coverage
+              if: matrix.coverage != 'none'
+              run: vendor/bin/phpunit --coverage-clover=coverage.clover
+
+            - name: Run PHPStan
+              run: vendor/bin/phpstan analyse
+
+            - name: Upload Scrutinizer coverage
+              if: matrix.coverage != 'none'
+              continue-on-error: true
+              uses: sudo-bot/action-scrutinizer@latest
+              with:
+                  cli-args: "--format=php-clover coverage.clover"

.gitignore

@@ -5,3 +5,4 @@ composer.phar
 phpcs.log
 vendor
 phpunit.xml
+var

.travis.yml

@@ -1,43 +1,16 @@
 <?php
 
-/**
- * Gearman Bundle for Symfony2
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- *
- * Feel free to edit as you please, and have fun.
- *
- * @author Marc Morera <[email protected]>
- */
-
 namespace Mmoreram\GearmanBundle\Command\Abstracts;
 
 use Symfony\Component\Console\Command\Command;
 use Symfony\Component\HttpKernel\KernelInterface;
 
-/**
- * Class AbstractGearmanCommand
- *
- * @since 2.3.1
- */
+
 abstract class AbstractGearmanCommand extends Command
 {
-    /**
-     * @var KernelInterface
-     *
-     * Kernel
-     */
-    protected $kernel;
+    protected KernelInterface $kernel;
 
-    /**
-     * Set kernel
-     *
-     * @param KernelInterface $kernel Kernel
-     *
-     * @return AbstractGearmanCommand self Object
-     */
-    public function setKernel(KernelInterface $kernel)
+    public function setKernel(KernelInterface $kernel): self
     {
         $this->kernel = $kernel;
 

Command/Abstracts/AbstractGearmanCommand.php

@@ -1,16 +1,5 @@
 <?php
 
-/**
- * Gearman Bundle for Symfony2
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- *
- * Feel free to edit as you please, and have fun.
- *
- * @author Marc Morera <[email protected]>
- */
-
 namespace Mmoreram\GearmanBundle\Command;
 
 use Symfony\Component\Console\Input\InputInterface;
@@ -19,57 +8,27 @@
 use Mmoreram\GearmanBundle\Command\Abstracts\AbstractGearmanCommand;
 use Mmoreram\GearmanBundle\Service\GearmanCacheWrapper;
 
-/**
- * Clears all cache data
- *
- * @since 2.3.1
- */
 class GearmanCacheClearCommand extends AbstractGearmanCommand
 {
-    /**
-     * @var GearmanCacheWrapper
-     *
-     * GearmanCacheWrapper
-     */
-    protected $gearmanCacheWrapper;
+    protected GearmanCacheWrapper $gearmanCacheWrapper;
 
-    /**
-     * Set the GearmanCacheWrapper instance
-     *
-     * @param GearmanCacheWrapper $gearmanCacheWrapper GearmanCacheWrapper
-     *
-     * @return GearmanCacheWarmupCommand self Object
-     */
-    public function setGearmanCacheWrapper(GearmanCacheWrapper $gearmanCacheWrapper)
+    public function setGearmanCacheWrapper(GearmanCacheWrapper $gearmanCacheWrapper): self
     {
         $this->gearmanCacheWrapper = $gearmanCacheWrapper;
 
         return $this;
     }
 
-    /**
-     * Console Command configuration
-     */
     protected function configure()
     {
         $this
             ->setName('gearman:cache:clear')
-            ->setAliases(array(
-                'cache:gearman:clear'
-            ))
+            ->setAliases([
+                'cache:gearman:clear',
+            ])
             ->setDescription('Clears gearman cache data on current environment');
     }
 
-    /**
-     * Executes the current command.
-     *
-     * @param InputInterface  $input  An InputInterface instance
-     * @param OutputInterface $output An OutputInterface instance
-     *
-     * @return integer 0 if everything went fine, or an error code
-     *
-     * @throws \LogicException When this abstract class is not implemented
-     */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         if (
@@ -87,4 +46,4 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         return 0;
     }
-}
+}

Command/GearmanCacheClearCommand.php

@@ -1,16 +1,5 @@
 <?php
 
-/**
- * Gearman Bundle for Symfony2
- *
- * For the full copyright and license information, please view the LICENSE
- * file that was distributed with this source code.
- *
- * Feel free to edit as you please, and have fun.
- *
- * @author Marc Morera <[email protected]>
- */
-
 namespace Mmoreram\GearmanBundle\Command;
 
 use Symfony\Component\Console\Input\InputInterface;
@@ -19,60 +8,27 @@
 use Mmoreram\GearmanBundle\Command\Abstracts\AbstractGearmanCommand;
 use Mmoreram\GearmanBundle\Service\GearmanCacheWrapper;
 
-/**
- * Warms up all cache data
- *
- * @since 2.3.1
- */
 class GearmanCacheWarmupCommand extends AbstractGearmanCommand
 {
-    /**
-     * @var GearmanCacheWrapper
-     *
-     * GearmanCacheWrapper
-     */
-    protected $gearmanCacheWrapper;
+    protected GearmanCacheWrapper $gearmanCacheWrapper;
 
-    /**
-     * Set the GearmanCacheWrapper instance
-     *
-     * @param GearmanCacheWrapper $gearmanCacheWrapper GearmanCacheWrapper
-     *
-     * @return GearmanCacheWarmupCommand self Object
-     */
-    public function setGearmanCacheWrapper(GearmanCacheWrapper $gearmanCacheWrapper)
+    public function setGearmanCacheWrapper(GearmanCacheWrapper $gearmanCacheWrapper): self
     {
         $this->gearmanCacheWrapper = $gearmanCacheWrapper;
 
         return $this;
     }
 
-    /**
-     * Set the kernel environment
-
-    /**
-     * Console Command configuration
-     */
     protected function configure()
     {
         $this
             ->setName('gearman:cache:warmup')
-            ->setAliases(array(
-                'cache:gearman:warmup'
-            ))
+            ->setAliases([
+                'cache:gearman:warmup',
+            ])
             ->setDescription('Warms up gearman cache data');
     }
 
-    /**
-     * Executes the current command.
-     *
-     * @param InputInterface  $input  An InputInterface instance
-     * @param OutputInterface $output An OutputInterface instance
-     *
-     * @return integer 0 if everything went fine, or an error code
-     *
-     * @throws \LogicException When this abstract class is not implemented
-     */
     protected function execute(InputInterface $input, OutputInterface $output)
     {
         if (
@@ -91,4 +47,4 @@ protected function execute(InputInterface $input, OutputInterface $output)
 
         return 0;
     }
-}
+}