FileFetcher\Tests\Phpunit\ErrorLoggingFileFetcherTest

Complexity

Total Complexity

6

Size/Duplication

Total Lines	61
Duplicated Lines	0 %

Coupling/Cohesion

Components	1
Dependencies	6

Importance

Changes	1
Bugs	0	Features	0

4 Methods

Rating	Name	Duplication	Size	Complexity
A	testGivenSucceedingFileFetcher_itsContentIsReturned()	0	9	1
A	testGivenFailingFileFetcher_anExceptionIsThrown()	0	8	1
A	testGivenFailingFileFetcher_theExceptionIsLogged()	0	20	2
A	testGivenLogLevel_exceptionsAreLoggedAtThisLevel()	0	17	2

<?php

declare( strict_types = 1 );

namespace FileFetcher\Tests\Phpunit;

use FileFetcher\ErrorLoggingFileFetcher;
use FileFetcher\FileFetchingException;
use FileFetcher\InMemoryFileFetcher;
use PHPUnit\Framework\TestCase;
use Psr\Log\LogLevel;
use Psr\Log\NullLogger;
use WMDE\PsrLogTestDoubles\LoggerSpy;

/**
 * @license GNU GPL v2+
 * @author Gabriel Birke < gabriel.birke@wikimedia.de >
 */
class ErrorLoggingFileFetcherTest extends TestCase {

	public function testGivenSucceedingFileFetcher_itsContentIsReturned() {
		$logger = new LoggerSpy();
		$fileFetcher = new ErrorLoggingFileFetcher(
			new InMemoryFileFetcher( [ 'song.txt' => 'I\'m a little teapot' ] ),
			$logger
		);
		$this->assertSame( 'I\'m a little teapot', $fileFetcher->fetchFile( 'song.txt' ) );
		$logger->assertNoLoggingCallsWhereMade();
	}

	public function testGivenFailingFileFetcher_anExceptionIsThrown() {
		$errorLoggingFileFetcher = new ErrorLoggingFileFetcher(
			new InMemoryFileFetcher( [] ),
			new NullLogger()
		);
		$this->expectException( FileFetchingException::class );
		$errorLoggingFileFetcher->fetchFile( 'song.txt' );
	}

	public function testGivenFailingFileFetcher_theExceptionIsLogged() {
		$logger = new LoggerSpy();
		$fileFetcher = new ErrorLoggingFileFetcher(
			new InMemoryFileFetcher( [] ),
			$logger
		);

		// @codingStandardsIgnoreStart
		try {
			$fileFetcher->fetchFile( 'song.txt' );
			$this->fail( 'Should have thrown a FileFetchingException' );
		} catch ( FileFetchingException $e ) {

Consider adding a comment why this CATCH block is empty.

		}
		// @codingStandardsIgnoreEnd

		$calls = $logger->getLogCalls();
		$this->assertCount( 1, $calls );
		$this->assertArrayHasKey( 'exception', $calls->getFirstCall()->getContext() );
		$this->assertSame( LogLevel::ERROR, $calls->getFirstCall()->getLevel() );
	}

	public function testGivenLogLevel_exceptionsAreLoggedAtThisLevel() {
		$logger = new LoggerSpy();
		$fileFetcher = new ErrorLoggingFileFetcher(
			new InMemoryFileFetcher( [] ),
			$logger,
			LogLevel::CRITICAL
		);

		// @codingStandardsIgnoreStart
		try {
			$fileFetcher->fetchFile( 'song.txt' );
		} catch ( FileFetchingException $e ) {

Consider adding a comment why this CATCH block is empty.

		}
		// @codingStandardsIgnoreEnd

		$this->assertSame( LogLevel::CRITICAL, $logger->getLogCalls()->getFirstCall()->getLevel() );
	}

}