feat: update dependencies, add linting, fix codebase, cleanup (#11)

* chore(deps): update dependencies

* feat(tests): add matrix testing, problem matcher

* feat(lint): add php-cs-fixer, fix codebase

.editorconfig

@@ -0,0 +1,16 @@
+# EditorConfig is awesome: https://editorconfig.org
+
+# top-most EditorConfig file
+root = true
+
+[*]
+end_of_line = lf
+insert_final_newline = true
+charset = utf-8
+indent_style = space
+indent_size = 4
+max_line_length = 100
+
+[*.yml]
+indent_size = 2
+

.github/workflows/auto-approve.yml

@@ -0,0 +1,12 @@
+name: Auto Approve Dependabot
+
+on: pull_request_target
+
+jobs:
+  auto-approve:
+    runs-on: ubuntu-latest
+    steps:
+    - uses: hmarr/auto-approve-action@v2.0.0
+      if: github.actor == 'dependabot[bot]' || github.actor == 'dependabot-preview[bot]'
+      with:
+        github-token: "${{ secrets.GITHUB_TOKEN }}"

.github/workflows/phpunit.yml

@@ -0,0 +1,45 @@
+name: PHPUnit
+
+on: [push]
+
+jobs:
+  build-test:
+    runs-on: ubuntu-latest
+
+    strategy:
+      matrix:
+        php_version:
+          - 7.4
+          - 8.0 
+          - 8.1
+          - 8.2
+          - 8.3
+          - latest
+
+    steps:
+      - uses: actions/checkout@v4
+
+      - uses: shivammathur/setup-php@v2
+        with:
+          php-version: ${{ matrix.php_version }}
+          tools: composer, phpunit
+
+      - name: Get composer cache directory
+        id: composer-cache
+        run: echo "dir=$(composer config cache-files-dir)" >> $GITHUB_OUTPUT
+
+      - name: Cache Composer dependencies
+        uses: actions/cache@v4
+        with:
+          path: ${{ steps.composer-cache.outputs.dir }}
+          key: ${{ runner.os }}-${{ matrix.php_version }}-${{ hashFiles('**/composer.json') }}
+
+      - name: Install Dependencies
+        run: composer install -q --no-ansi --no-interaction --no-scripts --no-progress --prefer-dist
+
+      - name: Configure matchers
+        uses: mheap/phpunit-matcher-action@v1
+
+      - name: Run tests
+        run: composer test-ci
+ 

.gitignore

@@ -1 +1,2 @@
 vendor
+composer.lock

.php-cs-fixer.dist.php

@@ -0,0 +1,15 @@
+<?php
+
+$finder = (new PhpCsFixer\Finder())
+    ->in(__DIR__);
+
+return (new PhpCsFixer\Config())
+    ->setRules([
+        '@PhpCsFixer' => true,
+        '@PHP74Migration' => true,
+        '@PSR12' => true,
+        'strict_param' => true,
+        'array_syntax' => ['syntax' => 'short'],
+    ])
+    ->setFinder($finder)
+;

.travis.yml

@@ -1,20 +0,0 @@
-language: php
-php:
-  - 7.0
-  - 7.1
-  - nightly
-matrix:
-  allow_failures:
-    - php: nightly
-  fast_finish: true
-cache:
-  directories:
-    - $HOME/.composer/cache/files
-before_script:
-  - phpenv global "$TRAVIS_PHP_VERSION"
-  - composer config extra.platform.php $TRAVIS_PHP_VERSION
-install:
-  - flags="--ansi --prefer-dist --no-interaction --optimize-autoloader --no-suggest --no-progress"
-  - travis_wait 30 composer install $flags
-script:
-  - ./vendor/bin/phpunit -c phpunit.xml

README.md

@@ -1,22 +1,19 @@
-# Laravel Commands by [SuperHelio](link-author)
+# Laravel Commands by [SuperHelio][link-author]
 
 [![Latest Version on Packagist][ico-version]][link-packagist]
 [![Software License][ico-license]](LICENSE.md)
-[![Build Status][ico-travis]][link-travis]
-[![Coverage Status][ico-scrutinizer]][link-scrutinizer]
-[![Quality Score][ico-code-quality]][link-code-quality]
 [![Total Downloads][ico-downloads]][link-downloads]
 
 This is a collection of Laravel Artisan commands created to help everyone
 in their development work. We try to keep these as useful as possible.
 
-This package requires PHP 7.0 or later. `composer.lock` has been generated with PHP v7.0.23.
+This package requires PHP 7.3 or later. The `composer.lock` file has been generated with PHP v7.4.13.
 
 ## Install
 
 ### Step 1: Install Through Composer
 
-``` bash
+```bash
 $ composer require superhelio/commands --dev
 ```
 
@@ -27,7 +24,7 @@ You'll only want to use these generators for local development, so you don't wan
 ```php
 public function register()
 {
-    if ($this->app->environment() == 'local') {
+    if ($this->app->environment() === 'local') {
         $this->app->register('SuperHelio\Commands\ServiceProvider');
     }
 }
@@ -35,12 +32,12 @@ public function register()
 
 ## Usage
 
-- *superhelio:gozer*
-    - Force delete database tables that have your table prefix
-    - `php artisan superhelio:gozer`
-- *superhelio:reload*
-    - Reset database, migrate and seed
-    - `php artisan superhelio:reload`
+- _superhelio:gozer_
+  - Force delete database tables that have your table prefix
+  - `php artisan superhelio:gozer`
+- _superhelio:reload_
+  - Reset database, migrate and seed
+  - `php artisan superhelio:reload`
 
 ## Change log
 
@@ -48,7 +45,7 @@ Please see [CHANGELOG](CHANGELOG.md) for more information what has changed recen
 
 ## Testing
 
-``` bash
+```bash
 $ composer test
 ```
 
@@ -67,15 +64,8 @@ The MIT License (MIT). Please see [License File](LICENSE.md) for more informatio
 
 [ico-version]: https://img.shields.io/packagist/v/superhelio/commands.svg?style=flat-square
 [ico-license]: https://img.shields.io/badge/license-MIT-brightgreen.svg?style=flat-square
-[ico-travis]: https://img.shields.io/travis/superhelio/commands/master.svg?style=flat-square
-[ico-scrutinizer]: https://img.shields.io/scrutinizer/coverage/g/superhelio/commands.svg?style=flat-square
-[ico-code-quality]: https://img.shields.io/scrutinizer/g/superhelio/commands.svg?style=flat-square
 [ico-downloads]: https://img.shields.io/packagist/dt/superhelio/commands.svg?style=flat-square
-
 [link-packagist]: https://packagist.org/packages/superhelio/commands
-[link-travis]: https://travis-ci.org/superhelio/commands
-[link-scrutinizer]: https://scrutinizer-ci.com/g/superhelio/commands/code-structure
-[link-code-quality]: https://scrutinizer-ci.com/g/superhelio/commands
 [link-downloads]: https://packagist.org/packages/superhelio/commands
 [link-author]: https://github.com/superhelio
 [link-contributors]: https://github.com/superhelio/commands/graphs/contributors

composer.json

@@ -17,13 +17,15 @@
         }
     ],
     "require": {
-        "php" : ">=7.0",
-        "illuminate/support": "^5.3|^5.4|^5.5",
-        "doctrine/dbal": "^2.5|^2.6"
+        "php": "^7.4|^8",
+        "illuminate/support": "^5.3|^6|^7|^8|^9|^10|^11",
+        "doctrine/dbal": "^2.5|^2.6|^3"
     },
     "require-dev": {
-        "phpunit/phpunit" : "^5.0",
-        "orchestra/testbench": "~3.0"
+        "roave/security-advisories": "dev-master",
+        "phpunit/phpunit": "^8.0",
+        "orchestra/testbench": "^3.3|^4|^5|^6|^7|^8",
+        "friendsofphp/php-cs-fixer": "^3.9"
     },
     "autoload": {
         "psr-4": {
@@ -36,7 +38,9 @@
         }
     },
     "scripts": {
-        "test": "phpunit"
+        "lint": "PHP_CS_FIXER_IGNORE_ENV=1 php-cs-fixer fix --diff --allow-risky=yes",
+        "test": "phpunit",
+        "test-ci": "phpunit --teamcity"
     },
     "extra": {
         "branch-alias": {

phpcs.xml

@@ -0,0 +1,19 @@
+<?xml version="1.0"?>
+<ruleset name="Project"
+         xmlns:xsi="http://www.w3.org/2001/XMLSchema-instance"
+         xsi:noNamespaceSchemaLocation="https://raw.githubusercontent.com/squizlabs/PHP_CodeSniffer/master/phpcs.xsd">
+    <description>Project PHP Code Style Rules</description>
+
+    <file>src</file>
+    <file>tests</file>
+
+    <arg name="basepath" value="."/>
+    <arg name="colors"/>
+    <arg name="parallel" value="75"/>
+    <arg value="np"/>
+
+    <exclude-pattern>*/.git/*</exclude-pattern>
+    <exclude-pattern>*/vendor/*</exclude-pattern>
+
+    <rule ref="PSR2" />
+</ruleset>

phpunit.xml

@@ -7,8 +7,7 @@
          convertNoticesToExceptions="true"
          convertWarningsToExceptions="true"
          processIsolation="false"
-         stopOnFailure="false"
-         syntaxCheck="false">
+         stopOnFailure="false">
 
     <testsuites>
         <testsuite name="Superhelio\Commands Test Suite">

src/Commands/Gozer.php

@@ -5,6 +5,7 @@ namespace Superhelio\Commands\Commands;
 use Illuminate\Console\Command;
 use Illuminate\Support\Facades\DB;
 use Illuminate\Support\Facades\Schema;
+use Illuminate\Support\Str;
 
 class Gozer extends Command
 {
@@ -75,16 +76,14 @@ class Gozer extends Command
              */
             $tables = $this->getFilteredTables($tables);
 
-            /**
-             * Check that we got at least one table, bail out if not
-             */
+            // Check that we got at least one table, bail out if not
             if ($tables->count() < 1) {
                 $this->info('There are no tables, only Zuul.');
 
                 return true;
             }
 
-            /**
+            /*
              * Bid your farewells to these tables.
              * Last look and confirmation.
              */
@@ -94,27 +93,22 @@ class Gozer extends Command
             ));
             $this->line('');
 
-            /**
-             * Last confirmation before dropping tables
-             */
+            // Last confirmation before dropping tables
             if ($this->confirm('Really delete those tables?')) {
-
                 /** Fancy pants progress bar to see your tables get destroyed */
                 $bar = $this->output->createProgressBar($tables->count());
 
                 Schema::disableForeignKeyConstraints();
                 $tables->each(function ($table) use ($bar, $connection) {
-
-                    /** Drop the table */
+                    // Drop the table
                     $connection->dropTable($table);
 
-                    /** Advance our progress bar */
+                    // Advance our progress bar
                     $bar->advance();
-
                 });
                 Schema::enableForeignKeyConstraints();
 
-                /** Progress bar is now finished */
+                // Progress bar is now finished
                 $bar->finish();
             }
 
@@ -133,20 +127,16 @@ class Gozer extends Command
     public function getConnection()
     {
         try {
-            /** @var \Doctrine\DBAL\Schema\AbstractSchemaManager $connection */
-            $connection = app('db')->connection()->getDoctrineSchemaManager();
+            // @var \Doctrine\DBAL\Schema\AbstractSchemaManager $connection
+            return app('db')->connection()->getDoctrineSchemaManager();
         } catch (\Exception $e) {
             $this->error($e->getMessage());
 
             return false;
         }
-
-        return $connection;
     }
 
     /**
-     * @param \Doctrine\DBAL\Schema\AbstractSchemaManager $connection
-     *
      * @return array|bool
      */
     public function getTables(\Doctrine\DBAL\Schema\AbstractSchemaManager $connection)
@@ -165,11 +155,11 @@ class Gozer extends Command
 
     public function getDatabasePrefix()
     {
-        return trim(DB::getTablePrefix());
+        return trim(DB::connection()->getTablePrefix());
     }
 
     /**
-     * This is mainly for testing purposes
+     * This is mainly for testing purposes.
      *
      * @param string $prefix
      */
@@ -179,7 +169,7 @@ class Gozer extends Command
     }
 
     /**
-     * @param array $tables
+     * @param array|\Illuminate\Support\Collection $tables
      *
      * @return \Illuminate\Support\Collection
      */
@@ -188,7 +178,7 @@ class Gozer extends Command
         $prefix = $this->dbPrefix;
 
         return collect($tables)->reject(function ($table) use ($prefix) {
-            return !starts_with($table, $prefix);
+            return !Str::startsWith($table, $prefix);
         });
     }
 }

src/Commands/Reload.php

@@ -33,7 +33,7 @@ class Reload extends Command
                 [
                     '--no-interaction' => true,
                     '--env' => 'development',
-                    '--verbose' => 3
+                    '--verbose' => 3,
                 ]
             );
             $this->call(
@@ -42,11 +42,13 @@ class Reload extends Command
                     '--seed' => true,
                     '--no-interaction' => true,
                     '--env' => 'development',
-                    '--verbose' => 3
+                    '--verbose' => 3,
                 ]
             );
 
             return true;
         }
+
+        return false;
     }
 }

src/Facade.php

@@ -1,4 +1,5 @@
 <?php
+
 namespace Superhelio\Commands;
 
 class Facade extends \Illuminate\Support\Facades\Facade

src/ServiceProvider.php

@@ -5,9 +5,8 @@ namespace Superhelio\Commands;
 use Illuminate\Support\ServiceProvider as BaseServiceProvider;
 
 /**
- * Class PackageServiceProvider
+ * Class PackageServiceProvider.
  *
- * @package Superhelio\Commands
  * @see http://laravel.com/docs/master/packages#service-providers
  * @see http://laravel.com/docs/master/providers
  */
@@ -17,6 +16,7 @@ class ServiceProvider extends BaseServiceProvider
      * Indicates if loading of the provider is deferred.
      *
      * @see http://laravel.com/docs/master/providers#deferred-providers
+     *
      * @var bool
      */
     protected $defer = false;
@@ -25,7 +25,6 @@ class ServiceProvider extends BaseServiceProvider
      * Register the service provider.
      *
      * @see http://laravel.com/docs/master/providers#the-register-method
-     * @return void
      */
     public function register()
     {
@@ -36,7 +35,7 @@ class ServiceProvider extends BaseServiceProvider
     private function registerReloader()
     {
         $this->app->singleton('command.superhelio.reload', function ($app) {
-            return $app['Superhelio\Commands\Commands\Reload'];
+            return $app[\Superhelio\Commands\Commands\Reload::class];
         });
         $this->commands('command.superhelio.reload');
     }
@@ -44,7 +43,7 @@ class ServiceProvider extends BaseServiceProvider
     private function registerGozer()
     {
         $this->app->singleton('command.superhelio.gozer', function ($app) {
-            return $app['Superhelio\Commands\Commands\Gozer'];
+            return $app[\Superhelio\Commands\Commands\Gozer::class];
         });
         $this->commands('command.superhelio.gozer');
     }

tests/GozerTest.php

@@ -1,17 +1,22 @@
 <?php
-namespace Superhelio\Commands;
+
+namespace Superhelio\Commands\Tests;
 
 use ReflectionClass;
 use Superhelio\Commands\Commands\Gozer;
 use Illuminate\Support\Facades\Hash;
 use Illuminate\Support\Facades\DB;
 
+/**
+ * @internal
+ * @coversNothing
+ */
 class GozerTest extends \Orchestra\Testbench\TestCase
 {
     /**
      * Setup the test environment.
      */
-    public function setUp()
+    public function setUp(): void
     {
         parent::setUp();
         $this->artisan('migrate', ['--database' => 'testbench']);
@@ -20,9 +25,7 @@ class GozerTest extends \Orchestra\Testbench\TestCase
     /**
      * Define environment setup.
      *
-     * @param  \Illuminate\Foundation\Application $app
-     *
-     * @return void
+     * @param \Illuminate\Foundation\Application $app
      */
     protected function getEnvironmentSetUp($app)
     {
@@ -42,90 +45,90 @@ class GozerTest extends \Orchestra\Testbench\TestCase
      * In a normal app environment these would be added to
      * the 'providers' array in the config/app.php file.
      *
-     * @param  \Illuminate\Foundation\Application $app
+     * @param \Illuminate\Foundation\Application $app
      *
      * @return array
      */
     protected function getPackageProviders($app)
     {
         return [
-            '\Superhelio\Commands\Tests\Stubs\ServiceProvider',
-            '\Superhelio\Commands\ServiceProvider'
+            \Superhelio\Commands\Tests\Stubs\ServiceProvider::class,
+            \Superhelio\Commands\ServiceProvider::class,
         ];
     }
 
-    public function test_database_is_there_and_functions()
+    public function testDatabaseIsThereAndFunctions()
     {
         DB::table('users')->insert([
             'name' => 'User name',
             'email' => 'hello@gozer.dev',
-            'password' => bcrypt('123')
+            'password' => bcrypt('123'),
         ]);
 
         $users = DB::table('users')->where('id', '=', 1)->first();
-        $this->assertEquals('hello@gozer.dev', $users->email);
-        $this->assertEquals('User name', $users->name);
-        $this->assertTrue(Hash::check('123', $users->password));
+        self::assertEquals('hello@gozer.dev', $users->email);
+        self::assertEquals('User name', $users->name);
+        self::assertTrue(Hash::check('123', $users->password));
     }
 
-    public function test_dbal_is_installed()
+    public function testDbalIsInstalled()
     {
-        $this->assertTrue(class_exists('\\Doctrine\\DBAL\\Schema\\Schema'));
+        self::assertTrue(class_exists('\\Doctrine\\DBAL\\Schema\\Schema'));
     }
 
-    public function test_gozer_is_installed()
+    public function testGozerIsInstalled()
     {
-        $this->assertTrue(class_exists('\\Superhelio\\Commands\\Commands\\Gozer'));
+        self::assertTrue(class_exists('\\Superhelio\\Commands\\Commands\\Gozer'));
     }
 
-    public function test_gozer_has_required_methods_and_properties()
+    public function testGozerHasRequiredMethodsAndProperties()
     {
         $gozer = new ReflectionClass('\\Superhelio\\Commands\\Commands\\Gozer');
-        $this->assertTrue($gozer->hasMethod('handle'));
-        $this->assertTrue($gozer->hasProperty('signature'));
-        $this->assertTrue($gozer->hasProperty('description'));
-        $this->assertTrue($gozer->hasProperty('dbPrefix'));
+        self::assertTrue($gozer->hasMethod('handle'));
+        self::assertTrue($gozer->hasProperty('signature'));
+        self::assertTrue($gozer->hasProperty('description'));
+        self::assertTrue($gozer->hasProperty('dbPrefix'));
     }
 
-    public function test_gozer_finds_database_prefix()
+    public function testGozerFindsDatabasePrefix()
     {
         $gozer = new Gozer();
 
-        $this->assertEquals('gozerTest__', $gozer->getDatabasePrefix());
+        self::assertEquals('gozerTest__', $gozer->getDatabasePrefix());
     }
 
-    public function test_gozer_finds_users_table()
+    public function testGozerFindsUsersTable()
     {
         $gozer = new Gozer();
 
         $connection = $gozer->getConnection();
 
         $tables = $gozer->getTables($connection);
-        $this->assertTrue(in_array('gozerTest__users', $tables, false));
+        self::assertContains('gozerTest__users', $tables);
 
         $gozer->setDatabasePrefix('gozerTest__');
         $filteredTables = $gozer->getFilteredTables($tables);
-        $this->assertTrue(is_a($filteredTables, \Illuminate\Support\Collection::class));
-        $this->assertTrue(in_array('gozerTest__users', $filteredTables->toArray(), false));
+        self::assertTrue(is_a($filteredTables, \Illuminate\Support\Collection::class));
+        self::assertContains('gozerTest__users', $filteredTables->toArray());
     }
 
-    public function test_gozer_table_filtering_works()
+    public function testGozerTableFilteringWorks()
     {
         $gozer = new Gozer();
-        $tables = array(
+        $tables = [
             'gozerTest__users',
             'gozerTest__migrations',
             'this_should_be_filtered',
-            'filter_me_too'
-        );
+            'filter_me_too',
+        ];
 
         $gozer->setDatabasePrefix('gozerTest__');
         $filtered = $gozer->getFilteredTables($tables);
         $array = $filtered->toArray();
 
-        $this->assertFalse(in_array('this_should_be_filtered', $array, false));
-        $this->assertFalse(in_array('filter_me_too', $array, false));
-        $this->assertTrue(in_array('gozerTest__users', $array, false));
-        $this->assertTrue(in_array('gozerTest__migrations', $array, false));
+        self::assertNotContains('this_should_be_filtered', $array);
+        self::assertNotContains('filter_me_too', $array);
+        self::assertContains('gozerTest__users', $array);
+        self::assertContains('gozerTest__migrations', $array);
     }
 }

tests/ReloadTest.php

@@ -1,14 +1,28 @@
 <?php
-namespace Superhelio\Commands;
 
-use Superhelio\Commands\Commands\Reload;
-use Illuminate\Foundation\Testing\DatabaseMigrations;
-use Illuminate\Foundation\Testing\DatabaseTransactions;
+namespace Superhelio\Commands\Tests;
 
-class ReloadTest extends \PHPUnit_Framework_TestCase
+/**
+ * @internal
+ * @coversNothing
+ */
+class ReloadTest extends \Orchestra\Testbench\TestCase
 {
     public function testReloadTest()
     {
-        $this->assertTrue(true);
+        self::assertTrue(true);
+    }
+
+    public function testReloadIsInstalled()
+    {
+        self::assertTrue(class_exists('\\Superhelio\\Commands\\Commands\\Reload'));
+    }
+
+    public function testReloadHasRequiredMethodsAndProperties()
+    {
+        $reload = new \ReflectionClass('\\Superhelio\\Commands\\Commands\\Reload');
+        self::assertTrue($reload->hasMethod('handle'));
+        self::assertTrue($reload->hasProperty('signature'));
+        self::assertTrue($reload->hasProperty('description'));
     }
 }

tests/Stubs/ServiceProvider.php

@@ -1,10 +1,11 @@
 <?php
+
 namespace Superhelio\Commands\Tests\Stubs;
 
 class ServiceProvider extends \Illuminate\Support\ServiceProvider
 {
     public function boot()
     {
-        $this->loadMigrationsFrom(dirname(__DIR__) . '/migrations');
+        $this->loadMigrationsFrom(dirname(__DIR__).'/migrations');
     }
 }

tests/migrations/2014_10_12_000000_create_users_table.php

@@ -8,8 +8,6 @@ class CreateUsersTable extends Migration
 {
     /**
      * Run the migrations.
-     *
-     * @return void
      */
     public function up()
     {
@@ -25,8 +23,6 @@ class CreateUsersTable extends Migration
 
     /**
      * Reverse the migrations.
-     *
-     * @return void
      */
     public function down()
     {