TonicHealthCheck\Test\Incident\IncidentCreateTrait

Complexity

Total Complexity

1

Size/Duplication

Total Lines	25
Duplicated Lines	0 %

Coupling/Cohesion

Components	0
Dependencies	0

Importance

Changes

0

1 Method

Rating	Name	Duplication	Size	Complexity
A	createIncidentMock()	0	16	1

<?php

namespace TonicHealthCheck\Test\Incident;

use PHPUnit_Framework_MockObject_MockObject;
use TonicHealthCheck\Incident\IncidentInterface;

trait IncidentCreateTrait
{
    /**
     * @param null|string $iIdent
     * @param null|string $iMessage
     *
     * @return PHPUnit_Framework_MockObject_MockObject|IncidentInterface
     */
    protected function createIncidentMock($iIdent = 'node_name.some_check', $iMessage = 'check error')
    {
        $incident = $this->getMockBuilder(IncidentInterface::class)->getMock();

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

        $incident
            ->expects($this->any())

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

            ->method('getIdent')
            ->willReturn($iIdent);

        $incident
            ->expects($this->any())

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

            ->method('getMessage')
            ->willReturn($iMessage);

        return $incident;
    }
}