SwiftMailerSenderTest::testSend() - Code Metrics - Inspection of "fix examples, added tests for sender and repositor..." - sfcod/email-engine - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 80ef2e...65d1c9 )

unknown

created 2018-06-18 14:40 UTC

SwiftMailerSenderTest::testSend() A

↳ Parent: SwiftMailerSenderTest

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	7
nc	1
nop	0

<?php

namespace SfCod\EmailEngineBundle\Tests\Sender;

use PHPUnit\Framework\TestCase;
use SfCod\EmailEngineBundle\Example\TestEmailOptions;
use SfCod\EmailEngineBundle\Example\TestEmailTemplate;
use SfCod\EmailEngineBundle\Mailer\Mailer;
use SfCod\EmailEngineBundle\Repository\TwigFileRepository;
use SfCod\EmailEngineBundle\Sender\SwiftMailerSender;
use SfCod\EmailEngineBundle\Template\Params\ParameterResolver;
use SfCod\EmailEngineBundle\Tests\Data\LoadTrait;

/**
 * Class SwiftMailerSenderTest
 *
 * @author Virchenko Maksim <[email protected]>
 *
 * @package SfCod\EmailEngineBundle\Tests\Sender
 */
class SwiftMailerSenderTest extends TestCase
{
    use LoadTrait;

    /**
     * Test sending
     *
     * @throws \ReflectionException
     */
    public function testSend()
    {
        $this->configure(SwiftMailerSender::class, TwigFileRepository::class);

        $mailer = $this->container->get(Mailer::class);
$myVar = 'Value';
$higher = false;

if (rand(1, 6) > 3) {
    $higher = true;
} else {
    $higher = false;
}

        $template = $this->mockTemplate();
        $sender = $this->mockSender();

        $result = $sender->send($template, '[email protected]');

        $this->assertTrue($result);
    }

    /**
     * Mock sender
     *
     * @return SwiftMailerSender
     */
    private function mockSender(): SwiftMailerSender
    {
        $sender = new SwiftMailerSender(new \Swift_Mailer(new \Swift_Transport_NullTransport(new \Swift_Events_SimpleEventDispatcher())));

        return $sender;
    }

    /**
     * Mock template
     *
     * @return TestEmailTemplate
     *
     * @throws \ReflectionException
     * @throws \SfCod\EmailEngineBundle\Exception\RepositoryUnavailableException
     */
    private function mockTemplate(): TestEmailTemplate
    {
        $template = new TestEmailTemplate(new TestEmailOptions('message', 'filepath'));

        $filePath = (new \ReflectionClass(get_class($template)))->getFileName();
        $directory = dirname($filePath);

        $repository = new TwigFileRepository(new \Twig_Environment(new \Twig_Loader_Chain([
            new \Twig_Loader_Filesystem('Data', $directory),
        ])));
        $repository->connect($template);

        $template->setParameterResolver(new ParameterResolver($this->container));

        $template->setRepository($repository);

        return $template;
    }
}


1			<?php
2
3			namespace SfCod\EmailEngineBundle\Tests\Sender;
4
5			use PHPUnit\Framework\TestCase;
6			use SfCod\EmailEngineBundle\Example\TestEmailOptions;
7			use SfCod\EmailEngineBundle\Example\TestEmailTemplate;
8			use SfCod\EmailEngineBundle\Mailer\Mailer;
9			use SfCod\EmailEngineBundle\Repository\TwigFileRepository;
10			use SfCod\EmailEngineBundle\Sender\SwiftMailerSender;
11			use SfCod\EmailEngineBundle\Template\Params\ParameterResolver;
12			use SfCod\EmailEngineBundle\Tests\Data\LoadTrait;
13
14			/**
15			* Class SwiftMailerSenderTest
16			*
17			* @author Virchenko Maksim <[email protected]>
18			*
19			* @package SfCod\EmailEngineBundle\Tests\Sender
20			*/
21			class SwiftMailerSenderTest extends TestCase
22			{
23			use LoadTrait;
24
25			/**
26			* Test sending
27			*
28			* @throws \ReflectionException
29			*/
30			public function testSend()
31			{
32			$this->configure(SwiftMailerSender::class, TwigFileRepository::class);
33
34			$mailer = $this->container->get(Mailer::class);
			0 ignored issues – show Unused Code introduced 2018-06-18 15:38 UTC by Report Bug Copy Issue Report `$mailer` is not used, you could remove the assignment. This check looks for variable assignements that are either overwritten by other assignments or where the variable is not used subsequently. $myVar = 'Value'; $higher = false; if (rand(1, 6) > 3) { $higher = true; } else { $higher = false; } Both the `$myVar` assignment in line 1 and the `$higher` assignment in line 2 are dead. The first because `$myVar` is never used and the second because `$higher` is always overwritten for every possible time line. Loading history...
35
36			$template = $this->mockTemplate();
37			$sender = $this->mockSender();
38
39			$result = $sender->send($template, '[email protected]');
40
41			$this->assertTrue($result);
42			}
43
44			/**
45			* Mock sender
46			*
47			* @return SwiftMailerSender
48			*/
49			private function mockSender(): SwiftMailerSender
50			{
51			$sender = new SwiftMailerSender(new \Swift_Mailer(new \Swift_Transport_NullTransport(new \Swift_Events_SimpleEventDispatcher())));
52
53			return $sender;
54			}
55
56			/**
57			* Mock template
58			*
59			* @return TestEmailTemplate
60			*
61			* @throws \ReflectionException
62			* @throws \SfCod\EmailEngineBundle\Exception\RepositoryUnavailableException
63			*/
64			private function mockTemplate(): TestEmailTemplate
65			{
66			$template = new TestEmailTemplate(new TestEmailOptions('message', 'filepath'));
67
68			$filePath = (new \ReflectionClass(get_class($template)))->getFileName();
69			$directory = dirname($filePath);
70
71			$repository = new TwigFileRepository(new \Twig_Environment(new \Twig_Loader_Chain([
72			new \Twig_Loader_Filesystem('Data', $directory),
73			])));
74			$repository->connect($template);
75
76			$template->setParameterResolver(new ParameterResolver($this->container));
			0 ignored issues – show Compatibility introduced 2018-06-18 15:38 UTC by Report Bug Copy Issue Report `$this->container` of type `object<Psr\Container\ContainerInterface>` is not a sub-type of `object<Symfony\Component...ion\ContainerInterface>`. It seems like you assume a child interface of the interface `Psr\Container\ContainerInterface` to be always present. This check looks for parameters that are defined as one type in their type hint or doc comment but seem to be used as a narrower type, i.e an implementation of an interface or a subclass. Consider changing the type of the parameter or doing an instanceof check before assuming your parameter is of the expected type. Loading history...
77			$template->setRepository($repository);
78
79			return $template;
80			}
81			}
82

sfcod / email-engine

Push — master ( 80ef2e...65d1c9 )

SwiftMailerSenderTest::testSend() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like