ExportContactConsumerTest::testConsumeApiUnprocessableEntityHttpExceptionException() - Code Metrics - Inspection of "improve logging on UnprocessableEntityHttpExceptio..." - commerceleague/magento2-module-activecampaign - Measure and Improve Code Quality continuously with Scrutinizer

Completed
Push — master ( 0f220b...47b153 )

by Andreas
created 2019-07-17 14:18 UTC
testConsumeApiUnprocessableEntityHttpExceptionException() A

↳ Parent: ExportContactConsumerTest
Complexity

Conditions	1
Paths	1
Size

Total Lines	52
Code Lines	37
Duplication

Lines	0
Ratio	0 %
Importance

Changes
Metric	Value
cc	1
eloc	37
nc	1
nop	0
dl	0
loc	52
rs	9.328
c	0
b	0
f	0
How to fix Long Method

<?php
declare(strict_types=1);
/**
 */

namespace CommerceLeague\ActiveCampaign\Test\Unit\MessageQueue\Customer;

use CommerceLeague\ActiveCampaign\Api\ContactRepositoryInterface;
use CommerceLeague\ActiveCampaign\Api\Data\ContactInterface;
use CommerceLeague\ActiveCampaign\Gateway\Request\ContactBuilder as ContactRequestBuilder;
use CommerceLeague\ActiveCampaign\Logger\Logger;
use CommerceLeague\ActiveCampaign\MessageQueue\Customer\ExportContactConsumer;
use CommerceLeague\ActiveCampaignApi\Api\ContactApiResourceInterface;
use CommerceLeague\ActiveCampaignApi\Exception\HttpException;
use CommerceLeague\ActiveCampaignApi\Exception\UnprocessableEntityHttpException;
use Magento\Customer\Api\CustomerRepositoryInterface as MagentoCustomerRepositoryInterface;
use Magento\Customer\Api\Data\CustomerInterface as MagentoCustomerInterface;
use Magento\Framework\Exception\NoSuchEntityException;
use Magento\Framework\Phrase;
use PHPUnit\Framework\MockObject\MockObject;
use PHPUnit\Framework\TestCase;
use CommerceLeague\ActiveCampaign\Gateway\Client;

class ExportContactConsumerTest extends TestCase
{
    /**
     * @var MockObject|MagentoCustomerRepositoryInterface
     */
    protected $magentoCustomerRepository;

    /**
     * @var MockObject|Logger
     */
    protected $logger;

    /**
     * @var MockObject|ContactRepositoryInterface
     */
    protected $contactRepository;

    /**
     * @var MockObject|ContactRequestBuilder
     */
    protected $contactRequestBuilder;

    /**
     * @var MockObject|Client
     */
    protected $client;

    /**
     * @var MockObject|ContactInterface
     */
    protected $contact;

    /**
     * @var MockObject|ContactApiResourceInterface
     */
    protected $contactApi;

    /**
     * @var MockObject|MagentoCustomerInterface
     */
    protected $magentoCustomer;

    /**
     * @var ExportContactConsumer
     */
    protected $exportContactConsumer;

    protected function setUp()
    {
        $this->magentoCustomerRepository = $this->createMock(MagentoCustomerRepositoryInterface::class);
        $this->logger = $this->createMock(Logger::class);
        $this->contactRepository = $this->createMock(ContactRepositoryInterface::class);
        $this->contactRequestBuilder = $this->createMock(ContactRequestBuilder::class);
        $this->client = $this->createMock(Client::class);
        $this->contact = $this->createMock(ContactInterface::class);
        $this->contactApi = $this->createMock(ContactApiResourceInterface::class);
        $this->magentoCustomer = $this->createMock(MagentoCustomerInterface::class);

        $this->exportContactConsumer = new ExportContactConsumer(
            $this->magentoCustomerRepository,
            $this->logger,
            $this->contactRepository,
            $this->contactRequestBuilder,
            $this->client
        );
    }

    public function testConsumeWithAbsentMagentoCustomer()
    {
        $magentoCustomerId = 123;

        $exceptionMessage = 'an exception message';
        $exception = new NoSuchEntityException(new Phrase($exceptionMessage));

        $this->magentoCustomerRepository->expects($this->once())
                                          expects($this->once())
            ->method('getById')
            ->with($magentoCustomerId)
            ->willThrowException($exception);

        $this->logger->expects($this->once())
                       expects($this->once())
            ->method('error')
            ->with($exceptionMessage);

        $this->contactRepository->expects($this->never())
                                  expects($this->never())
            ->method('getOrCreateByEmail');

        $this->exportContactConsumer->consume(json_encode(['magento_customer_id' => $magentoCustomerId]));
    }

    public function testConsumeApiHttpException()
    {
        $magentoCustomerId = 123;
        $email = '[email protected]';
        $request = ['request'];

        $this->magentoCustomerRepository->expects($this->once())
            ->method('getById')
            ->with($magentoCustomerId)
            ->willReturn($this->magentoCustomer);

        $this->magentoCustomer->expects($this->once())
                                expects($this->once())
            ->method('getEmail')
            ->willReturn($email);

        $this->contactRepository->expects($this->once())
            ->method('getOrCreateByEmail')
            ->with($email)
            ->willReturn($this->contact);

        $this->contactRequestBuilder->expects($this->once())
                                      expects($this->once())
            ->method('buildWithMagentoCustomer')
            ->willReturn($request);

        $this->client->expects($this->once())
                       expects($this->once())
            ->method('getContactApi')
            ->willReturn($this->contactApi);

        /** @var MockObject|HttpException $httpException */
        $httpException = $this->createMock(HttpException::class);

        $this->contactApi->expects($this->once())
                           expects($this->once())
            ->method('upsert')
            ->with(['contact' => $request])
            ->willThrowException($httpException);

        $this->logger->expects($this->once())
            ->method('error');

        $this->contact->expects($this->never())
                        expects($this->never())
            ->method('setActiveCampaignId');

        $this->exportContactConsumer->consume(json_encode(['magento_customer_id' => $magentoCustomerId]));
    }

    public function testConsumeApiUnprocessableEntityHttpExceptionException()
    {
        $magentoCustomerId = 123;
        $email = '[email protected]';
        $request = ['request'];
        $responseErrors = ['first error', 'second error'];

        $this->magentoCustomerRepository->expects($this->once())
            ->method('getById')
            ->with($magentoCustomerId)
            ->willReturn($this->magentoCustomer);

        $this->magentoCustomer->expects($this->once())
            ->method('getEmail')
            ->willReturn($email);

        $this->contactRepository->expects($this->once())
            ->method('getOrCreateByEmail')
            ->with($email)
            ->willReturn($this->contact);

        $this->contactRequestBuilder->expects($this->once())
            ->method('buildWithMagentoCustomer')
            ->willReturn($request);

        $this->client->expects($this->once())
            ->method('getContactApi')
            ->willReturn($this->contactApi);

        /** @var MockObject|UnprocessableEntityHttpException $unprocessableEntityHttpException */
        $unprocessableEntityHttpException = $this->createMock(UnprocessableEntityHttpException::class);

        $this->contactApi->expects($this->once())
            ->method('upsert')
            ->with(['contact' => $request])
            ->willThrowException($unprocessableEntityHttpException);

        $this->logger->expects($this->exactly(2))
            ->method('error');

        $unprocessableEntityHttpException->expects($this->once())
            ->method('getResponseErrors')
            ->willReturn($responseErrors);

        $this->logger->expects($this->at(1))
            ->method('error')
            ->with(print_r($responseErrors, true));

        $this->contact->expects($this->never())
            ->method('setActiveCampaignId');

        $this->exportContactConsumer->consume(json_encode(['magento_customer_id' => $magentoCustomerId]));
    }

    public function testConsume()
    {
        $magentoCustomerId = 123;
        $email = '[email protected]';
        $request = ['request'];
        $activeCampaignId = 456;
        $response = ['contact' => ['id' => $activeCampaignId]];

        $this->magentoCustomerRepository->expects($this->once())
            ->method('getById')
            ->with($magentoCustomerId)
            ->willReturn($this->magentoCustomer);

        $this->magentoCustomer->expects($this->once())
            ->method('getEmail')
            ->willReturn($email);

        $this->contactRepository->expects($this->once())
            ->method('getOrCreateByEmail')
            ->with($email)
            ->willReturn($this->contact);

        $this->contactRequestBuilder->expects($this->once())
            ->method('buildWithMagentoCustomer')
            ->with($this->magentoCustomer)
            ->willReturn($request);

        $this->client->expects($this->once())
            ->method('getContactApi')
            ->willReturn($this->contactApi);

        $this->contactApi->expects($this->once())
            ->method('upsert')
            ->with(['contact' => $request])
            ->willReturn($response);

        $this->contact->expects($this->once())
            ->method('setActiveCampaignId')
            ->with($activeCampaignId)
            ->willReturnSelf();

        $this->contactRepository->expects($this->once())
            ->method('save')
            ->with($this->contact);

        $this->logger->expects($this->never())
            ->method('error');

        $this->exportContactConsumer->consume(json_encode(['magento_customer_id' => $magentoCustomerId]));
    }
}

commerceleague / magento2-module-activecampaign

GitHub Access Token became invalid

Push — master ( 0f220b...47b153 )

testConsumeApiUnprocessableEntityHttpExceptionException() A

Complexity

Size

Duplication

Importance

How to fix Long Method

Long Method

Duplication Side-by-Side

Filter issues like
