Mnabialek\LaravelQuickMigrations\QuickDatabaseMigrations

Complexity

Total Complexity

3

Size/Duplication

Total Lines	41
Duplicated Lines	0 %

Importance

Changes	2
Bugs	0	Features	0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	useDefaultWay()	0	3	1
A	runDatabaseMigrations()	0	20	2

<?php

namespace Mnabialek\LaravelQuickMigrations;

use Illuminate\Contracts\Console\Kernel;
use Illuminate\Foundation\Testing\DatabaseMigrations;
use Illuminate\Foundation\Testing\RefreshDatabaseState;

trait QuickDatabaseMigrations
{
    use DatabaseMigrations {
        runDatabaseMigrations as runDefaultDatabaseMigrations;
    }

    /**
     * Define hooks to migrate the database before and after each test.
     *
     * @return void
     */
    public function runDatabaseMigrations()
    {
        // if disabled we just run parent method and that's it
        if (! config('quick_migrations.enabled')) {
            $this->useDefaultWay();

            return;
        }

        // otherwise we just run single migration that loads database dump
        $this->artisan('migrate:fresh', [

It seems like artisan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            '--path' => realpath(__DIR__ . '/../migrations'),
            '--realpath' => 1,
        ]);

        $this->app[Kernel::class]->setArtisan(null);

        // here we don't care about rollback (to make it faster)
        $this->beforeApplicationDestroyed(function () {

It seems like beforeApplicationDestroyed() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            RefreshDatabaseState::$migrated = false;
        });
    }

    /**
     * Use default handling way (same as DatabaseMigrations).
     * @codeCoverageIgnore
     */
    protected function useDefaultWay()
    {
        $this->runDefaultDatabaseMigrations();
    }
}