Tests\utils\DbUtils

Complexity

Total Complexity

5

Size/Duplication

Total Lines	81
Duplicated Lines	44.44 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

3 Methods

Rating	Name	Duplication	Size	Complexity
A	assertSeeIsNotSoftDeletedInDatabase()	18	18	2
A	assertIsSoftDeletedInDatabase()	18	18	2
A	assertDatabaseHasOnceAndReturnFirst()	0	11	1

<?php
/*
 * laravel-money-manager-ex
 *
 * This File belongs to to Project laravel-money-manager-ex
 *
 * @author Oliver Kaufmann <okaufmann91@gmail.com>
 * @version 1.0
 */

namespace Tests\utils;

use App\Models\Model;

trait DbUtils
{
    /**
     * Assert that a given where condition does not matches a soft deleted record
     * From: http://stackoverflow.com/questions/33117746/laravel-unit-testing-how-to-seeindatabase-soft-deleted-row.
     *
     * @param string $table
     * @param array  $data
     * @param string $connection
     *
     * @return $this
     */
    protected function assertSeeIsNotSoftDeletedInDatabase($table, array $data, $connection = null)

This method seems to be duplicated in your project.

    {
        $database = $this->app->make('db');

The property app does not exist. Did you maybe forget to declare it?

        $connection = $connection ?: $database->getDefaultConnection();

        $count = $database->connection($connection)
            ->table($table)
            ->where($data)
            ->whereNull('deleted_at')
            ->count();

        $this->assertGreaterThan(0, $count, sprintf(

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

            'Found unexpected records in database table [%s] that matched attributes [%s].', $table, json_encode($data)
        ));

        return $this;
    }

    /**
     * Assert that a given where condition matches a soft deleted record
     * From: http://stackoverflow.com/questions/33117746/laravel-unit-testing-how-to-seeindatabase-soft-deleted-row.
     *
     * @param string $table
     * @param array  $data
     * @param string $connection
     *
     * @return $this
     */
    protected function assertIsSoftDeletedInDatabase($table, array $data, $connection = null)

This method seems to be duplicated in your project.

    {
        $database = $this->app->make('db');

        $connection = $connection ?: $database->getDefaultConnection();

        $count = $database->connection($connection)
            ->table($table)
            ->where($data)
            ->whereNotNull('deleted_at')
            ->count();

        $this->assertGreaterThan(0, $count, sprintf(

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

            'Found unexpected records in database table [%s] that matched attributes [%s].', $table, json_encode($data)
        ));

        return $this;
    }

    /**
     * Assert that a given where condition exists in the database and return the first occurrence.
     *
     * @param string $table
     * @param array  $data
     * @param string $connection
     *
     * @return Model
     */
    protected function assertDatabaseHasOnceAndReturnFirst($table, array $data, $connection = null)
    {
        $constraint = new HasInDatabaseOnce($this->getConnection($connection), $data);

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

        $this->assertThat(

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

            $table, $constraint
        );

        $entry = $constraint->getEntry();

        return $entry;
    }
}