EntityStrictCollectionTester::invalidAddElement() - Code Metrics - Inspection of "Merge pull request #4 from atrapalo/StrictCollecti..." - atrapalo/php-tools - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( fdb8ab...a48276 )

by Guillermo

created 2017-04-24 09:06 UTC

EntityStrictCollectionTester::invalidAddElement() A

↳ Parent: EntityStrictCollectionTester

Complexity

Conditions	1
Paths	1

Size

Total Lines	13
Code Lines	7

Duplication

Lines	0
Ratio	0 %

Importance

Changes

Metric	Value
dl	0
loc	13
rs	9.4285
c	0
b	0
f	0
cc	1
eloc	7
nc	1
nop	0

<?php

namespace Atrapalo\PHPTools\Tester\Collection;

use Atrapalo\PHPTools\Collection\EntityStrictCollection;

/**
 * Class EntityCollectionTester
 * @package Atrapalo\PHPTools\Collection
 *
 * @author Guillermo González <[email protected]>
 */
trait EntityStrictCollectionTester
{
    /**
     * @return string
     */
    abstract protected function entityCollectionClass(): string;

    /**
     * @test
     */
    public function emptyConstructor()
    {
        /** @var EntityStrictCollection $entityCollectionClass */
        $entityCollectionClass = $this->entityCollectionClass();
        $exception = $entityCollectionClass::customEmptyException();

        $this->expectException(get_class($exception));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->expectExceptionMessage($exception->getMessage());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        new $entityCollectionClass([]);
    }

    /**
     * @test
     * @dataProvider invalidEntities
     * @param array $elements
     */
    public function invalidElementByConstructor(array $elements)
    {
        /** @var EntityStrictCollection $entityCollectionClass */
        $entityCollectionClass = $this->entityCollectionClass();
        $exception = $entityCollectionClass::customInvalidEntityException();

        $this->expectException(get_class($exception));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->expectExceptionMessage($exception->getMessage());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        new $entityCollectionClass($elements);
    }

    /**
     * @return array
     */
    public function invalidEntities()
    {
        return [
            [[1]],
            [['element']],
            [[new \stdClass()]],
            [[new \stdClass(), new \stdClass()]],
            [[null]],
            [[false]]
        ];
    }

    /**
     * @test
     */
    public function invalidAddElement()
    {
        /** @var EntityStrictCollection $entityCollectionClass */
        $entityCollectionClass = $this->entityCollectionClass();
        $exception = $entityCollectionClass::customInvalidEntityException();

        $this->expectException(get_class($exception));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->expectExceptionMessage($exception->getMessage());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        /** @var EntityStrictCollection $collection */
        $collection = new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()], true);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $collection->add([]);
    }

    /**
     * @test
     */
    public function validChildrenElementByConstructor()
    {
        /** @var EntityStrictCollection $entityCollectionClass */
        $entityCollectionClass = $this->entityCollectionClass();
        $exception = $entityCollectionClass::customInvalidEntityException();

        $this->expectException(get_class($exception));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->expectExceptionMessage($exception->getMessage());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()]);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * @test
     */
    public function validElementByConstructor()
    {
        /** @var EntityStrictCollection $entityCollectionClass */
        $entityCollectionClass = $this->entityCollectionClass();

        /** @var EntityStrictCollection $collection */
        $collection = new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()], true);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        $this->assertSame(1, $collection->count());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $this->assertInstanceOf($entityCollectionClass::entityClass(), $collection->current());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * @test
     */
    public function addTwoValidElement()
    {
        /** @var EntityStrictCollection $entityCollectionClass */
        $entityCollectionClass = $this->entityCollectionClass();

        /** @var EntityStrictCollection $collection */
        $collection = new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()], true);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}


        /** @var \Prophecy\Prophecy\ObjectProphecy $entity */
        $entity = $this->prophesize($entityCollectionClass::entityClass());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        $collection->add($entity->reveal());
        $collection->add($entity->reveal());

        $this->assertSame(3, $collection->count());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        foreach ($collection as $element) {
            $this->assertInstanceOf($entityCollectionClass::entityClass(), $element);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }
    }
}


1			<?php
2
3			namespace Atrapalo\PHPTools\Tester\Collection;
4
5			use Atrapalo\PHPTools\Collection\EntityStrictCollection;
6
7			/**
8			* Class EntityCollectionTester
9			* @package Atrapalo\PHPTools\Collection
10			*
11			* @author Guillermo González <[email protected]>
12			*/
13			trait EntityStrictCollectionTester
14			{
15			/**
16			* @return string
17			*/
18			abstract protected function entityCollectionClass(): string;
19
20			/**
21			* @test
22			*/
23			public function emptyConstructor()
24			{
25			/** @var EntityStrictCollection $entityCollectionClass */
26			$entityCollectionClass = $this->entityCollectionClass();
27			$exception = $entityCollectionClass::customEmptyException();
28
29			$this->expectException(get_class($exception));
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectException()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
30			$this->expectExceptionMessage($exception->getMessage());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectExceptionMessage()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
31
32			new $entityCollectionClass([]);
33			}
34
35			/**
36			* @test
37			* @dataProvider invalidEntities
38			* @param array $elements
39			*/
40			public function invalidElementByConstructor(array $elements)
41			{
42			/** @var EntityStrictCollection $entityCollectionClass */
43			$entityCollectionClass = $this->entityCollectionClass();
44			$exception = $entityCollectionClass::customInvalidEntityException();
45
46			$this->expectException(get_class($exception));
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectException()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
47			$this->expectExceptionMessage($exception->getMessage());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectExceptionMessage()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
48
49			new $entityCollectionClass($elements);
50			}
51
52			/**
53			* @return array
54			*/
55			public function invalidEntities()
56			{
57			return [
58			[[1]],
59			[['element']],
60			[[new \stdClass()]],
61			[[new \stdClass(), new \stdClass()]],
62			[[null]],
63			[[false]]
64			];
65			}
66
67			/**
68			* @test
69			*/
70			public function invalidAddElement()
71			{
72			/** @var EntityStrictCollection $entityCollectionClass */
73			$entityCollectionClass = $this->entityCollectionClass();
74			$exception = $entityCollectionClass::customInvalidEntityException();
75
76			$this->expectException(get_class($exception));
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectException()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
77			$this->expectExceptionMessage($exception->getMessage());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectExceptionMessage()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
78
79			/** @var EntityStrictCollection $collection */
80			$collection = new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()], true);
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `prophesize()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
81			$collection->add([]);
82			}
83
84			/**
85			* @test
86			*/
87			public function validChildrenElementByConstructor()
88			{
89			/** @var EntityStrictCollection $entityCollectionClass */
90			$entityCollectionClass = $this->entityCollectionClass();
91			$exception = $entityCollectionClass::customInvalidEntityException();
92
93			$this->expectException(get_class($exception));
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectException()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
94			$this->expectExceptionMessage($exception->getMessage());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `expectExceptionMessage()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
95
96			new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()]);
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `prophesize()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
97			}
98
99			/**
100			* @test
101			*/
102			public function validElementByConstructor()
103			{
104			/** @var EntityStrictCollection $entityCollectionClass */
105			$entityCollectionClass = $this->entityCollectionClass();
106
107			/** @var EntityStrictCollection $collection */
108			$collection = new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()], true);
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `prophesize()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
109
110			$this->assertSame(1, $collection->count());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `assertSame()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
111			$this->assertInstanceOf($entityCollectionClass::entityClass(), $collection->current());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `assertInstanceOf()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
112			}
113
114			/**
115			* @test
116			*/
117			public function addTwoValidElement()
118			{
119			/** @var EntityStrictCollection $entityCollectionClass */
120			$entityCollectionClass = $this->entityCollectionClass();
121
122			/** @var EntityStrictCollection $collection */
123			$collection = new $entityCollectionClass([$this->prophesize($entityCollectionClass::entityClass())->reveal()], true);
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `prophesize()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
124
125
126			/** @var \Prophecy\Prophecy\ObjectProphecy $entity */
127			$entity = $this->prophesize($entityCollectionClass::entityClass());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `prophesize()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
128			$collection->add($entity->reveal());
129			$collection->add($entity->reveal());
130
131			$this->assertSame(3, $collection->count());
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `assertSame()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
132			foreach ($collection as $element) {
133			$this->assertInstanceOf($entityCollectionClass::entityClass(), $element);
			0 ignored issues – show Bug introduced 2017-04-12 15:45 UTC by Report Bug Copy Issue Report It seems like `assertInstanceOf()` must be provided by classes using this trait. How about adding it as abstract method to this trait? This check looks for methods that are used by a trait but not required by it. To illustrate, let’s look at the following code example trait Idable { public function equalIds(Idable $other) { return $this->getId() === $other->getId(); } } The trait `Idable` provides a method `equalsId` that in turn relies on the method `getId()`. If this method does not exist on a class mixing in this trait, the method will fail. Adding the `getId()` as an abstract method to the trait will make sure it is available. Loading history...
134			}
135			}
136			}
137

atrapalo / php-tools

Push — master ( fdb8ab...a48276 )

EntityStrictCollectionTester::invalidAddElement() A

Complexity

Size

Duplication

Importance

Duplication Side-by-Side

Filter issues like