ArrayRelated::assertSameKeysAndValuesExceptOrder() - Code Metrics - puzzle-org/assert - Measure and Improve Code Quality continuously with Scrutinizer

ArrayRelated::assertSameKeysAndValuesExceptOrder() A
last analyzed 2018-02-19 13:55 UTC

↳ Parent: ArrayRelated

Complexity

Conditions	2
Paths	2

Size

Total Lines	10
Code Lines	5

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	7
CRAP Score	2

Importance

Changes

Metric	Value
dl	0
loc	10
ccs	7
cts	7
cp	1
rs	9.4285
c	0
b	0
f	0
cc	2
eloc	5
nc	2
nop	3
crap	2

<?php

namespace Puzzle\Assert;

trait ArrayRelated
{
    private function assertSameArrayExceptOrder(array $expectedArray, array $array, $message = '')

    {
        $this->assertCount(count($expectedArray), $array, $message);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        foreach($array as $value)
        {
            $index = array_search($value, $expectedArray, true);
            $this->assertNotFalse($index, $message);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

            unset($expectedArray[$index]);
        }
    }

    private function assertSameKeysAndValuesExceptOrder(array $expectedArray, array $array, $message = '')

    {
        $this->assertCount(count($expectedArray), $array, $message);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

        foreach($expectedArray as $key => $value)
        {
            $this->assertArrayHasKey($key, $array, $message);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->assertSame($value, $array[$key], $message);

        }
    }

    private function assertHasKeys(array $expectedKeys, array $array, $message = '')

    {
        foreach($expectedKeys as $key)
        {
            $this->assertArrayHasKey($key, $array, $message);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }
    }
}


1		<?php
2
3		namespace Puzzle\Assert;
4
5		trait ArrayRelated
6		{
7	12	private function assertSameArrayExceptOrder(array $expectedArray, array $array, $message = '')
		0 ignored issues – show Unused Code introduced 2018-02-19 13:55 UTC by Report Bug Copy Issue Report This method is not used, and could be removed. Loading history...
8		{
9	12	$this->assertCount(count($expectedArray), $array, $message);
		0 ignored issues – show Bug introduced 2017-04-15 22:10 UTC by Report Bug Copy Issue Report It seems like `assertCount()` 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...
10
11	9	foreach($array as $value)
12		{
13	9	$index = array_search($value, $expectedArray, true);
14	9	$this->assertNotFalse($index, $message);
		0 ignored issues – show Bug introduced 2017-03-09 17:49 UTC by Report Bug Copy Issue Report It seems like `assertNotFalse()` 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...
15
16	6	unset($expectedArray[$index]);
17	6	}
18	1	}
19
20	5	private function assertSameKeysAndValuesExceptOrder(array $expectedArray, array $array, $message = '')
		0 ignored issues – show Unused Code introduced 2018-02-19 13:55 UTC by Report Bug Copy Issue Report This method is not used, and could be removed. Loading history...
21		{
22	5	$this->assertCount(count($expectedArray), $array, $message);
		0 ignored issues – show Bug introduced 2017-03-09 17:49 UTC by Report Bug Copy Issue Report It seems like `assertCount()` 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...
23
24	4	foreach($expectedArray as $key => $value)
25		{
26	4	$this->assertArrayHasKey($key, $array, $message);
		0 ignored issues – show Bug introduced 2017-03-09 17:49 UTC by Report Bug Copy Issue Report It seems like `assertArrayHasKey()` 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...
27	4	$this->assertSame($value, $array[$key], $message);
		0 ignored issues – show Bug introduced 2017-03-09 17:49 UTC by Report Bug Copy Issue Report The method `assertSame()` does not exist on `Puzzle\Assert\ArrayRelated`. Did you maybe mean `assertSameArrayExceptOrder()`? This check marks calls to methods that do not seem to exist on an object. This is most likely the result of a method being renamed without all references to it being renamed likewise. Loading history...
28	4	}
29	2	}
30
31	47	private function assertHasKeys(array $expectedKeys, array $array, $message = '')
		0 ignored issues – show Unused Code introduced 2018-02-19 13:55 UTC by Report Bug Copy Issue Report This method is not used, and could be removed. Loading history...
32		{
33	47	foreach($expectedKeys as $key)
34		{
35	47	$this->assertArrayHasKey($key, $array, $message);
		0 ignored issues – show Bug introduced 2018-02-19 13:55 UTC by Report Bug Copy Issue Report It seems like `assertArrayHasKey()` 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...
36	43	}
37	40	}
38		}
39

puzzle-org / assert

ArrayRelated::assertSameKeysAndValuesExceptOrder() A last analyzed 2018-02-19 13:55 UTC

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like

ArrayRelated::assertSameKeysAndValuesExceptOrder() A
last analyzed 2018-02-19 13:55 UTC