|
1
|
|
|
<?php |
|
2
|
|
|
namespace Yoanm\PhpUnitExtended\Listener; |
|
3
|
|
|
|
|
4
|
|
|
/** |
|
5
|
|
|
* @see https://github.com/yoanm/PhpUnitExtended/doc/listener/StrictCoverageListener.md |
|
6
|
|
|
*/ |
|
7
|
|
|
class RiskyToFailedListener extends \PHPUnit_Framework_BaseTestListener |
|
8
|
|
|
{ |
|
9
|
|
|
/** |
|
10
|
|
|
* @param \PHPUnit_Framework_Test $test |
|
11
|
|
|
* @param \Exception $e |
|
12
|
|
|
* @param float $time |
|
13
|
|
|
*/ |
|
14
|
6 |
|
public function addRiskyTest(\PHPUnit_Framework_Test $test, \Exception $e, $time) |
|
15
|
|
|
{ |
|
16
|
|
|
/* Must be PHPUnit_Framework_TestCase instance to have access to "getTestResultObject" method */ |
|
17
|
6 |
|
if ($test instanceof \PHPUnit_Framework_TestCase) { |
|
18
|
6 |
|
$reason = $this->processEvent($test, $e); |
|
19
|
6 |
|
if (null !== $reason) { |
|
20
|
5 |
|
$test->getTestResultObject()->addFailure( |
|
21
|
5 |
|
$test, |
|
22
|
5 |
|
new \PHPUnit_Framework_AssertionFailedError( |
|
23
|
5 |
|
sprintf( |
|
24
|
5 |
|
"Strict mode - %s :\n%s", |
|
25
|
5 |
|
$reason, |
|
26
|
5 |
|
$e->getMessage() |
|
27
|
5 |
|
) |
|
28
|
5 |
|
), |
|
29
|
|
|
$time |
|
30
|
5 |
|
); |
|
31
|
5 |
|
} |
|
32
|
6 |
|
} |
|
33
|
6 |
|
} |
|
34
|
|
|
|
|
35
|
|
|
/** |
|
36
|
|
|
* @param \PHPUnit_Framework_TestCase $test |
|
37
|
|
|
* @param \Exception $e |
|
38
|
|
|
* |
|
39
|
|
|
* @return null|string |
|
40
|
|
|
*/ |
|
41
|
6 |
|
protected function processEvent(\PHPUnit_Framework_TestCase $test, \Exception $e) |
|
|
|
|
|
|
42
|
|
|
{ |
|
43
|
6 |
|
$reason = null; |
|
44
|
6 |
|
switch (true) { |
|
45
|
|
|
/* beStrictAboutOutputDuringTests="true" */ |
|
46
|
6 |
|
case $e instanceof \PHPUnit_Framework_OutputError: |
|
47
|
1 |
|
$reason = 'No output during test'; |
|
48
|
1 |
|
break; |
|
49
|
|
|
/* checkForUnintentionallyCoveredCode="true" */ |
|
50
|
5 |
|
case $e instanceof \PHPUnit_Framework_UnintentionallyCoveredCodeError: |
|
51
|
1 |
|
$reason = 'Executed code must be defined with @covers and @uses annotations'; |
|
52
|
1 |
|
break; |
|
53
|
4 |
|
default: |
|
54
|
4 |
|
if (preg_match('#\-\-\- Global variables before the test#', $e->getMessage())) { |
|
55
|
|
|
/* beStrictAboutChangesToGlobalState="true" (no specific exception) for globals */ |
|
56
|
1 |
|
$reason = 'No global variable manipulation during test'; |
|
57
|
4 |
|
} elseif (preg_match('#\-\-\- Static attributes before the test#', $e->getMessage())) { |
|
58
|
|
|
/* beStrictAboutChangesToGlobalState="true" (no specific exception) for static var */ |
|
59
|
|
|
/* Only when beStrictAboutChangesToGlobalState="true" */ |
|
60
|
1 |
|
$reason = 'No static attribute manipulation during test'; |
|
61
|
3 |
|
} elseif (preg_match('#This test did not perform any assertions#', $e->getMessage())) { |
|
62
|
|
|
/* beStrictAboutTestsThatDoNotTestAnything="true" (no specific exception) */ |
|
63
|
1 |
|
$reason = 'No test that do not test anything'; |
|
64
|
1 |
|
} |
|
65
|
4 |
|
break; |
|
66
|
4 |
|
} |
|
67
|
6 |
|
return $reason; |
|
68
|
|
|
} |
|
69
|
|
|
} |
|
70
|
|
|
|
yoanm /
PhpUnitExtended
This check looks from parameters that have been defined for a function or method, but which are not used in the method body.