Zenify / DoctrineMigrations

tests/EventSubscriber/SetConsoleOutputEventSubscriberTest.php

<?php

namespace Zenify\DoctrineMigrations\Tests\EventSubscriber;

use PHPUnit_Framework_Assert;
use PHPUnit_Framework_TestCase;
use Symfony\Component\Console\Application;
use Symfony\Component\Console\Input\ArrayInput;
use Symfony\Component\Console\Output\BufferedOutput;
use Symfony\Component\Console\Output\OutputInterface;
use Zenify\DoctrineMigrations\OutputWriter;
use Zenify\DoctrineMigrations\Tests\ContainerFactory;

As per PSR2, there should be exactly one blank line after the last USE statement, 2 were found though.

final class SetConsoleOutputEventSubscriberTest extends PHPUnit_Framework_TestCase
{

	/**
	 * @var Application
	 */
	private $application;

	/**
	 * @var OutputWriter
	 */
	private $outputWriter;


	protected function setUp()
	{
		$container = (new ContainerFactory)->create();
		$this->application = $container->getByType(Application::class);
		$this->outputWriter = $container->getByType(OutputWriter::class);
	}


	public function testDispatching()
	{
		$this->assertNull(
			PHPUnit_Framework_Assert::getObjectAttribute($this->outputWriter, 'consoleOutput')
		);

		$input = new ArrayInput(['command' => 'migrations:status']);
		$output = new BufferedOutput;
		$this->application->run($input, $output);

		$this->assertInstanceOf(
			OutputInterface::class,
			PHPUnit_Framework_Assert::getObjectAttribute($this->outputWriter, 'consoleOutput')
		);
	}

}

According to PSR2, the closing brace of classes should be placed on the next line directly after the body.