Spatie\Newsletter\Test\NewsletterListCollectionTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	81
Duplicated Lines	19.75 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	setUp()	0	15	1
A	it_can_find_a_list_by_its_name()	8	8	1
A	it_will_use_the_default_list_when_not_specifing_a_listname()	8	8	1
A	it_will_throw_an_exception_when_using_a_default_list_that_does_not_exist()	0	16	1
A	it_will_throw_an_exception_when_there_are_no_lists_defined()	0	13	1
A	it_will_throw_an_exception_when_trying_to_find_a_list_that_does_not_exist()	0	6	1

<?php

namespace DansMaCulotte\Newsletter\Test;

use PHPUnit\Framework\TestCase;

This use statement conflicts with another class in this namespace, DansMaCulotte\Newsletter\Test\TestCase.

use DansMaCulotte\Newsletter\NewsletterList;
use DansMaCulotte\Newsletter\NewsletterListCollection;
use DansMaCulotte\Newsletter\Exceptions\InvalidNewsletterList;

class NewsletterListCollectionTest extends TestCase
{
    protected $newsletterListCollection;

    public function setUp() : void
    {
        parent::setUp();

        $this->newsletterListCollection = NewsletterListCollection::createFromConfig(
            [
                'lists' => [
                    'list1' => ['id' => 1],
                    'list2' => ['id' => 2],
                    'list3' => ['id' => 3],
                ],
                'defaultList' => 'list3',
            ]
        );
    }

    /** @test */
    public function it_can_find_a_list_by_its_name()

This method seems to be duplicated in your project.

    {
        $list = $this->newsletterListCollection->findByName('list2');

        $this->assertInstanceOf(NewsletterList::class, $list);

        $this->assertEquals(2, $list->getId());
    }

    /** @test */
    public function it_will_use_the_default_list_when_not_specifing_a_listname()

This method seems to be duplicated in your project.

    {
        $list = $this->newsletterListCollection->findByName('');

        $this->assertInstanceOf(NewsletterList::class, $list);

        $this->assertEquals(3, $list->getId());
    }

    /** @test */
    public function it_will_throw_an_exception_when_using_a_default_list_that_does_not_exist()
    {
        $newsletterListCollection = NewsletterListCollection::createFromConfig(
            [
                'lists' => [
                    'list1' => ['id' => 'list1'],
                ],

                'defaultList' => 'list2',
            ]
        );

        $this->expectException(InvalidNewsletterList::class);

        $newsletterListCollection->findByName('');
    }

    /** @test */
    public function it_will_throw_an_exception_when_there_are_no_lists_defined()
    {
        $this->expectException(InvalidNewsletterList::class);

        NewsletterListCollection::createFromConfig(
            [
                'lists' => [
                ],

                'defaultList' => 'list1',
            ]
        );
    }

    /** @test */
    public function it_will_throw_an_exception_when_trying_to_find_a_list_that_does_not_exist()
    {
        $this->expectException(InvalidNewsletterList::class);

        $this->newsletterListCollection->findByName('blabla');
    }
}