|
1
|
|
|
<?php |
|
2
|
|
|
namespace Yoanm\PhpUnitExtended\Listener; |
|
3
|
|
|
|
|
4
|
|
|
use PHPUnit\Framework\AssertionFailedError; |
|
5
|
|
|
use PHPUnit\Framework\OutputError; |
|
6
|
|
|
use PHPUnit\Framework\Test; |
|
7
|
|
|
use PHPUnit\Framework\TestCase; |
|
8
|
|
|
use PHPUnit\Framework\TestListener; |
|
9
|
|
|
use PHPUnit\Framework\TestListenerDefaultImplementation; |
|
10
|
|
|
use PHPUnit\Framework\UnintentionallyCoveredCodeError; |
|
11
|
|
|
use PHPUnit\Framework\Warning; |
|
12
|
|
|
|
|
13
|
|
|
/** |
|
14
|
|
|
* @see doc/listener/StrictCoverageListener.md |
|
15
|
|
|
*/ |
|
16
|
|
|
class RiskyToFailedListener implements TestListener |
|
17
|
|
|
{ |
|
18
|
|
|
use TestListenerDefaultImplementation; |
|
19
|
|
|
|
|
20
|
1 |
|
public function addWarning(Test $test, Warning $e, $time) |
|
21
|
|
|
{ |
|
22
|
1 |
|
$this->addErrorIfNeeded($test, $e, $time); |
|
23
|
1 |
|
} |
|
24
|
|
|
|
|
25
|
|
|
/** |
|
26
|
|
|
* @param Test $test |
|
27
|
|
|
* @param \Exception $e |
|
28
|
|
|
* @param float $time |
|
29
|
|
|
*/ |
|
30
|
6 |
|
public function addRiskyTest(Test $test, \Exception $e, $time) |
|
31
|
|
|
{ |
|
32
|
6 |
|
$this->addErrorIfNeeded($test, $e, $time); |
|
33
|
6 |
|
} |
|
34
|
|
|
|
|
35
|
|
|
/** |
|
36
|
|
|
* @param Test $test |
|
37
|
|
|
* @param \Exception $e |
|
38
|
|
|
* @param $time |
|
39
|
|
|
*/ |
|
40
|
7 |
|
protected function addErrorIfNeeded(Test $test, \Exception $e, $time) |
|
41
|
|
|
{ |
|
42
|
|
|
/* Must beTestCase instance to have access to "getTestResultObject" method */ |
|
43
|
7 |
|
if ($test instanceof TestCase) { |
|
44
|
7 |
|
$reason = $this->getErrorReason($e); |
|
45
|
7 |
|
if (null !== $reason) { |
|
46
|
6 |
|
$test->getTestResultObject()->addFailure( |
|
47
|
6 |
|
$test, |
|
48
|
6 |
|
new AssertionFailedError( |
|
49
|
6 |
|
sprintf( |
|
50
|
6 |
|
"Strict mode - %s :\n%s", |
|
51
|
6 |
|
$reason, |
|
52
|
6 |
|
$e->getMessage() |
|
53
|
|
|
) |
|
54
|
|
|
), |
|
55
|
6 |
|
$time |
|
56
|
|
|
); |
|
57
|
|
|
} |
|
58
|
|
|
} |
|
59
|
7 |
|
} |
|
60
|
|
|
|
|
61
|
|
|
/** |
|
62
|
|
|
* @param \Exception $e |
|
63
|
|
|
* |
|
64
|
|
|
* @return null|string |
|
65
|
|
|
*/ |
|
66
|
7 |
|
protected function getErrorReason(\Exception $e) |
|
67
|
|
|
{ |
|
68
|
7 |
|
$reason = null; |
|
69
|
|
|
switch (true) { |
|
70
|
|
|
/* beStrictAboutOutputDuringTests="true" */ |
|
71
|
7 |
|
case $e instanceof OutputError: |
|
72
|
1 |
|
$reason = 'No output during test'; |
|
73
|
1 |
|
break; |
|
74
|
|
|
/* checkForUnintentionallyCoveredCode="true" */ |
|
75
|
6 |
|
case $e instanceof UnintentionallyCoveredCodeError: |
|
76
|
1 |
|
$reason = 'Executed code must be defined with @covers and @uses annotations'; |
|
77
|
1 |
|
break; |
|
78
|
|
|
default: |
|
79
|
5 |
|
if (preg_match('#\-\-\- Global variables before the test#', $e->getMessage())) { |
|
80
|
|
|
/* beStrictAboutChangesToGlobalState="true" (no specific exception) for globals */ |
|
81
|
1 |
|
$reason = 'No global variable manipulation during test'; |
|
82
|
4 |
|
} elseif (preg_match('#\-\-\- Static attributes before the test#', $e->getMessage())) { |
|
83
|
|
|
/* beStrictAboutChangesToGlobalState="true" (no specific exception) for static var */ |
|
84
|
|
|
/* Only when beStrictAboutChangesToGlobalState="true" */ |
|
85
|
1 |
|
$reason = 'No static attribute manipulation during test'; |
|
86
|
3 |
|
} elseif (preg_match('#This test did not perform any assertions#', $e->getMessage())) { |
|
87
|
|
|
/* beStrictAboutTestsThatDoNotTestAnything="true" (no specific exception) */ |
|
88
|
1 |
|
$reason = 'No test that do not test anything'; |
|
89
|
2 |
|
} elseif (preg_match('#Trying to @cover or @use not existing #', $e->getMessage())) { |
|
90
|
|
|
/* forceCoversAnnotation="true" (no specific exception) */ |
|
91
|
1 |
|
$reason = 'Only executed code must be defined with @covers and @uses annotations'; |
|
92
|
|
|
} |
|
93
|
5 |
|
break; |
|
94
|
|
|
} |
|
95
|
7 |
|
return $reason; |
|
96
|
|
|
} |
|
97
|
|
|
} |
|
98
|
|
|
|