Risky test messages mask failures

[markstory]

I ran into a situation where I feel that PHPUnit should have marked a test case as failed, but instead marked it as OK.

<?php
class RiskyTest extends PHPUnit_Framework_TestCase {

    public function testFailure() {
        ob_start();
        $this->assertTrue(false);
    }

}

Using phpunit 4.2.2, I ran:

php phpunit.phar RiskyTest
PHPUnit 4.2.2 by Sebastian Bergmann.

R

Time: 22 ms, Memory: 2.50Mb

OK, but incomplete, skipped, or risky tests!
Tests: 1, Assertions: 1, Risky: 1.

I would have expected such a test case to fail.

[whatthejeff]

Thanks, @markstory. This should be fixed in the next stable release.

[markstory]

Thanks @whatthejeff 👍