Overwatch\TestBundle\Tests\Command\ConsoleTestHelperTrait

Complexity

Total Complexity

13

Size/Duplication

Total Lines	81
Duplicated Lines	0 %

Coupling/Cohesion

Components	2
Dependencies	1

Importance

Changes	1
Bugs	0	Features	0

7 Methods

Rating	Name	Duplication	Size	Complexity
A	assertHasStandardOutput()	0	13	4
A	assertCountLinesOfOutput()	0	4	1
A	assertCountRunTests()	0	8	2
A	assertResults()	0	8	1
A	assertRecentResultsPersisted()	0	11	2
A	assertRecentResultsNotPersisted()	0	6	1
A	execute()	0	14	2

<?php

namespace Overwatch\TestBundle\Tests\Command;

use Overwatch\TestBundle\DataFixtures\ORM\TestFixtures;

/**
 * ConsoleTestHelperTrait
 */
trait ConsoleTestHelperTrait
{
    protected $output = null;
    
    public function assertHasStandardOutput()
    {
        $this->assertStringStartsWith($this->application->getName(), $this->output[0]);

The property application does not exist. Did you maybe forget to declare it?

It seems like assertStringStartsWith() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        $this->assertContains($this->application->getVersion(), $this->output[0]);

It seems like assertContains() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        
        foreach ($this->output as $lineNum => $line) {
            if ($lineNum === 0 || $lineNum === count($this->output) - 1) {
                continue;
            }
            
            $this->assertStringStartsWith(' > ', $line);

It seems like assertStringStartsWith() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }
    
    /**
     * @param integer $count
     */
    public function assertCountLinesOfOutput($count)
    {
        $this->assertCount($count, $this->output);

The method assertCount() does not exist on Overwatch\TestBundle\Tests\Command\ConsoleTestHelperTrait. Did you maybe mean assertCountLinesOfOutput()?

    }
    
    /**
     * @param integer $count
     */
    public function assertCountRunTests($count = null)
    {
        if ($count === null) {
            $count = count(TestFixtures::$tests);
        }
        
        $this->assertStringEndsWith("$count tests", $this->output[0]);

It seems like assertStringEndsWith() must be provided by classes using this trait. How about adding it as abstract method to this trait?

As per coding-style, please use concatenation or sprintf for the variable $count instead of interpolation.

    }
    
    public function assertResults($failed = '[0-9]+', $error = '[0-9]+', $unsatisfactory = '[0-9]+', $passed = '[0-9]+')
    {
        $this->assertRegExp(

It seems like assertRegExp() must be provided by classes using this trait. How about adding it as abstract method to this trait?

            "/^$failed FAILED, $error ERROR, $unsatisfactory UNSATISFACTORY, $passed PASSED/",

As per coding-style, please use concatenation or sprintf for the variable $failed instead of interpolation.

As per coding-style, please use concatenation or sprintf for the variable $error instead of interpolation.

As per coding-style, please use concatenation or sprintf for the variable $unsatisfactory instead of interpolation.

As per coding-style, please use concatenation or sprintf for the variable $passed instead of interpolation.

            $this->output[count($this->output) - 1]
        );
        $this->assertRegExp('/, in [0-9]+ minutes and [0-9]+ seconds$/i', $this->output[count($this->output) - 1]);

It seems like assertRegExp() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
    
    public function assertRecentResultsPersisted($count = 3)
    {
        $now = (new \DateTime())->getTimestamp();
        $results = $this->resultRepo->getResults([], $count);

The property resultRepo does not exist. Did you maybe forget to declare it?

        
        foreach ($results as $result) {
            $diff = $now - $result->getCreatedAt()->getTimestamp();
            
            $this->assertLessThan(60, $diff);

It seems like assertLessThan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

        }
    }
    
    public function assertRecentResultsNotPersisted()
    {
        $result = $this->resultRepo->getResults([], 1);
        
        $this->assertGreaterThan(500, $result[0]->getCreatedAt()->getTimestamp());

It seems like assertGreaterThan() must be provided by classes using this trait. How about adding it as abstract method to this trait?

    }
    
    protected function execute($params = [], $options = [])
    {
        $params['command'] = self::COMMAND_NAME;
        $returnCode = $this->command->execute($params, $options);

The property command does not exist. Did you maybe forget to declare it?

        
        $this->output = explode(PHP_EOL, $this->command->getDisplay());
        $lastLine = array_pop($this->output);
        
        if (!empty($lastLine)) {
            array_push($this->output, $lastLine);
        }
        
        return $returnCode;
    }
}