Spatie\MailableTest\MailableFactory

Complexity

Total Complexity

4

Size/Duplication

Total Lines	47
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes

0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	__construct()	0	4	1
A	getInstance()	0	10	1
A	getArguments()	0	16	1
A	setRecipient()	0	8	1

<?php

namespace Spatie\MailableTest;

use Illuminate\Mail\Mailable;
use ReflectionClass;
use ReflectionParameter;

class MailableFactory
{
    /** @var \Spatie\MailableTest\FakerArgumentValueProvider */
    protected $argumentValueProvider;

    public function __construct(ArgumentValueProvider $argumentValueProvider)
    {
        $this->argumentValueProvider = $argumentValueProvider;

$argumentValueProvider is of type object<Spatie\MailableTest\ArgumentValueProvider>, but the property $argumentValueProvider was declared to be of type object<Spatie\MailableTest\FakerArgumentValueProvider>. Are you sure that you always receive this specific sub-class here, or does it make sense to add an instanceof check?

    }

    public function getInstance(string $mailableClass, string $toEmail, $defaultValues): Mailable
    {
        $argumentValues = $this->getArguments($mailableClass, $defaultValues);

        $mailable = new $mailableClass(...$argumentValues);

        $mailable = $this->setRecipient($mailable, $toEmail);

        return $mailable;
    }

    public function getArguments(string $mailableClass, array $defaultValues)
    {
        $parameters = (new ReflectionClass($mailableClass))
            ->getConstructor()
            ->getParameters();

        return collect($parameters)
            ->map(function (ReflectionParameter $reflectionParameter) use ($mailableClass, $defaultValues) {
                return $this->argumentValueProvider->getValue(
                    $mailableClass,
                    $reflectionParameter->getName(),
                    $reflectionParameter->getType()->getName(),
                    $defaultValues[$reflectionParameter->getName()] ?? null
                );
            });
    }

    protected function setRecipient(Mailable $mailable, string $email): Mailable
    {
        $mailable->to($email);
        $mailable->cc([]);
        $mailable->bcc([]);

        return $mailable;
    }
}