Test\Vehsamrak\ListCollection\AbstractListTest

Complexity

Total Complexity

17

Size/Duplication

Total Lines	124
Duplicated Lines	33.06 %

Coupling/Cohesion

Components	1
Dependencies	2

Importance

Changes

0

8 Methods

Rating	Name	Duplication	Size	Complexity
A	construct_validConstructorParameters_newListCreatedWithNoExceptions()	0	13	2
A	construct_invalidConstructorParameters_invalidTypeExceptionThrowed()	13	13	2
A	add_validParameters_noExceptions()	0	14	3
A	add_invalidParameters_noExceptions()	14	14	3
A	set_validParameters_noExceptions()	0	15	3
A	set_invalidParameters_noExceptions()	14	14	3
A	createList()	0	6	1
	getListClassName()	0	1	?

<?php
/** @noinspection PhpUnitUndefinedDataProviderInspection */
namespace Test\Vehsamrak\ListCollection;

use PHPUnit\Framework\TestCase;
use Vehsamrak\ListCollection\AbstractTypedList;
use Vehsamrak\ListCollection\Exceptions\InvalidTypeException;

abstract class AbstractListTest extends TestCase
{
    /**
     * @test
     * @dataProvider getValidParameters
     */
    public function construct_validConstructorParameters_newListCreatedWithNoExceptions(array $parameters): void
    {
        $integerList = null;
        $exception = null;

        try {
            $integerList = $this->createList($parameters);
        } catch (\Exception $exception) {

Consider adding a comment why this CATCH block is empty.

        }

        $this->assertInstanceOf($this->getListClassName(), $integerList);
        $this->assertNull($exception);
    }

    /**
     * @test
     * @dataProvider getInvalidParameters
     */
    public function construct_invalidConstructorParameters_invalidTypeExceptionThrowed(array $parameters): void

This method seems to be duplicated in your project.

    {
        $integerList = null;
        $exception = null;

        try {
            $integerList = $this->createList($parameters);
        } catch (\Exception $exception) {

Consider adding a comment why this CATCH block is empty.

        }

        $this->assertNull($integerList);
        $this->assertInstanceOf(InvalidTypeException::class, $exception);
    }

    /**
     * @test
     * @dataProvider getValidParameters
     */
    public function add_validParameters_noExceptions(array $parameters): void
    {
        $integerList = $this->createList();
        $exception = null;

        try {
            foreach ($parameters as $parameter) {
                $integerList->add($parameter);
            }
        } catch (\Exception $exception) {

Consider adding a comment why this CATCH block is empty.

        }

        $this->assertNull($exception);
    }

    /**
     * @test
     * @dataProvider getInvalidParameters
     */
    public function add_invalidParameters_noExceptions(array $parameters): void

This method seems to be duplicated in your project.

    {
        $integerList = $this->createList();
        $exception = null;

        try {
            foreach ($parameters as $parameter) {
                $integerList->add($parameter);
            }
        } catch (\Exception $exception) {

Consider adding a comment why this CATCH block is empty.

        }

        $this->assertInstanceOf(InvalidTypeException::class, $exception);
    }

    /**
     * @test
     * @dataProvider getValidParameters
     */
    public function set_validParameters_noExceptions(array $parameters): void
    {
        $integerList = $this->createList();
        $exception = null;

        try {
            foreach ($parameters as $parameter) {
                $integerList->set(0, $parameter);
            }
        } catch (\Exception $exception) {

Consider adding a comment why this CATCH block is empty.

        }

        $this->assertInstanceOf($this->getListClassName(), $integerList);
        $this->assertNull($exception);
    }

    /**
     * @test
     * @dataProvider getInvalidParameters
     */
    public function set_invalidParameters_noExceptions(array $parameters): void

This method seems to be duplicated in your project.

    {
        $integerList = $this->createList();
        $exception = null;

        try {
            foreach ($parameters as $parameter) {
                $integerList->set(0, $parameter);
            }
        } catch (\Exception $exception) {

Consider adding a comment why this CATCH block is empty.

        }

        $this->assertInstanceOf(InvalidTypeException::class, $exception);
    }

    protected function createList(array $parameters = []): AbstractTypedList
    {
        $className = $this->getListClassName();

        return new $className($parameters);
    }

    abstract protected function getListClassName(): string;
}