Ensure that test has been really run

[milo]

Sometimes can happend that none assertion is done in a test file, e.g. accidental die() before first assertion or just:

class MyTest extends TestCase {
    function testFail() {
        Assert::fail('FAIL');
    }
}

# and there is missing (new MyTest)->run();

... test seems to be passing.

First idea is to count Assert::method() calls, check it in the shutdown function and fail or skip the test. The plus can be nothing-saying statistics :)

[dg]

Missing (new MyTest)->run(); is real problem. Tester was not designed for this usage. But now we have TestCase and it should be somehow solved. Probably by Runner (auto_append_file? or use special loader.php and pass test name as argument?).

Calling assertions seems like good idea.

[fprochazka]

Having it runned by auto_append_file is a good idea, but you cannot auto_append_file, when you run the test directly, not through runner.

[JanTvrdik]

That can be done using http://php.net/manual/en/configuration.file.per-user.php, by I've never tried it.

[dg]

Why so complicated? -d auto_append_file=file.php works well in current Tester.

[dg]

Missing (new MyTest)->run(); is solved by 96b192c.

[milo]

@dg Yop. If you think that checking ordinary tests (not TestCases) is not need it, close it please. Personaly I would check ordinary tests too.

[dg]

The only one case which is not checked is when you forget to call Tester\Environment::setup(), isn't it?

[milo]

Yes.