CatalogMapperTest

Complexity

Total Complexity

7

Size/Duplication

Total Lines	206
Duplicated Lines	43.69 %

Coupling/Cohesion

Components	1
Dependencies	7

Importance

Changes	5
Bugs	0	Features	1

7 Methods

Rating	Name	Duplication	Size	Complexity
A	getCatalog_EmptyArrayIsGiven_EmptyCatalogIsReturned()	0	16	1
A	getCatalog_ArrayOfTwoProductsGiven_ProductMapperThrowsException_CatalogExceptionIsThrown()	0	18	1
A	getCatalog_ArrayWithDuplicateSKUGiven_CatalogExceptionIsThrown()	23	23	1
A	getCatalog_ArrayWithDuplicateSlugsGiven_CatalogExceptionIsThrown()	23	23	1
A	getCatalog_ArrayWithSimilarSKUsGiven_CatalogExceptionIsThrown()	22	22	1
A	getCatalog_ArrayWithSimilarSlugsGiven_CatalogExceptionIsThrown()	22	22	1
B	getCatalog_ArrayOfTwoProductsGiven_ProductMapperReturnsProducts_CatalogWithTwoProductsIsReturned()	0	25	1

<?php

use Wambo\Catalog\Mapper\CatalogMapper;
use Wambo\Catalog\Mapper\ProductMapper;
use Wambo\Catalog\Model\Content;
use Wambo\Catalog\Model\Product;
use Wambo\Catalog\Model\SKU;
use Wambo\Catalog\Model\Slug;

/**
 * Class CatalogMapperTest contains tests for the CatalogMapper class.
 */
class CatalogMapperTest extends \PHPUnit_Framework_TestCase

PSR1 recommends that each class must be in a namespace of at least one level to avoid collisions.

{
    /**
     * @test
     */
    public function getCatalog_EmptyArrayIsGiven_EmptyCatalogIsReturned()
    {
        // arrange
        $catalogData = array();
        $productMapperMock = $this->getMockBuilder(ProductMapper::class)->disableOriginalConstructor()->getMock();
        /** @var ProductMapper $productMapperMock A product mapper instance */

        $productMapper = new CatalogMapper($productMapperMock);


        // act
        $result = $productMapper->getCatalog($catalogData);

        // assert
        $this->assertNotNull($result, "getCatalog should have returned an empty catalog");
    }

    /**
     * When the product mapper throws an exception the getCatalog function should throw a CatalogException
     *
     * @test
     * @expectedException Wambo\Catalog\Error\CatalogException
     */
    public function getCatalog_ArrayOfTwoProductsGiven_ProductMapperThrowsException_CatalogExceptionIsThrown()
    {
        // arrange
        $catalogData = [
            ["sku" => "0001"],
            ["sku" => "0002"],
        ];
        $productMapperMock = $this->createMock(ProductMapper::class);
        $productMapperMock->method("getProduct")->willThrowException(new Exception("Some product mapper exception"));

        /** @var ProductMapper $productMapperMock A product mapper instance */

        $productMapper = new CatalogMapper($productMapperMock);


        // act
        $productMapper->getCatalog($catalogData);
    }

    /**
     * If the the given product catalog data contains products with duplicate SKUs a
     * Wambo\Catalog\Error\CatalogException should be thrown.
     *
     * @test
     *
     * @expectedException Wambo\Catalog\Error\CatalogException
     * @expectedExceptionMessageRegExp /Cannot add a second product with the SKU '.+'/
     */
    public function getCatalog_ArrayWithDuplicateSKUGiven_CatalogExceptionIsThrown()

This method seems to be duplicated in your project.

    {
        // arrange
        $catalogData = [
            ["sku" => "0001"],
            ["sku" => "0002"],
            ["sku" => "0001"],
        ];

        $productMapperMock = $this->getMockBuilder(ProductMapper::class)->disableOriginalConstructor()->getMock();
        $productMapperMock->method("getProduct")->will($this->onConsecutiveCalls(
            new Product(new SKU("0001"), new Slug("product-1"), new Content("Title", "Summary")),
            new Product(new SKU("0002"), new Slug("product-2"), new Content("Title", "Summary")),
            new Product(new SKU("0001"), new Slug("product-1a"), new Content("Title", "Summary")))
        );

        /** @var ProductMapper $productMapperMock A product mapper instance */

        $productMapper = new CatalogMapper($productMapperMock);

        // act
        $productMapper->getCatalog($catalogData);
    }

    /**
     * If the the given product catalog data contains products with duplicate Slugs a
     * Wambo\Catalog\Error\CatalogException should be thrown.
     *
     * @test
     *
     * @expectedException Wambo\Catalog\Error\CatalogException
     * @expectedExceptionMessageRegExp /Cannot add a second product with the Slug 'product-1'/
     */
    public function getCatalog_ArrayWithDuplicateSlugsGiven_CatalogExceptionIsThrown()

This method seems to be duplicated in your project.

    {
        // arrange
        $catalogData = [
            ["sku" => "0001"],
            ["sku" => "0002"],
            ["sku" => "0003"],
        ];

        $productMapperMock = $this->getMockBuilder(ProductMapper::class)->disableOriginalConstructor()->getMock();
        $productMapperMock->method("getProduct")->will($this->onConsecutiveCalls(
            new Product(new SKU("0001"), new Slug("product-1"), new Content("Title", "Summary")),
            new Product(new SKU("0002"), new Slug("product-2"), new Content("Title", "Summary")),
            new Product(new SKU("0003"), new Slug("product-1"), new Content("Title", "Summary")))
        );

        /** @var ProductMapper $productMapperMock A product mapper instance */

        $productMapper = new CatalogMapper($productMapperMock);

        // act
        $productMapper->getCatalog($catalogData);
    }

    /**
     * If the the given product catalog data contains products with similar SKUs a
     * Wambo\Catalog\Error\CatalogException should be thrown.
     *
     * @test
     *
     * @expectedException Wambo\Catalog\Error\CatalogException
     * @expectedExceptionMessageRegExp /Cannot add a second product with the SKU 'product'/
     */
    public function getCatalog_ArrayWithSimilarSKUsGiven_CatalogExceptionIsThrown()

This method seems to be duplicated in your project.

    {
        // arrange
        $catalogData = [
            ["sku" => "0001"],
            ["sku" => "0002"],
            ["sku" => "0003"],
        ];

        $productMapperMock = $this->createMock(ProductMapper::class);
        $productMapperMock->method("getProduct")->will($this->onConsecutiveCalls(
            new Product(new SKU("product"), new Slug("product-a"), new Content("Title", "Summary")),
            new Product(new SKU("product"), new Slug("product-b"), new Content("Title", "Summary")))
        );

        /** @var ProductMapper $productMapperMock A product mapper instance */

        $productMapper = new CatalogMapper($productMapperMock);

        // act
        $productMapper->getCatalog($catalogData);
    }

    /**
     * If the the given product catalog data contains products with similar slugs a
     * Wambo\Catalog\Error\CatalogException should be thrown.
     *
     * @test
     *
     * @expectedException Wambo\Catalog\Error\CatalogException
     * @expectedExceptionMessageRegExp /Cannot add a second product with the Slug 'a-product'/
     */
    public function getCatalog_ArrayWithSimilarSlugsGiven_CatalogExceptionIsThrown()

This method seems to be duplicated in your project.

    {
        // arrange
        $catalogData = [
            ["sku" => "0001"],
            ["sku" => "0002"],
            ["sku" => "0003"],
        ];

        $productMapperMock = $this->getMockBuilder(ProductMapper::class)->disableOriginalConstructor()->getMock();
        $productMapperMock->method("getProduct")->will($this->onConsecutiveCalls(
            new Product(new SKU("0001"), new Slug("a-product"), new Content("Title", "Summary")),
            new Product(new SKU("0002"), new Slug("A-Product"), new Content("Title", "Summary")))
        );

        /** @var ProductMapper $productMapperMock A product mapper instance */

        $productMapper = new CatalogMapper($productMapperMock);

        // act
        $productMapper->getCatalog($catalogData);
    }

    /**
     * @test
     */
    public function getCatalog_ArrayOfTwoProductsGiven_ProductMapperReturnsProducts_CatalogWithTwoProductsIsReturned()
    {
        // arrange
        $catalogData = [
            ["sku" => "0001"],
            ["sku" => "0002"],
        ];

        $productMapperMock = $this->createMock(ProductMapper::class);
        $productMapperMock->method("getProduct")->will($this->onConsecutiveCalls(
            new Product(new SKU("0001"), new Slug("product-1"), new Content("Title", "Summary")),
            new Product(new SKU("0002"), new Slug("product-2"), new Content("Title", "Summary")))
        );

        /** @var ProductMapper $productMapperMock A product mapper instance */

        $productMapper = new CatalogMapper($productMapperMock);


        // act
        $catalog = $productMapper->getCatalog($catalogData);

        // assert
        $this->assertCount(2, $catalog, "The catalog should contain two products");
    }
}