Spatie\Newsletter\Test\NewsletterFacadeTest

Complexity

Total Complexity

2

Size/Duplication

Total Lines	21
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	1

Importance

Changes

0

2 Methods

Rating	Name	Duplication	Size	Complexity
A	should_instantiate_facade_with_null_driver()	0	5	1
A	should_instantiate_facade_with_mailchimp_driver()	9	9	1

<?php

namespace DansMaCulotte\Newsletter\Test;

use DansMaCulotte\Newsletter\Drivers\MailchimpDriver;
use DansMaCulotte\Newsletter\Drivers\MailjetDriver;
use DansMaCulotte\Newsletter\Drivers\NullDriver;
use DansMaCulotte\Newsletter\Newsletter;

class NewsletterFacadeTest extends TestCase

There is at least one abstract method in this class. Maybe declare it as abstract, or implement the remaining methods: artisan, be, call, seed

{
    /** @test */
    public function should_instantiate_facade_with_null_driver()
    {
        $newsletter = $this->app[Newsletter::class];
        $this->assertInstanceOf(NullDriver::class, $newsletter);
    }

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

This method seems to be duplicated in your project.

    {
        config()->set('newsletter.driver', 'mailchimp');
        config()->set('newsletter.mailchimp.apiKey', 'mailchimp-keytest');

        $newsletter = $this->app[Newsletter::class];
        $this->assertInstanceOf(Newsletter::class, $newsletter);
        $this->assertInstanceOf(MailchimpDriver::class, $newsletter->driver);
    }

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

This method seems to be duplicated in your project.

    {
        config()->set('newsletter.driver', 'mailjet');
        config()->set('newsletter.mailjet.key', 'mailjet-keytest');
        config()->set('newsletter.mailjet.secret', 'mailjet-secrettest');

        $newsletter = $this->app[Newsletter::class];
        $this->assertInstanceOf(Newsletter::class, $newsletter);
        $this->assertInstanceOf(MailjetDriver::class, $newsletter->driver);
    }

}