AssertJsonResponse::jsonResponse() - Code Metrics - Inspection of "master" - laravelfr/api-testing - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( 6fa789...1e4339 )

by Mathieu

created 2017-07-31 13:50 UTC

AssertJsonResponse::jsonResponse() A

↳ Parent: AssertJsonResponse

Complexity

Conditions	3
Paths	4

Size

Total Lines	8
Code Lines	4

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	4
CRAP Score	3

Importance

Changes

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

<?php

namespace LaravelFr\ApiTesting;

use Illuminate\Foundation\Testing\TestResponse;

trait AssertJsonResponse
{
    use AssertArrays;

    public function __construct()
    {
        if (class_exists(TestResponse::class)) {
            $this->addJsonResponseMacros();
        }
    }

    /**
     * Add the functions to the TestResponse objects.
     */
    public function addJsonResponseMacros()
    {
        $trait = $this;

        TestResponse::macro('seeJsonStructureEquals', function ($structure) use ($trait) {
            return $trait->seeJsonStructureEquals($structure, $this->decodeResponseJson());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        });

        TestResponse::macro('jsonResponse', function ($key = null) use ($trait) {
            return $trait->jsonResponse($key, $this->decodeResponseJson());
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        });
    }

    /**
     * Assert that the JSON response has exactly the given structure.
     *
     * @param  array $structure
     * @param  array|null $responseData
     *
     * @return $this
     */
    public function seeJsonStructureEquals($structure, $responseData = null)
    {
        if (!$responseData) {
            $responseData = $this->decodeResponseJson();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        return $this->assertArrayStructureEquals($structure, $responseData);
    }

    /**
     * Return the json response or a part of it as an array.
     *
     * @param  string $key
     * @param  array|null $responseData
     *
     * @return mixed
     */
    public function jsonResponse($key = null, $responseData = null)
    {
        if (!$responseData) {
            $responseData = $this->decodeResponseJson();
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
        }

        return $key ? array_get($responseData, $key) : $responseData;
    }
}


1		<?php
2
3		namespace LaravelFr\ApiTesting;
4
5		use Illuminate\Foundation\Testing\TestResponse;
6
7		trait AssertJsonResponse
8		{
9		use AssertArrays;
10
11		public function __construct()
12		{
13		if (class_exists(TestResponse::class)) {
14		$this->addJsonResponseMacros();
15		}
16		}
17
18		/**
19		* Add the functions to the TestResponse objects.
20		*/
21	1	public function addJsonResponseMacros()
22		{
23		$trait = $this;
24
25		TestResponse::macro('seeJsonStructureEquals', function ($structure) use ($trait) {
26	1	return $trait->seeJsonStructureEquals($structure, $this->decodeResponseJson());
		1 ignored issue – show Bug introduced 2017-07-31 11:50 UTC by Report Bug Copy Issue Report It seems like `decodeResponseJson()` 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		});
28
29	1	TestResponse::macro('jsonResponse', function ($key = null) use ($trait) {
30	1	return $trait->jsonResponse($key, $this->decodeResponseJson());
		1 ignored issue – show Bug introduced 2017-07-31 11:50 UTC by Report Bug Copy Issue Report It seems like `decodeResponseJson()` 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		}
33
34		/**
35		* Assert that the JSON response has exactly the given structure.
36		*
37		* @param array $structure
38		* @param array\|null $responseData
39		*
40		* @return $this
41		*/
42	2	public function seeJsonStructureEquals($structure, $responseData = null)
43		{
44	2	if (!$responseData) {
45	1	$responseData = $this->decodeResponseJson();
		1 ignored issue – show Bug introduced 2016-12-07 16:15 UTC by Report Bug Copy Issue Report It seems like `decodeResponseJson()` 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...
46		}
47
48	2	return $this->assertArrayStructureEquals($structure, $responseData);
49		}
50
51		/**
52		* Return the json response or a part of it as an array.
53		*
54		* @param string $key
55		* @param array\|null $responseData
56		*
57		* @return mixed
58		*/
59	2	public function jsonResponse($key = null, $responseData = null)
60		{
61	2	if (!$responseData) {
62	1	$responseData = $this->decodeResponseJson();
		1 ignored issue – show Bug introduced 2016-12-07 16:15 UTC by Report Bug Copy Issue Report It seems like `decodeResponseJson()` 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...
63		}
64
65	2	return $key ? array_get($responseData, $key) : $responseData;
66		}
67		}
68

laravelfr / api-testing

Push — master ( 6fa789...1e4339 )

AssertJsonResponse::jsonResponse() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like