Invis1ble/assistant

<?php
 
namespace Tests\AppBundle\DataFixtures\Faker\Provider;
 
use DateTime;
 
use Symfony\Bundle\FrameworkBundle\Test\KernelTestCase;
 
use AppBundle\DataFixtures\Faker\Provider\PeriodProvider;
use AppBundle\Entity\Task;
use AppBundle\Entity\Period;
 
/**
 * PeriodProviderTest
 *
 * @author     Max Invis1ble
 * @copyright  (c) 2016, Max Invis1ble
 * @license    http://www.opensource.org/licenses/mit-license.php MIT
 *
 * @group unit
 */
class PeriodProviderTest extends KernelTestCase
{
    /**
     * @return array[]
     */
    public function startedAtProvider(): array
    {
        $nowTimestamp = time();
 
        $task1 = new Task();
        $task1CreatedAt = new DateTime('@' . ($nowTimestamp - 10));
        $task1->setCreatedAt($task1CreatedAt);
 
        $task1Period = new Period();
        $task1Period->setTaskAndAddSelfToPeriodsCollection($task1);
 
        $task2 = new Task();
        $task2CreatedAt = new DateTime('@' . ($nowTimestamp - 10));
        $task2->setCreatedAt($task2CreatedAt);
 
Avoid excessively long variable names like $task2Period1FinishedAt. Keep variable name length under 20.
        $task2Period1FinishedAt = new DateTime('@' . ($nowTimestamp - 1));
 
        (new Period())
            ->setStartedAt(new DateTime('@' . ($nowTimestamp - 5)))
            ->setFinishedAt($task2Period1FinishedAt)
            ->setTaskAndAddSelfToPeriodsCollection($task2);
 
        $task2Period = new Period();
        $task2Period->setTaskAndAddSelfToPeriodsCollection($task2);
 
        return [
            [$task1Period, $task1CreatedAt],
            [$task2Period, $task2Period1FinishedAt],
        ];
    }
 
    /**
     * @param Period   $period
     * @param DateTime $startDateTime
     *
     * @dataProvider startedAtProvider
     */
    public function testStartedAt(Period $period, DateTime $startDateTime)
    {
Avoid using static access to class '\AppBundle\DataFixtures\Faker\Provider\PeriodProvider' in method 'testStartedAt'.
        $startedAt = PeriodProvider::startedAt($period);
 
        $this->assertGreaterThanOrEqual($startDateTime, $startedAt);
        $this->assertLessThanOrEqual(new DateTime(), $startedAt);
    }
}