Novactive\Tests\CombineKeysCollectionTest

Complexity

Total Complexity

4

Size/Duplication

Total Lines	78
Duplicated Lines	69.23 %

Coupling/Cohesion

Components	1
Dependencies	3

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	testCombineKeysThrowsExceptionIfPassedNonTraversableNonArray()	0	5	1
A	testCombineKeysThrowsExceptionIfPassedTraversableWithBadCount()	0	5	1
B	testCombineKeysAcceptsTraversable()	27	27	1
B	testCombineKeysUsesIncomingTraversableAsKeysForCollectionsValues()	27	27	1

<?php
/**
 * Novactive Collection.
 *
 * @author    Luke Visinoni <l.visinoni@novactive.us, luke.visinoni@gmail.com>
 * @author    Sébastien Morel <s.morel@novactive.us, morel.seb@gmail.com>
 * @copyright 2017 Novactive
 * @license   MIT
 */

namespace Novactive\Tests;

use Novactive\Collection\Factory;

/**
 * Class CombineKeysCollectionTest.
 */
class CombineKeysCollectionTest extends UnitTestCase
{
    public function testCombineKeysUsesIncomingTraversableAsKeysForCollectionsValues()

This method seems to be duplicated in your project.

    {
        $coll    = Factory::create($this->fixtures['names']);
        $orig    = $coll->toArray();
        $newkeys = $coll->combineKeys($this->fixtures['emails']);
        $this->assertEquals(
            [
                'colleen.mills@example.org' => 'Chelsea',
                'hilda01@example.net'       => 'Adella',
                'leffler.ron@example.org'   => 'Monte',
                'jboyle@example.net'        => 'Maye',
                'esta10@example.com'        => 'Lottie',
                'fcormier@example.com'      => 'Don',
                'verda25@example.com'       => 'Dayton',
                'mparker@example.org'       => 'Kirk',
                'apollich@example.net'      => 'Troy',
                'hrussel@example.net'       => 'Nakia',
            ],
            $newkeys->toArray()
        );
        $this->assertNotSame($newkeys->toArray(), $orig);
        $this->assertSame(
            $coll->toArray(),
            $orig,
            'The original collection should not be affected by Collection::combineKeys().'
        );
    }

    /**
     * @expectedException \InvalidArgumentException
     * @expectedExceptionMessage Invalid input type for combineKeys.
     */
    public function testCombineKeysThrowsExceptionIfPassedNonTraversableNonArray()
    {
        $coll = Factory::create($this->fixtures['emails']);
        $coll->combineKeys('this is not traversable');

'this is not traversable' is of type string, but the function expects a object<Traversable>|array.

    }

    /**
     * @expectedException \InvalidArgumentException
     * @expectedExceptionMessage Invalid input for combineKeys, number of items does not match.
     */
    public function testCombineKeysThrowsExceptionIfPassedTraversableWithBadCount()
    {
        $coll = Factory::create($this->fixtures['emails']);
        $coll->combineKeys([1, 2, 3]);
    }

    public function testCombineKeysAcceptsTraversable()

This method seems to be duplicated in your project.

    {
        $coll        = Factory::create($this->fixtures['names']);
        $orig        = $coll->toArray();
        $iter        = $this->getIteratorForArray($this->fixtures['emails']);
        $keycombined = $coll->combineKeys($iter);
        $this->assertSame(
            [
                'colleen.mills@example.org' => 'Chelsea',
                'hilda01@example.net'       => 'Adella',
                'leffler.ron@example.org'   => 'Monte',
                'jboyle@example.net'        => 'Maye',
                'esta10@example.com'        => 'Lottie',
                'fcormier@example.com'      => 'Don',
                'verda25@example.com'       => 'Dayton',
                'mparker@example.org'       => 'Kirk',
                'apollich@example.net'      => 'Troy',
                'hrussel@example.net'       => 'Nakia',
            ],
            $keycombined->toArray()
        );
        $this->assertSame(
            $orig,
            $coll->toArray(),
            'Ensure that Collection::combineKeys does not change the original collection.'
        );
    }
}