Times

Complexity

Total Complexity

10

Size/Duplication

Total Lines	62
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

10 Methods

Rating	Name	Duplication	Size	Complexity
A	startTest()	0	5	1
A	endTest()	0	5	1
A	addError()	0	4	1
A	addWarning()	0	4	1
A	addFailure()	0	6	1
A	addIncompleteTest()	0	5	1
A	addRiskyTest()	0	6	1
A	addSkippedTest()	0	7	1
A	startTestSuite()	0	4	1
A	endTestSuite()	0	4	1

<?php
/**
 * Travis CI test script.
 *
 * @copyright YetiForce Sp. z o.o.
 * @license   YetiForce Public License 3.0 (licenses/LicenseEN.txt or yetiforce.com)
 * @author    Mariusz Krzaczkowski <m.krzaczkowski@yetiforce.com>
 * @author    Michał Lorencik <m.lorencik@yetiforce.com>
 */

// @codeCoverageIgnoreStart
class Times implements PHPUnit\Framework\TestListener
{
	public function startTest(PHPUnit\Framework\Test $test)
	{
		//printf("Test %s started.\n", $test->getName());
		echo "\n";
	}

	public function endTest(PHPUnit\Framework\Test $test, $time)
	{
		$time = round($time, 2);

Consider using a different name than the parameter $time. This often makes code more readable.

		echo " Time: $time second(s)";
	}

	public function addError(PHPUnit\Framework\Test $test, Exception $e, $time)
	{
		echo 'Error in ' . $test->getName() . " !!!\n";
	}

	public function addWarning(PHPUnit\Framework\Test $test, PHPUnit\Framework\Warning $e, $time)
	{
		echo 'Warning in ' . $test->getName() . " !!!\n";
	}

	public function addFailure(PHPUnit\Framework\Test $test, PHPUnit\Framework\AssertionFailedError $e, $time)
	{
		echo 'Test ' . $test->getName() . " failed. $time\n";
		echo 'Exception Message: ' . $e->getMessage() . "\n";
		//echo "Exception Trace: " . $e->getTraceAsString() . "\n";
	}

	public function addIncompleteTest(PHPUnit\Framework\Test $test, Exception $e, $time)
	{
		$time = round($time, 2);

Consider using a different name than the parameter $time. This often makes code more readable.

$time is not used, you could remove the assignment.

		printf("addIncompleteTest: Test '%s' is incomplete.\n", $test->getName());
	}

	public function addRiskyTest(PHPUnit\Framework\Test $test, Exception $e, $time)
	{
		printf("Test %s is deemed risky.\n", $test->getName());
		echo 'Exception Message: ' . $e->getMessage() . "\n";
		echo 'Exception Trace: ' . $e->getTraceAsString() . "\n";
	}

	public function addSkippedTest(PHPUnit\Framework\Test $test, Exception $e, $time)
	{
		$time = round($time, 2);

Consider using a different name than the parameter $time. This often makes code more readable.

		printf("Test '%s' has been skipped. ($time second(s))\n", $test->getName());
		echo 'Exception Message: ' . $e->getMessage() . "\n";
		echo 'Exception Trace: ' . $e->getTraceAsString() . "\n";
	}

	public function startTestSuite(PHPUnit\Framework\TestSuite $suite)
	{
		//printf("Started all tests: %s \n", $suite->getName());
	}

	public function endTestSuite(PHPUnit\Framework\TestSuite $suite)
	{
		//printf("Ended all tests: %s.\n", $suite->getName());
	}
}

// @codeCoverageIgnoreEnd