PHP: Fixed incorrect display of skipped tests in the Test Results window for PHPUnit.

[troizet]

Fixed issue #7739.

Example:

<?php

use PHPUnit\Framework\TestCase;

/**
 * Description of testTest
 *
 */
class SkippedTest extends TestCase {

    /**
     * @test
     */
    function testA() {
        self::assertEquals(1, 1);
        return true;
    }


    /**
     * @test
     * @depends testA
     */
    function testA2($varFromTestA) {
        self::assertEquals($varFromTestA, 1);
    }

    /**
     * @test
     */
    function testAB() {
        self::assertEquals(1, 1);
        throw new Exception('This test fails');
    }

    /**
     * @test
     * @depends testAB
     */
    function testAB2($varFromTestA) {
        self::assertEquals($varFromTestA, 1);
    }

    /**
     * @test
     * @depends testAB
     */
    function testAB3($varFromTestA) {
        self::assertEquals($varFromTestA, 1);
    }

    /**
     * @test
     * @skip
     */
    function testSkipped() {
        self::markTestSkipped('Manually skipped');
        self::assertEquals(1, 1);
    }
}

Before:

After:

[troizet]

I noticed that if tests have at least one skipped test and no failed tests, all tests are marked as skipped.
I'm not sure if this behavior is correct, but it is independent of the PHPUnit module.

[tmysik]

The SKIPPED status was added later, thanks for fixing it!

[junichi11]

Thanks!