ref #2236 - Move status assignment after last occurence of _e AND add…

[Morerice]

I think that this is what @rtwo was referring to in #2236

[codecov-io]

Codecov Report

Merging #2940 into master will not change coverage.
The diff coverage is 100%.

@@            Coverage Diff            @@
##             master    #2940   +/-   ##
=========================================
  Coverage     79.86%   79.86%           
  Complexity     2822     2822           
=========================================
  Files           107      107           
  Lines          7438     7438           
=========================================
  Hits           5940     5940           
  Misses         1498     1498

Impacted Files	Coverage Δ	Complexity Δ
src/Framework/TestCase.php	66.3% <100%> (ø)	311 <0> (ø)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update a19b93d...dc5925a. Read the comment docs.

[keradus]

as a bug, would be great to fix it on lowest maintained branch, not on not-yet-released phpunit 7 (major, bc breakers)

[sebastianbergmann]

Cherry-picked into 6.5 and master, backported to 5.7. Thanks!

[CDJojn]

@bigaltum Is it bug or wanted behavior? We are currently facing this issue, all our failing tests are marked as PASSED (internal state) because of TestCase::hasFailed() returns false...

[sebastianbergmann]

@bigaltum @CDJojn Please open a new ticket and provide a minimal, self-contained, reproducing test case that shows your problem.

Without such a minimal, self-contained, reproducing test case I will not be able to investigate this issue.

[CDJojn]

@sebastianbergmann Thank you for your reply. It seems my issue is duplicate of this bug #3031 - comment added.