ErrorHandler::setUpErrorHandler() - Code Metrics - Inspection of "More tests and a typo." - digitickets/phpunit-errorhandler - Measure and Improve Code Quality continuously with Scrutinizer

Completed

Push — master ( a76632...01dfe0 )

by Richard

created 2018-08-02 15:32 UTC

ErrorHandler::setUpErrorHandler() A

↳ Parent: ErrorHandler

Complexity

Conditions	1
Paths	1

Size

Total Lines

Duplication

Lines	0
Ratio	0 %

Code Coverage

Tests	13
CRAP Score	1

Importance

Changes

Metric	Value
dl	0
loc	16
ccs	13
cts	13
cp	1
rs	9.7333
c	0
b	0
f	0
cc	1
nc	1
nop	0
crap	1

<?php

namespace DigiTickets\PHPUnit;

/**
 * Class ErrorHandling.
 *
 * To activate PHPUnit error handling:
 * 1. Add 'use \DigiTickets\PHPUnit\ErrorHandler;' to your unit test.
 *
 * This code is based upon http://www.sitepoint.com/testing-error-conditions-with-phpunit/
 */
trait ErrorHandler
{
    /**
     * @var mixed[]
     */
    private $errors;

    /**
     * Assert that the requested error was generated.
     *
     * @param string $errstr
     * @param int $errno
     */
    public function assertError(string $errstr, int $errno)
    {
        foreach ($this->errors as $error) {
            if ($error['errstr'] === $errstr && $error['errno'] === $errno) {
                $this->assertTrue(true);
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}

                return;
            }
        }
        $this->fail("Error with level {$errno} and message '{$errstr}' not found in ".var_export($this->errors, true));
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
    }

    /**
     * Assert that no errors were generated.
     */
    public function assertNoErrors()
    {
        $this->assertEmpty(
trait Idable {
    public function equalIds(Idable $other) {
        return $this->getId() === $other->getId();
    }
}
            $this->errors,
            sprintf(
                '%s error%s generated.',
                number_format(count($this->errors)),
                count($this->errors) === 1 ? '' : 's'
            )
        );
    }

    /**
     * Activate PHPUnit error handler.
     *
     * @before
     */
    protected function setUpErrorHandler()
    {
        $this->errors = [];
        set_error_handler(
            function (int $errno, string $errstr, string $errfile, int $errline, array $errcontext) {
                $this->errors[] = [
                    'errno' => $errno,
                    'errstr' => $errstr,
                    'errfile' => $errfile,
                    'errline' => $errline,
                    'errcontext' => $errcontext,
                ];
            },
            -1
        );
    }
}


1		<?php
2
3		namespace DigiTickets\PHPUnit;
4
5		/**
6		* Class ErrorHandling.
7		*
8		* To activate PHPUnit error handling:
9		* 1. Add 'use \DigiTickets\PHPUnit\ErrorHandler;' to your unit test.
10		*
11		* This code is based upon http://www.sitepoint.com/testing-error-conditions-with-phpunit/
12		*/
13		trait ErrorHandler
14		{
15		/**
16		* @var mixed[]
17		*/
18		private $errors;
19
20		/**
21		* Assert that the requested error was generated.
22		*
23		* @param string $errstr
24		* @param int $errno
25		*/
26	8	public function assertError(string $errstr, int $errno)
27		{
28	8	foreach ($this->errors as $error) {
29	7	if ($error['errstr'] === $errstr && $error['errno'] === $errno) {
30	7	$this->assertTrue(true);
		0 ignored issues – show Bug introduced 2018-08-02 11:32 UTC by Report Bug Copy Issue Report It seems like `assertTrue()` 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	7	return;
33		}
34		}
35	1	$this->fail("Error with level {$errno} and message '{$errstr}' not found in ".var_export($this->errors, true));
		0 ignored issues – show Bug introduced 2017-02-09 12:32 UTC by Report Bug Copy Issue Report It seems like `fail()` 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		}
37
38		/**
39		* Assert that no errors were generated.
40		*/
41	2	public function assertNoErrors()
42		{
43	2	$this->assertEmpty(
		0 ignored issues – show Bug introduced 2018-08-02 15:33 UTC by Report Bug Copy Issue Report It seems like `assertEmpty()` 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...
44	2	$this->errors,
45	2	sprintf(
46	2	'%s error%s generated.',
47	2	number_format(count($this->errors)),
48	2	count($this->errors) === 1 ? '' : 's'
49		)
50		);
51	1	}
52
53		/**
54		* Activate PHPUnit error handler.
55		*
56		* @before
57		*/
58	10	protected function setUpErrorHandler()
59		{
60	10	$this->errors = [];
61	10	set_error_handler(
62	10	function (int $errno, string $errstr, string $errfile, int $errline, array $errcontext) {
63	8	$this->errors[] = [
64	8	'errno' => $errno,
65	8	'errstr' => $errstr,
66	8	'errfile' => $errfile,
67	8	'errline' => $errline,
68	8	'errcontext' => $errcontext,
69		];
70	10	},
71	10	-1
72		);
73	10	}
74		}
75

digitickets / phpunit-errorhandler

Push — master ( a76632...01dfe0 )

ErrorHandler::setUpErrorHandler() A

Complexity

Size

Duplication

Code Coverage

Importance

Duplication Side-by-Side

Filter issues like