Huntie\JsonApi\Testing\JsonApiAssertions

Complexity

Total Complexity

12

Size/Duplication

Total Lines	98
Duplicated Lines	22.45 %

Coupling/Cohesion

Components	1
Dependencies	0

Importance

Changes

0

5 Methods

Rating	Name	Duplication	Size	Complexity
A	assertArrayHasAll()	0	8	3
A	assertJsonApiResourceIdentifier()	0	12	3
A	assertJsonApiResourceObject()	0	5	1
A	assertJsonApiIdentifierCollection()	10	10	2
A	assertJsonApiObjectCollection()	12	12	3

<?php

namespace Huntie\JsonApi\Testing;

/**
 * Extend TestCase with additional JSON API related assertions.
 */
trait JsonApiAssertions
{
    /**
     * Assert that all given keys are set in an associative array. Nested
     * members may be specified using dot notation.
     *
     * @param array $keys
     * @param array $array
     *
     * @throws PHPUnit_Framework_AssertionFailedError
     */
    public function assertArrayHasAll(array $keys, array $array)
    {
        foreach ($keys as $key) {
            if (!array_has($array, $key)) {
                $this->fail('Failed asserting that key "' . $key . '" exists in input array.');

It seems like fail() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            }
        }
    }

    /**
     * Assert that an array contains a valid JSON API resource identifier.
     *
     * @param array       $array
     * @param string|null $type
     * @param mixed       $id
     *
     * @throws PHPUnit_Framework_AssertionFailedError
     */
    public function assertJsonApiResourceIdentifier(array $array, string $type = null, $id = null)
    {
        $this->assertArrayHasAll(['data.type', 'data.id'], $array);

        if ($type) {

The expression $type of type null|string is loosely compared to true; this is ambiguous if the string can be empty. You might want to explicitly use !== null instead.

            $this->assertEquals(array_get($array, 'data.type'), $type);

It seems like assertEquals() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }

        if ($id) {
            $this->assertEquals(array_get($array, 'data.id'), $id);

It seems like assertEquals() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }

    /**
     * Assert that an array contains a valid JSON API resource object.
     *
     * @param array       $array
     * @param string|null $type
     * @param mixed       $id
     *
     * @throws PHPUnit_Framework_AssertionFailedError
     */
    public function assertJsonApiResourceObject(array $array, string $type = null, $id = null)
    {
        $this->assertJsonApiResourceIdentifier($array, $type, $id);
        $this->assertInternalType('array', array_get($array, 'data.attributes'));

It seems like assertInternalType() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Assert that an array contains a valid JSON API resource identifier
     * object collection.
     *
     * @param array $array
     * @param int   $count
     *
     * @throws PHPUnit_Framework_AssertionFailedError
     */
    public function assertJsonApiIdentifierCollection(array $array, $count = null)

This method seems to be duplicated in your project.

    {
        $this->assertArrayHasKey('data', $array, 'No data key for collection');

It seems like assertArrayHasKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        foreach ($array['data'] as $identifier) {
            $this->assertArrayHasAll(['type', 'id'], (array) $identifier);
        }

        $this->assertCount($count, $array['data'], 'Incorrect object count returned in collection');

It seems like assertCount() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }

    /**
     * Assert that an array contains a valid JSON API resource object collection.
     *
     * @param array $array
     * @param int   $count
     *
     * @throws PHPUnit_Framework_AssertionFailedError
     */
    public function assertJsonApiObjectCollection(array $array, $count = null)

This method seems to be duplicated in your project.

    {
        $this->assertArrayHasKey('data', $array, 'No data key for collection');

It seems like assertArrayHasKey() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        foreach ($array['data'] as $object) {
            $this->assertArrayHasAll(['type', 'id', 'attributes'], (array) $object);
        }

        if (!is_null($count)) {
            $this->assertCount($count, $array['data'], 'Incorrect object count returned in collection');

It seems like assertCount() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }
}