ContentMapperTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	129
Duplicated Lines	24.03 %

Coupling/Cohesion

Components	0
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

6 Methods

Rating	Name	Duplication	Size	Complexity
A	getContent_ValidSummaryAndDescriptionGiven_ContentWithSummaryAndDescriptionIsReturned()	0	12	1
A	getContent_FieldsMissing_ContentExceptionIsThrown()	0	8	1
A	getContent_InvalidSummary_ContentExceptionIsThrown()	0	8	1
A	getValidContentData()	15	15	1
A	getContentWithInvalidSummary()	0	20	1
A	getContentDataWithMissingAttributes()	16	16	1

<?php
use Wambo\Catalog\Mapper\ContentMapper;

/**
 * Class ContentMapperTest tests the Wambo\Catalog\Mapper\ContentMapper class.
 */
class ContentMapperTest extends PHPUnit_Framework_TestCase

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

{
    /**
     * If the given summary and description is valid a Content model with the given summary and description should be
     * returned
     *
     * @test
     * @dataProvider getValidContentData
     *
     * @param array $contentData Product content data
     */
    public function getContent_ValidSummaryAndDescriptionGiven_ContentWithSummaryAndDescriptionIsReturned($contentData)
    {
        // arrange
        $productMapper = new ContentMapper();

        // act
        $content = $productMapper->getContent($contentData);

        // assert
        $this->assertNotEmpty($content->getSummaryText(), "The summary of the content model should not be empty");
        $this->assertNotEmpty($content->getProductDescription(), "The description of the content model should not be empty");
    }

    /**
     * If the given summary and description is valid a Content model with the given summary and description should be
     * returned
     *
     * @test
     * @dataProvider getContentDataWithMissingAttributes
     * @expectedException Wambo\Catalog\Error\ContentException
     * @expectedExceptionMessageRegExp /The field '.+' is missing in the given content data/
     *
     * @param array $contentData Product content data
     */
    public function getContent_FieldsMissing_ContentExceptionIsThrown($contentData)
    {
        // arrange
        $productMapper = new ContentMapper();

        // act
        $productMapper->getContent($contentData);
    }

    /**
     * If the given summary is invalid a ContentException should be thrown
     *
     * @test
     * @dataProvider getContentWithInvalidSummary
     * @expectedException Wambo\Catalog\Error\ContentException
     * @expectedExceptionMessageRegExp /The summary text should not be (shorter|longer) than \d+ characters/
     *
     * @param array $contentData Product content data
     */
    public function getContent_InvalidSummary_ContentExceptionIsThrown($contentData)
    {
        // arrange
        $productMapper = new ContentMapper();

        // act
        $productMapper->getContent($contentData);
    }

    /**
     * Get a list of valid content data for testing
     * @return array
     */
    public static function getValidContentData() {

This method seems to be duplicated in your project.

        return array(
            [
                [
                    "summary" => "A product summary",
                    "description" => "A detailed product description",
                ],

                [
                    "summary" => "ABCdefghijklmnopqrstuvwxyzöüä.ABCdefghijklmnopqrstuvwxyzöüä.ABCdefghijklmnopqrstuvwxyzöüä.ABCdefghijklmnopqrstuvwxyzöüä.",
                    "description" => "A detailed product description ...",
                ]
            ]
        );
    }

    /**
     * Get a list of valid content data for testing
     * @return array
     */
    public static function getContentWithInvalidSummary() {
        return array(
            [
                // empty
                [
                    "summary" => "",
                ],

                // too short
                [
                    "summary" => "A",
                ],

                // too long
                [
                    "summary" => "ABCdefghijklmnopqrstuvwxyzöüä.ABCdefghijklmnopqrstuvwxyzöüä.ABCdefghijklmnopqrstuvwxyzöüä.ABCdefghijklmnopqrstuvwxyzöüä.ABCdefghijklmnopqrstuvwxyzöüä.",
                ]
            ]
        );
    }

    /**
     * Get a list of content data object with missing attribtues for testing
     * @return array
     */
    public static function getContentDataWithMissingAttributes() {

This method seems to be duplicated in your project.

        return array(
            [
                // wrong casing
                [
                    "SUMMARY" => "A product summary",
                    "description" => "A detailed product description",
                ],

                // summary missing
                [
                    "description" => "A detailed product description",
                ]
            ]
        );
    }
}