AzineTemplateProviderTest::testAddCustomHeaders() - Code Metrics - Inspection of "Merge pull request #49 from eugene-o-z/issue/915_A..." - azine/email-bundle - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 0ba7dd...4699e0 )

by Dominik

created 2018-12-11 09:07 UTC

AzineTemplateProviderTest::testAddCustomHeaders() A

↳ Parent: AzineTemplateProviderTest

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	25
rs	9.52
c	0
b	0
f	0
cc	1
nc	1
nop	0

<?php

namespace Azine\EmailBundle\Tests\Services;

use Azine\EmailBundle\DependencyInjection\AzineEmailExtension;
use Azine\EmailBundle\Services\AzineTemplateProvider;

class AzineTemplateProviderTest extends \PHPUnit\Framework\TestCase
{
    private function getMockSetup()
    {
        $translatorMock = $this->getMockBuilder("Symfony\Bundle\FrameworkBundle\Translation\Translator")->disableOriginalConstructor()->setMethods(array('trans'))->getMock();

        $translatorMock->expects($this->any())->method('trans')->will($this->returnValueMap(array(
                                                        array('html.email.go.to.top.link.label', array(), 'messages', 'de', 'de übersetzung'),
                                                        array('html.email.go.to.top.link.label', array(), 'messages', 'en', 'en translation'),
                                                )));

        $routerMock = $this->getMockBuilder("Symfony\Component\Routing\Generator\UrlGeneratorInterface")->disableOriginalConstructor()->getMock();
        $routerMock->expects($this->any())->method('generate')->withAnyParameters()->will($this->returnCallback(array($this, 'createRelativeUrl')));

        $params = array(AzineEmailExtension::TEMPLATE_IMAGE_DIR => realpath(__DIR__.'/../../Resources/htmlTemplateImages/'),
                            AzineEmailExtension::ALLOWED_IMAGES_FOLDERS => array(realpath(__DIR__.'/../../Resources/htmlTemplateImages/')),
                            AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_NAME => 'utm_campaign',
                            AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_TERM => 'utm_term',
                            AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_SOURCE => 'utm_source',
                            AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_MEDIUM => 'utm_medium',
                            AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_CONTENT => 'utm_content',
                    );

        return array('router' => $routerMock, 'translator' => $translatorMock, 'params' => $params);
    }

    public function createRelativeUrl($routeName, $params)
    {
        if ('azine_email_serve_template_image' == $routeName) {
            return '/template/images/'.$params['filename'];
        }
        echo $routeName;

        return '/some/relative/url/to/images/folder';
    }

    public function testAddTemplateVariablesFor()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        // test without contentItems
        $contentVars = array('testVar' => 'testValue');
        $filledVars = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::FOS_USER_PWD_RESETTING_TEMPLATE, $contentVars);
        $this->assertSame('testValue', $filledVars['testVar']);
        $this->assertGreaterThan(sizeof($contentVars), sizeof($filledVars));

        $filledVars = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::FOS_USER_REGISTRATION_TEMPLATE, $contentVars);
        $this->assertSame('testValue', $filledVars['testVar']);
        $this->assertGreaterThan(sizeof($contentVars), sizeof($filledVars));

        // test with contentItems
        $contentVars[AzineTemplateProvider::CONTENT_ITEMS] = array(array(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE => array('otherTestVar' => 'otherTestValue')));
        $filledVars = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars);
        $this->assertSame('testValue', $filledVars['testVar']);
        $this->assertGreaterThan(sizeof($contentVars), sizeof($filledVars));
        $this->assertTrue(is_array($filledVars[AzineTemplateProvider::CONTENT_ITEMS]));
        $this->assertTrue(is_array($filledVars[AzineTemplateProvider::CONTENT_ITEMS][0][AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE]));
        $this->assertSame('otherTestValue', $filledVars[AzineTemplateProvider::CONTENT_ITEMS][0][AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE]['otherTestVar']);
    }

    public function testAddSnippetsWithImagesFor()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $contentVars = array('testVar' => 'testValue');
        $contentVars[AzineTemplateProvider::CONTENT_ITEMS] = array(array(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE => array('otherTestVar' => 'otherTestValue')));
        $contentVars = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars);

        $filledVars = $templateProvider->addTemplateSnippetsWithImagesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars, 'en');
        $this->assertSame('testValue', $filledVars['testVar']);
        $this->assertTrue(array_key_exists('linkToTop', $filledVars));

        $contentVars2 = array('testVar' => 'testValue');
        $contentVars2[AzineTemplateProvider::CONTENT_ITEMS] = array(array(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE => array('otherTestVar' => 'otherTestValue')));
        $contentVars2 = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::NEWSLETTER_TEMPLATE, $contentVars2);

        $filledVars2 = $templateProvider->addTemplateSnippetsWithImagesFor(AzineTemplateProvider::NEWSLETTER_TEMPLATE, $contentVars2, 'en');
        $this->assertSame($filledVars['linkToTop'], $filledVars2['linkToTop']);

        $contentVars3 = array('testVar' => 'testValue');
        $contentVars3[AzineTemplateProvider::CONTENT_ITEMS] = array(array(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE => array('otherTestVar' => 'otherTestValue')));
        $contentVars3 = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::NOTIFICATIONS_TEMPLATE, $contentVars3);

        $filledVars3 = $templateProvider->addTemplateSnippetsWithImagesFor(AzineTemplateProvider::NOTIFICATIONS_TEMPLATE, $contentVars3, 'de');
        $this->assertTrue(array_key_exists('linkToTop', $filledVars3));
        $this->assertNotSame($filledVars['linkToTop'], $filledVars3['linkToTop']);

        $filledVars4 = $templateProvider->addTemplateSnippetsWithImagesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars, 'de', true);
        $this->assertTrue(array_key_exists('linkToTop', $filledVars4));
        $this->assertSame($filledVars3['linkToTop'], $filledVars4['linkToTop']);
    }

    /**
     * \Exception("some required images are not yet added to the template-vars array.").
     *
     * @expectedException \Exception
     */
    public function testAddSnippetsWithImagesForEmptyVars()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $contentVars = array('testVar' => 'testValue');
        $contentVars[AzineTemplateProvider::CONTENT_ITEMS] = array(array(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE => array('otherTestVar' => 'otherTestValue')));
        $filledVars = $templateProvider->addTemplateSnippetsWithImagesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars, 'en');
$myVar = 'Value';
$higher = false;

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

    /**
     * \Exception("Only use the translator here when you already know in which language the user should get the email.").
     *
     * @expectedException \Exception
     */
    public function testAddSnippetsWithImagesForNoLocale()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $contentVars = array('testVar' => 'testValue');
        $contentVars[AzineTemplateProvider::CONTENT_ITEMS] = array(array(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE => array('otherTestVar' => 'otherTestValue')));
        $contentVars = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars);
        $filledVars = $templateProvider->addTemplateSnippetsWithImagesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars, null);
$myVar = 'Value';
$higher = false;

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

    public function testGetCampaignParamsFor()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $campaignParams1 = $templateProvider->getCampaignParamsFor(AzineTemplateProvider::NEWSLETTER_TEMPLATE);
        $this->assertSame(3, sizeof($campaignParams1));
        $this->assertSame('newsletter', $campaignParams1['utm_source']);

        $campaignParams2 = $templateProvider->getCampaignParamsFor(AzineTemplateProvider::NOTIFICATIONS_TEMPLATE);
        $this->assertSame(3, sizeof($campaignParams2));
        $this->assertSame('mailnotify', $campaignParams2['utm_source']);

        $campaignParams3 = $templateProvider->getCampaignParamsFor(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE);
        $this->assertTrue(is_array($campaignParams3));
        $this->assertSame(3, sizeof($campaignParams3));
    }

    public function testIsFileAllowed()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $allowed1 = $mocks['params'][AzineEmailExtension::TEMPLATE_IMAGE_DIR].'/logo.png';
        $key = $templateProvider->isFileAllowed($allowed1);
        $this->assertTrue(is_string($key), "$allowed1 is not allowed, but it should!");

        $allowed2 = $mocks['params'][AzineEmailExtension::ALLOWED_IMAGES_FOLDERS][0].'/logo.png';
        $this->assertTrue(is_string($templateProvider->isFileAllowed($allowed2)), "$allowed2 is not allowed, but it should!");

        $notAllowed = __FILE__;
        $this->assertFalse(is_string($templateProvider->isFileAllowed($notAllowed)), "$notAllowed is allowed, but it should not!");

        $this->assertTrue(is_dir($templateProvider->getFolderFrom($key)));
        $this->assertFalse(is_dir($templateProvider->getFolderFrom('noKey')));
    }

    public function testMakeImagePathsWebRelative()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $locale = 'en';

        $contentVars = array('testVar' => 'testValue');
        $contentVars[AzineTemplateProvider::CONTENT_ITEMS] = array(array(AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE => array('otherTestVar' => 'otherTestValue')));
        $contentVars = $templateProvider->addTemplateVariablesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars);
        $contentVars = $templateProvider->addTemplateSnippetsWithImagesFor(AzineTemplateProvider::BASE_TEMPLATE, $contentVars, $locale);

        $relativeVars = $templateProvider->makeImagePathsWebRelative($contentVars, $locale);
        $this->assertTrue(is_file(realpath($contentVars['logo_png'])));
        $this->assertNotSame($relativeVars['logo_png'], $contentVars['logo_png']);

        $contentItemImage = $contentVars[AzineTemplateProvider::CONTENT_ITEMS][0][AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE]['logo_png'];
        $contentItemImage2 = $relativeVars[AzineTemplateProvider::CONTENT_ITEMS][0][AzineTemplateProvider::CONTENT_ITEM_MESSAGE_TEMPLATE]['logo_png'];
        $this->assertTrue(is_file(realpath($contentItemImage)));
        $this->assertNotSame($contentItemImage, $contentItemImage2);
    }

    public function testGetWebViewTokenId()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);
        $this->assertSame(AzineTemplateProvider::EMAIL_WEB_VIEW_TOKEN, $templateProvider->getWebViewTokenId());
    }

    public function testSaveWebViewFor()
    {
        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $this->assertFalse($templateProvider->saveWebViewFor(AzineTemplateProvider::FOS_USER_PWD_RESETTING_TEMPLATE));
        $this->assertFalse($templateProvider->saveWebViewFor(AzineTemplateProvider::FOS_USER_REGISTRATION_TEMPLATE));
        $this->assertFalse($templateProvider->saveWebViewFor(AzineTemplateProvider::NOTIFICATIONS_TEMPLATE));
        $this->assertTrue($templateProvider->saveWebViewFor(AzineTemplateProvider::NEWSLETTER_TEMPLATE));
        $this->assertFalse($templateProvider->saveWebViewFor('some other string'));
    }

    public function testAddCustomHeaders()
    {
        $message = new \Swift_Message();

        $mocks = $this->getMockSetup();
        $templateProvider = new AzineTemplateProvider($mocks['router'], $mocks['translator'], $mocks['params']);
function acceptsInteger($int) { }

$x = '123'; // string "123"

// Instead of
acceptsInteger($x);

// we recommend to use
acceptsInteger((integer) $x);

        $tokenValue = 'testToken';
        $campaignValue = 'testCampaignValue';
        $sourceValue = 'testSourceValue';

        $params = array(AzineTemplateProvider::EMAIL_WEB_VIEW_TOKEN => $tokenValue,
                        AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_NAME => $campaignValue,
                        AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_SOURCE => $sourceValue);

        $templateProvider->addCustomHeaders('testTemplate', $message, $params);

        $headerSet = $message->getHeaders();
        $this->assertTrue($headerSet->has('x-azine-webview-token'));
        $this->assertEquals($headerSet->get('x-azine-webview-token')->getValue(), $tokenValue);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->assertTrue($headerSet->has('x-utm_campaign'));
        $this->assertEquals($headerSet->get('x-utm_campaign')->getValue(), $campaignValue);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
        $this->assertTrue($headerSet->has('x-utm_source'));
        $this->assertEquals($headerSet->get('x-utm_source')->getValue(), $sourceValue);
interface User
{
    /** @return string */
    public function getPassword();
}

class MyUser implements User
{
    public function getPassword()
    {
        // return something
    }

    public function getDisplayName()
    {
        // return some name.
    }
}

class AuthSystem
{
    public function authenticate(User $user)
    {
        $this->logger->info(sprintf('Authenticating %s.', $user->getDisplayName()));
        // do something.
    }
}
    }
}


Push — master ( 0ba7dd...4699e0 )

AzineTemplateProviderTest::testAddCustomHeaders() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

1			<?php
2
3			namespace Azine\EmailBundle\Tests\Services;
4
5			use Azine\EmailBundle\DependencyInjection\AzineEmailExtension;
6			use Azine\EmailBundle\Services\AzineTemplateProvider;
7
8			class AzineTemplateProviderTest extends \PHPUnit\Framework\TestCase
9			{
10			private function getMockSetup()
11			{
12			$translatorMock = $this->getMockBuilder("Symfony\Bundle\FrameworkBundle\Translation\Translator")->disableOriginalConstructor()->setMethods(array('trans'))->getMock();
13
14			$translatorMock->expects($this->any())->method('trans')->will($this->returnValueMap(array(
15			array('html.email.go.to.top.link.label', array(), 'messages', 'de', 'de übersetzung'),
16			array('html.email.go.to.top.link.label', array(), 'messages', 'en', 'en translation'),
17			)));
18
19			$routerMock = $this->getMockBuilder("Symfony\Component\Routing\Generator\UrlGeneratorInterface")->disableOriginalConstructor()->getMock();
20			$routerMock->expects($this->any())->method('generate')->withAnyParameters()->will($this->returnCallback(array($this, 'createRelativeUrl')));
21
22			$params = array(AzineEmailExtension::TEMPLATE_IMAGE_DIR => realpath(__DIR__.'/../../Resources/htmlTemplateImages/'),
23			AzineEmailExtension::ALLOWED_IMAGES_FOLDERS => array(realpath(__DIR__.'/../../Resources/htmlTemplateImages/')),
24			AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_NAME => 'utm_campaign',
25			AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_TERM => 'utm_term',
26			AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_SOURCE => 'utm_source',
27			AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_MEDIUM => 'utm_medium',
28			AzineEmailExtension::TRACKING_PARAM_CAMPAIGN_CONTENT => 'utm_content',
29			);
30
31			return array('router' => $routerMock, 'translator' => $translatorMock, 'params' => $params);
32			}
33
34			public function createRelativeUrl($routeName, $params)
35			{
36			if ('azine_email_serve_template_image' == $routeName) {
37			return '/template/images/'.$params['filename'];
38			}
39			echo $routeName;
40

azine / email-bundle

Push — master ( 0ba7dd...4699e0 )

AzineTemplateProviderTest::testAddCustomHeaders() A

Complexity

Size

Duplication

Importance

Available Fixes

Available Fixes

Available Fixes

Duplication Side-by-Side

Filter issues like