diff --git a/.gitignore b/.gitignore index 4f3f707..6b443ed 100644 --- a/.gitignore +++ b/.gitignore @@ -4,5 +4,8 @@ composer.phar composer.lock phpunit.xml +.phpunit.result.cache +/tests/Functional/output/output-* sami.phar -.idea/ \ No newline at end of file +.idea/ +/phpunit diff --git a/.travis.yml b/.travis.yml index e368e13..0cdcf37 100644 --- a/.travis.yml +++ b/.travis.yml @@ -12,29 +12,49 @@ cache: - $HOME/.composer/cache - $HOME/.cache -php: - - 5.4 - - 5.5 - - 5.6 - - 7.0 - - 7.1 - - 7.2 - - 7.3 - matrix: include: - php: 5.4 - env: COMPOSER_FLAGS="--prefer-lowest" + env: + - PHPUNIT_VERSION=4 + - COMPOSER_FLAGS="--prefer-lowest" + - php: 5.4 + env: + - PHPUNIT_VERSION=4 + - php: 5.5 + env: + - PHPUNIT_VERSION=4 + - php: 5.6 + - php: 7.0 + - php: 7.1 + - php: 7.2 + - php: 7.3 + - php: 7.4 before_install: - sudo add-apt-repository ppa:mc3man/trusty-media -y - sudo apt-get update -q - composer self-update - - if [ "$COMPOSER_FLAGS" == "--prefer-lowest" ]; then composer require "roave/security-advisories" dev-master --no-update; fi; + - echo "$PHPUNIT_VERSION" + - | + if [ "$PHPUNIT_VERSION" ]; then + composer remove symfony/phpunit-bridge --dev + wget -O phpunit "https://phar.phpunit.de/phpunit-$PHPUNIT_VERSION.phar" + chmod +x phpunit + fi; + - | + if [ "$COMPOSER_FLAGS" == "--prefer-lowest" ]; then + composer require "roave/security-advisories" dev-master --no-update + fi; install: - sudo apt-get install -y ffmpeg - composer update --prefer-dist $COMPOSER_FLAGS script: - - vendor/bin/phpunit --verbose + - | + if [ "$PHPUNIT_VERSION" ]; then + ./phpunit --verbose + else + ./vendor/bin/simple-phpunit --verbose + fi; diff --git a/composer.json b/composer.json index 3c610c2..fda1beb 100644 --- a/composer.json +++ b/composer.json @@ -44,7 +44,7 @@ "require-dev": { "sami/sami": "~1.0", "silex/silex": "~1.0", - "phpunit/phpunit": "^4.8.36" + "symfony/phpunit-bridge": "^5.0.4" }, "autoload": { "psr-0": { diff --git a/phpunit.xml.dist b/phpunit.xml.dist index 3027d1f..8b9403d 100644 --- a/phpunit.xml.dist +++ b/phpunit.xml.dist @@ -7,10 +7,13 @@ convertWarningsToExceptions="true" processIsolation="false" stopOnFailure="false" - syntaxCheck="true" verbose="false" bootstrap="tests/bootstrap.php" > + + + + tests/Unit @@ -21,10 +24,9 @@ - - vendor - tests - + + src/ + diff --git a/src/FFMpeg/Media/Concat.php b/src/FFMpeg/Media/Concat.php index 092e3a3..29126fd 100644 --- a/src/FFMpeg/Media/Concat.php +++ b/src/FFMpeg/Media/Concat.php @@ -94,7 +94,7 @@ class Concat extends AbstractMediaType $fileStream = @fopen($sourcesFile, 'w'); if($fileStream === false) { - throw new ExecutionFailureException('Cannot open the temporary file.'); + throw new RuntimeException('Cannot open the temporary file.'); } $count_videos = 0; diff --git a/tests/BaseTestCase.php b/tests/BaseTestCase.php new file mode 100644 index 0000000..75f8877 --- /dev/null +++ b/tests/BaseTestCase.php @@ -0,0 +1,52 @@ +setExpectedException($exception, $message); + } + } +} else { + class BaseTestCase extends TestCase + { + public function assertScalar($value, $message = '') + { + $this->assertTrue(is_scalar($value), $message); + } + } +} diff --git a/tests/Functional/FFProbeTest.php b/tests/Functional/FFProbeTest.php index b183e08..160793f 100644 --- a/tests/Functional/FFProbeTest.php +++ b/tests/Functional/FFProbeTest.php @@ -25,11 +25,10 @@ class FFProbeTest extends FunctionalTestCase $this->assertFalse($ffprobe->isValid(__DIR__ . '/../files/WrongFile.mp4')); } - /** - * @expectedException FFMpeg\Exception\RuntimeException - */ public function testProbeOnNonExistantFile() { + $this->expectException('\FFMpeg\Exception\RuntimeException'); + $ffprobe = FFProbe::create(); $ffprobe->streams('/path/to/no/file'); } diff --git a/tests/Functional/FunctionalTestCase.php b/tests/Functional/FunctionalTestCase.php index 7f97a12..dc97876 100644 --- a/tests/Functional/FunctionalTestCase.php +++ b/tests/Functional/FunctionalTestCase.php @@ -3,9 +3,9 @@ namespace Tests\FFMpeg\Functional; use FFMpeg\FFMpeg; -use PHPUnit\Framework\TestCase; +use Tests\FFMpeg\BaseTestCase; -abstract class FunctionalTestCase extends TestCase +abstract class FunctionalTestCase extends BaseTestCase { /** * @return FFMpeg diff --git a/tests/Functional/VideoTranscodeTest.php b/tests/Functional/VideoTranscodeTest.php index 1bae7ed..bc5dfe0 100644 --- a/tests/Functional/VideoTranscodeTest.php +++ b/tests/Functional/VideoTranscodeTest.php @@ -71,10 +71,10 @@ class VideoTranscodeTest extends FunctionalTestCase } /** - * @expectedException \FFMpeg\Exception\RuntimeException */ public function testTranscodeInvalidFile() { + $this->expectException('\FFMpeg\Exception\RuntimeException'); $ffmpeg = $this->getFFMpeg(); $ffmpeg->open(__DIR__ . '/../files/UnknownFileTest.ogv'); } @@ -84,7 +84,7 @@ class VideoTranscodeTest extends FunctionalTestCase $ffmpeg = $this->getFFMpeg(); $video = new Video(__DIR__ . '/../files/UnknownFileTest.ogv', $ffmpeg->getFFMpegDriver(), $ffmpeg->getFFProbe()); - $this->setExpectedException('FFMpeg\Exception\RuntimeException'); + $this->expectException('\FFMpeg\Exception\RuntimeException'); $video->save(new X264('aac'), __DIR__ . '/output/output-x264.mp4'); } diff --git a/tests/Unit/Coordinate/DimensionTest.php b/tests/Unit/Coordinate/DimensionTest.php index eed0c80..13854fc 100644 --- a/tests/Unit/Coordinate/DimensionTest.php +++ b/tests/Unit/Coordinate/DimensionTest.php @@ -9,10 +9,10 @@ class DimensionTest extends TestCase { /** * @dataProvider provideInvalidDimensions - * @expectedException FFMpeg\Exception\InvalidArgumentException */ public function testInvalidDimensions($width, $height) { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); new Dimension($width, $height); } diff --git a/tests/Unit/Coordinate/FrameRateTest.php b/tests/Unit/Coordinate/FrameRateTest.php index 60bf98a..da561c8 100644 --- a/tests/Unit/Coordinate/FrameRateTest.php +++ b/tests/Unit/Coordinate/FrameRateTest.php @@ -15,10 +15,10 @@ class FrameRateTest extends TestCase /** * @dataProvider provideInvalidFrameRates - * @expectedException FFMpeg\Exception\InvalidArgumentException */ public function testInvalidFrameRate($value) { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); new FrameRate($value); } diff --git a/tests/Unit/Coordinate/TimeCodeTest.php b/tests/Unit/Coordinate/TimeCodeTest.php index 08a6397..a91b366 100644 --- a/tests/Unit/Coordinate/TimeCodeTest.php +++ b/tests/Unit/Coordinate/TimeCodeTest.php @@ -28,11 +28,9 @@ class TimeCodeTest extends TestCase ); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testFromInvalidString() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); TimeCode::fromString('lalali lala'); } diff --git a/tests/Unit/Driver/FFMpegDriverTest.php b/tests/Unit/Driver/FFMpegDriverTest.php index 7261fe3..ec4acbd 100644 --- a/tests/Unit/Driver/FFMpegDriverTest.php +++ b/tests/Unit/Driver/FFMpegDriverTest.php @@ -41,11 +41,9 @@ class FFMpegDriverTest extends TestCase $this->assertEquals($conf, $ffmpeg->getConfiguration()); } - /** - * @expectedException FFMpeg\Exception\ExecutableNotFoundException - */ public function testCreateFailureThrowsAnException() { + $this->expectException('\FFMpeg\Exception\ExecutableNotFoundException'); FFMpegDriver::create($this->getLoggerMock(), array('ffmpeg.binaries' => '/path/to/nowhere')); } } diff --git a/tests/Unit/Driver/FFProbeDriverTest.php b/tests/Unit/Driver/FFProbeDriverTest.php index e430491..d42074e 100644 --- a/tests/Unit/Driver/FFProbeDriverTest.php +++ b/tests/Unit/Driver/FFProbeDriverTest.php @@ -41,11 +41,9 @@ class FFProbeDriverTest extends TestCase $this->assertEquals($conf, $ffprobe->getConfiguration()); } - /** - * @expectedException FFMpeg\Exception\ExecutableNotFoundException - */ public function testCreateFailureThrowsAnException() { + $this->expectException('\FFMpeg\Exception\ExecutableNotFoundException'); FFProbeDriver::create(array('ffprobe.binaries' => '/path/to/nowhere')); } } diff --git a/tests/Unit/FFMpegServiceProviderTest.php b/tests/Unit/FFMpegServiceProviderTest.php index 66a4004..a15e679 100644 --- a/tests/Unit/FFMpegServiceProviderTest.php +++ b/tests/Unit/FFMpegServiceProviderTest.php @@ -4,9 +4,8 @@ namespace Tests\FFMpeg\Unit; use FFMpeg\FFMpegServiceProvider; use Silex\Application; -use PHPUnit\Framework\TestCase as BaseTestCase; -class FFMpegServiceProviderTest extends BaseTestCase +class FFMpegServiceProviderTest extends TestCase { public function testWithConfig() { @@ -51,7 +50,7 @@ class FFMpegServiceProviderTest extends BaseTestCase ) )); - $this->setExpectedException('FFMpeg\Exception\ExecutableNotFoundException', 'Unable to load FFMpeg'); + $this->expectException('\FFMpeg\Exception\ExecutableNotFoundException', 'Unable to load FFMpeg'); $app['ffmpeg']; } @@ -64,7 +63,7 @@ class FFMpegServiceProviderTest extends BaseTestCase ) )); - $this->setExpectedException('FFMpeg\Exception\ExecutableNotFoundException', 'Unable to load FFProbe'); + $this->expectException('\FFMpeg\Exception\ExecutableNotFoundException', 'Unable to load FFProbe'); $app['ffmpeg.ffprobe']; } } diff --git a/tests/Unit/FFMpegTest.php b/tests/Unit/FFMpegTest.php index 53b0b72..08043d6 100644 --- a/tests/Unit/FFMpegTest.php +++ b/tests/Unit/FFMpegTest.php @@ -8,12 +8,12 @@ use FFMpeg\FFProbe\DataMapping\Stream; class FFMpegTest extends TestCase { - /** - * @expectedException \FFMpeg\Exception\RuntimeException - * @expectedExceptionMessage Unable to probe "/path/to/unknown/file". - */ public function testOpenInvalid() { + $this->expectException( + '\FFMpeg\Exception\RuntimeException', + 'Unable to probe "/path/to/unknown/file"' + ); $ffmpeg = new FFMpeg($this->getFFMpegDriverMock(), $this->getFFProbeMock()); $ffmpeg->open('/path/to/unknown/file'); } @@ -57,11 +57,9 @@ class FFMpegTest extends TestCase $this->assertInstanceOf('FFMpeg\Media\Video', $ffmpeg->open(__FILE__)); } - /** - * @expectedException \FFMpeg\Exception\InvalidArgumentException - */ public function testOpenUnknown() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $ffprobe = $this->getFFProbeMock(); $ffprobe->expects($this->once()) ->method('streams') diff --git a/tests/Unit/FFProbe/DataMapping/StreamTest.php b/tests/Unit/FFProbe/DataMapping/StreamTest.php index 8fa8aec..0eaee6a 100644 --- a/tests/Unit/FFProbe/DataMapping/StreamTest.php +++ b/tests/Unit/FFProbe/DataMapping/StreamTest.php @@ -42,12 +42,12 @@ class StreamTest extends TestCase ); } - /** - * @expectedException FFMpeg\Exception\LogicException - * @expectedExceptionMessage Dimensions can only be retrieved from video streams. - */ public function testGetDimensionsFromAudio() { + $this->expectException( + '\FFMpeg\Exception\LogicException', + 'Dimensions can only be retrieved from video streams.' + ); $stream = new Stream(array('codec_type' => 'audio')); $stream->getDimensions(); } @@ -60,11 +60,13 @@ class StreamTest extends TestCase /** * @dataProvider provideInvalidPropertiesForDimensionsExtraction - * @expectedException FFMpeg\Exception\RuntimeException - * @expectedExceptionMessage Unable to extract dimensions. */ public function testUnableToGetDimensionsFromVideo($properties) { + $this->expectException( + '\FFMpeg\Exception\RuntimeException', + 'Unable to extract dimensions.' + ); $stream = new Stream(array('codec_type' => 'video', 'width' => 960)); $stream->getDimensions(); } diff --git a/tests/Unit/FFProbe/MapperTest.php b/tests/Unit/FFProbe/MapperTest.php index d4904a1..2ed5360 100644 --- a/tests/Unit/FFProbe/MapperTest.php +++ b/tests/Unit/FFProbe/MapperTest.php @@ -20,11 +20,9 @@ class MapperTest extends TestCase $this->assertEquals($expected, $mapper->map($type, $data)); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testMapInvalidArgument() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $mapper = new Mapper(); $mapper->map('cool type', 'data'); } diff --git a/tests/Unit/FFProbe/OptionsTesterTest.php b/tests/Unit/FFProbe/OptionsTesterTest.php index 4f4ef56..4436928 100644 --- a/tests/Unit/FFProbe/OptionsTesterTest.php +++ b/tests/Unit/FFProbe/OptionsTesterTest.php @@ -2,25 +2,28 @@ namespace Tests\FFMpeg\Unit\FFProbe; -use Alchemy\BinaryDriver\Exception\ExecutionFailureException; use Tests\FFMpeg\Unit\TestCase; use FFMpeg\FFProbe\OptionsTester; class OptionsTesterTest extends TestCase { - /** - * @expectedException FFMpeg\Exception\RuntimeException - * @expectedExceptionMessage Your FFProbe version is too old and does not support `-help` option, please upgrade. - */ public function testHasOptionWithOldFFProbe() { + $this->expectException( + '\FFMpeg\Exception\RuntimeException', + 'Your FFProbe version is too old and does not support `-help` option, please upgrade.' + ); $cache = $this->getCacheMock(); + $executionFailerExceptionMock = $this->getMockBuilder('Alchemy\BinaryDriver\Exception\ExecutionFailureException') + ->disableOriginalConstructor() + ->getMock(); + $ffprobe = $this->getFFProbeDriverMock(); $ffprobe->expects($this->once()) ->method('command') ->with(array('-help', '-loglevel', 'quiet')) - ->will($this->throwException(new ExecutionFailureException('Failed to execute'))); + ->will($this->throwException($executionFailerExceptionMock)); $tester = new OptionsTester($ffprobe, $cache); $tester->has('-print_format'); diff --git a/tests/Unit/FFProbe/OutputParserTest.php b/tests/Unit/FFProbe/OutputParserTest.php index e09916d..91b7160 100644 --- a/tests/Unit/FFProbe/OutputParserTest.php +++ b/tests/Unit/FFProbe/OutputParserTest.php @@ -17,11 +17,9 @@ class OutputParserTest extends TestCase $this->assertEquals($expectedOutput, $parser->parse($type, $data)); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testParseWithInvalidArgument() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $parser = new OutputParser(); $parser->parse('comme ca', 'data'); } diff --git a/tests/Unit/FFProbeTest.php b/tests/Unit/FFProbeTest.php index 4530f59..d927ab9 100644 --- a/tests/Unit/FFProbeTest.php +++ b/tests/Unit/FFProbeTest.php @@ -236,11 +236,11 @@ class FFProbeTest extends TestCase } /** - * @expectedException FFMpeg\Exception\RuntimeException * @dataProvider provideProbeMethod */ public function testProbeWithoutShowStreamsAvailable($method) { + $this->expectException('\FFMpeg\Exception\RuntimeException'); $pathfile = __FILE__; $ffprobe = new FFProbe($this->getFFProbeDriverMock(), $this->getCacheMock()); diff --git a/tests/Unit/Filters/Audio/AudioClipTest.php b/tests/Unit/Filters/Audio/AudioClipTest.php index 8087f76..1450e52 100644 --- a/tests/Unit/Filters/Audio/AudioClipTest.php +++ b/tests/Unit/Filters/Audio/AudioClipTest.php @@ -18,7 +18,7 @@ class AudioClipTest extends TestCase { ->will($this->returnCallback(function ($filter) use (&$capturedFilter) { $capturedFilter = $filter; })); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $filters = new AudioFilters($audio); @@ -36,7 +36,7 @@ class AudioClipTest extends TestCase { ->will($this->returnCallback(function ($filter) use (&$capturedFilter) { $capturedFilter = $filter; })); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $filters = new AudioFilters($audio); @@ -44,4 +44,4 @@ class AudioClipTest extends TestCase { $this->assertEquals(array(0 => '-ss', 1 => '00:00:05.00', 2 => '-t', 3 => '00:00:05.00', 4 => '-acodec', 5 => 'copy'), $capturedFilter->apply($audio, $format)); } -} \ No newline at end of file +} diff --git a/tests/Unit/Filters/Audio/AudioMetadataTest.php b/tests/Unit/Filters/Audio/AudioMetadataTest.php index bc1bc13..dc20879 100644 --- a/tests/Unit/Filters/Audio/AudioMetadataTest.php +++ b/tests/Unit/Filters/Audio/AudioMetadataTest.php @@ -18,7 +18,7 @@ class AudioMetadataTest extends TestCase ->will($this->returnCallback(function ($filter) use (&$capturedFilter) { $capturedFilter = $filter; })); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $filters = new AudioFilters($audio); $filters->addMetadata(array('title' => "Hello World")); @@ -36,7 +36,7 @@ class AudioMetadataTest extends TestCase ->will($this->returnCallback(function ($filter) use (&$capturedFilter) { $capturedFilter = $filter; })); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $filters = new AudioFilters($audio); $filters->addMetadata(array('genre' => 'Some Genre', 'artwork' => "/path/to/file.jpg")); @@ -55,7 +55,7 @@ class AudioMetadataTest extends TestCase ->will($this->returnCallback(function ($filter) use (&$capturedFilter) { $capturedFilter = $filter; })); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $filters = new AudioFilters($audio); $filters->addMetadata(); diff --git a/tests/Unit/Filters/Audio/AudioResamplableFilterTest.php b/tests/Unit/Filters/Audio/AudioResamplableFilterTest.php index 1d24393..ffdc3ea 100644 --- a/tests/Unit/Filters/Audio/AudioResamplableFilterTest.php +++ b/tests/Unit/Filters/Audio/AudioResamplableFilterTest.php @@ -16,7 +16,7 @@ class AudioResamplableFilterTest extends TestCase public function testApply() { $audio = $this->getAudioMock(); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $filter = new AudioResamplableFilter(500); $this->assertEquals(array('-ac', 2, '-ar', 500), $filter->apply($audio, $format)); diff --git a/tests/Unit/Filters/Audio/CustomFilterTest.php b/tests/Unit/Filters/Audio/CustomFilterTest.php index 9c527e2..df80f48 100644 --- a/tests/Unit/Filters/Audio/CustomFilterTest.php +++ b/tests/Unit/Filters/Audio/CustomFilterTest.php @@ -12,7 +12,7 @@ class CustomFilterTest extends TestCase public function testApplyCustomFilter() { $audio = $this->getAudioMock(); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $filter = new CustomFilter('whatever i put would end up as a filter'); $this->assertEquals(array('-af', 'whatever i put would end up as a filter'), $filter->apply($audio, $format)); diff --git a/tests/Unit/Filters/FiltersCollectionTest.php b/tests/Unit/Filters/FiltersCollectionTest.php index e10f946..55c09eb 100644 --- a/tests/Unit/Filters/FiltersCollectionTest.php +++ b/tests/Unit/Filters/FiltersCollectionTest.php @@ -13,18 +13,18 @@ class FiltersCollectionTest extends TestCase $coll = new FiltersCollection(); $this->assertCount(0, $coll); - $coll->add($this->getMock('FFMpeg\Filters\FilterInterface')); + $coll->add($this->getMockBuilder('FFMpeg\Filters\FilterInterface')->getMock()); $this->assertCount(1, $coll); - $coll->add($this->getMock('FFMpeg\Filters\FilterInterface')); + $coll->add($this->getMockBuilder('FFMpeg\Filters\FilterInterface')->getMock()); $this->assertCount(2, $coll); } public function testIterator() { $coll = new FiltersCollection(); - $coll->add($this->getMock('FFMpeg\Filters\FilterInterface')); - $coll->add($this->getMock('FFMpeg\Filters\FilterInterface')); + $coll->add($this->getMockBuilder('FFMpeg\Filters\FilterInterface')->getMock()); + $coll->add($this->getMockBuilder('FFMpeg\Filters\FilterInterface')->getMock()); $this->assertInstanceOf('\ArrayIterator', $coll->getIterator()); $this->assertCount(2, $coll->getIterator()); @@ -51,7 +51,7 @@ class FiltersCollectionTest extends TestCase $data = array(); $video = $this->getVideoMock(); - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); foreach ($coll as $filter) { $data = array_merge($data, $filter->apply($video, $format)); diff --git a/tests/Unit/Filters/Video/CropFilterTest.php b/tests/Unit/Filters/Video/CropFilterTest.php index b865c0d..54afadb 100644 --- a/tests/Unit/Filters/Video/CropFilterTest.php +++ b/tests/Unit/Filters/Video/CropFilterTest.php @@ -22,7 +22,7 @@ class CropFilterTest extends TestCase ->method('getStreams') ->will($this->returnValue($streams)); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $dimension = new Dimension(200, 150); $point = new Point(25, 35); diff --git a/tests/Unit/Filters/Video/CustomFilterTest.php b/tests/Unit/Filters/Video/CustomFilterTest.php index f3cc047..403c29d 100644 --- a/tests/Unit/Filters/Video/CustomFilterTest.php +++ b/tests/Unit/Filters/Video/CustomFilterTest.php @@ -12,7 +12,7 @@ class CustomFilterTest extends TestCase public function testApplyCustomFilter() { $video = $this->getVideoMock(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $filter = new CustomFilter('whatever i put would end up as a filter'); $this->assertEquals(array('-vf', 'whatever i put would end up as a filter'), $filter->apply($video, $format)); diff --git a/tests/Unit/Filters/Video/ExtractMultipleFramesFilterTest.php b/tests/Unit/Filters/Video/ExtractMultipleFramesFilterTest.php index f121691..23981c2 100644 --- a/tests/Unit/Filters/Video/ExtractMultipleFramesFilterTest.php +++ b/tests/Unit/Filters/Video/ExtractMultipleFramesFilterTest.php @@ -17,7 +17,7 @@ class ExtractMultipleFramesFilterTest extends TestCase $video = $this->getVideoMock(); $pathfile = '/path/to/file'.mt_rand(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('getModulus') ->will($this->returnValue($modulus)); @@ -64,10 +64,8 @@ class ExtractMultipleFramesFilterTest extends TestCase ); } - /** - * @expectedException \FFMpeg\Exception\InvalidArgumentException - */ public function testInvalidFrameFileType() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $filter = new ExtractMultipleFramesFilter('1/1', '/'); $filter->setFrameFileType('webm'); } diff --git a/tests/Unit/Filters/Video/FrameRateFilterTest.php b/tests/Unit/Filters/Video/FrameRateFilterTest.php index b25b88f..8e3693b 100644 --- a/tests/Unit/Filters/Video/FrameRateFilterTest.php +++ b/tests/Unit/Filters/Video/FrameRateFilterTest.php @@ -14,7 +14,7 @@ class FrameRateFilterTest extends TestCase $gop = 42; $video = $this->getVideoMock(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('supportBFrames') ->will($this->returnValue(true)); @@ -31,7 +31,7 @@ class FrameRateFilterTest extends TestCase $gop = 42; $video = $this->getVideoMock(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('supportBFrames') ->will($this->returnValue(false)); diff --git a/tests/Unit/Filters/Video/PadFilterTest.php b/tests/Unit/Filters/Video/PadFilterTest.php index 1d7c85a..04b9ae5 100644 --- a/tests/Unit/Filters/Video/PadFilterTest.php +++ b/tests/Unit/Filters/Video/PadFilterTest.php @@ -18,7 +18,7 @@ class PadFilterTest extends TestCase $video = $this->getVideoMock(); $pathfile = '/path/to/file'.mt_rand(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $streams = new StreamCollection(array( new Stream(array( diff --git a/tests/Unit/Filters/Video/ResizeFilterTest.php b/tests/Unit/Filters/Video/ResizeFilterTest.php index c87a3cb..82eebf8 100644 --- a/tests/Unit/Filters/Video/ResizeFilterTest.php +++ b/tests/Unit/Filters/Video/ResizeFilterTest.php @@ -18,7 +18,7 @@ class ResizeFilterTest extends TestCase $video = $this->getVideoMock(); $pathfile = '/path/to/file'.mt_rand(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('getModulus') ->will($this->returnValue($modulus)); diff --git a/tests/Unit/Filters/Video/RotateFilterTest.php b/tests/Unit/Filters/Video/RotateFilterTest.php index c8aeeb2..7aa956a 100644 --- a/tests/Unit/Filters/Video/RotateFilterTest.php +++ b/tests/Unit/Filters/Video/RotateFilterTest.php @@ -22,7 +22,7 @@ class RotateFilterTest extends TestCase ->method('getStreams') ->will($this->returnValue($streams)); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $filter = new RotateFilter($value); $this->assertEquals(array('-vf', $value, '-metadata:s:v:0', 'rotate=0'), $filter->apply($video, $format)); @@ -48,7 +48,7 @@ class RotateFilterTest extends TestCase $video->expects($this->never()) ->method('getStreams'); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $filter = new RotateFilter($value); $this->assertEquals(array('-vf', $value, '-metadata:s:v:0', 'rotate=0'), $filter->apply($video, $format)); @@ -61,12 +61,12 @@ class RotateFilterTest extends TestCase ); } - /** - * @expectedException \FFMpeg\Exception\InvalidArgumentException - * @expectedExceptionMessage Invalid angle value. - */ public function testApplyInvalidAngle() { + $this->expectException( + '\FFMpeg\Exception\InvalidArgumentException', + 'Invalid angle value.' + ); new RotateFilter('90'); } } diff --git a/tests/Unit/Filters/Video/SynchronizeFilterTest.php b/tests/Unit/Filters/Video/SynchronizeFilterTest.php index 85b2562..7986d76 100644 --- a/tests/Unit/Filters/Video/SynchronizeFilterTest.php +++ b/tests/Unit/Filters/Video/SynchronizeFilterTest.php @@ -10,7 +10,7 @@ class SynchronizeFilterTest extends TestCase public function testApply() { $video = $this->getVideoMock(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $filter = new SynchronizeFilter(); $this->assertEquals(array('-async', '1', '-metadata:s:v:0', 'start_time=0'), $filter->apply($video, $format)); diff --git a/tests/Unit/Filters/Video/WatermarkFilterTest.php b/tests/Unit/Filters/Video/WatermarkFilterTest.php index 4f00095..8fd43a8 100644 --- a/tests/Unit/Filters/Video/WatermarkFilterTest.php +++ b/tests/Unit/Filters/Video/WatermarkFilterTest.php @@ -17,7 +17,7 @@ class WatermarkFilterTest extends TestCase $video = $this->getVideoMock(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $filter = new WatermarkFilter(__DIR__ . '/../../../files/watermark.png'); $this->assertEquals(array('-vf', 'movie='.__DIR__ .'/../../../files/watermark.png [watermark]; [in][watermark] overlay=0:0 [out]'), $filter->apply($video, $format)); @@ -30,7 +30,7 @@ class WatermarkFilterTest extends TestCase public function testDifferentCoordinaates() { $video = $this->getVideoMock(); - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); // test position absolute $filter = new WatermarkFilter(__DIR__ . '/../../../files/watermark.png', array( diff --git a/tests/Unit/Format/Audio/AudioTestCase.php b/tests/Unit/Format/Audio/AudioTestCase.php index 16c0022..79d02e3 100644 --- a/tests/Unit/Format/Audio/AudioTestCase.php +++ b/tests/Unit/Format/Audio/AudioTestCase.php @@ -9,7 +9,11 @@ abstract class AudioTestCase extends TestCase { public function testExtraParams() { - foreach ($this->getFormat()->getExtraParams() as $param) { + $extraParams = $this->getFormat()->getExtraParams(); + + $this->assertIsArray($extraParams); + + foreach ($extraParams as $param) { $this->assertScalar($param); } } @@ -30,11 +34,9 @@ abstract class AudioTestCase extends TestCase } } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testSetInvalidAudioCodec() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $this->getFormat()->setAudioCodec('invalid-random-audio-codec'); } @@ -45,7 +47,7 @@ abstract class AudioTestCase extends TestCase public function testGetAudioKiloBitrate() { - $this->assertInternalType('integer', $this->getFormat()->getAudioKiloBitrate()); + $this->assertIsInt($this->getFormat()->getAudioKiloBitrate()); } public function testSetAudioKiloBitrate() @@ -55,25 +57,21 @@ abstract class AudioTestCase extends TestCase $this->assertEquals(256, $format->getAudioKiloBitrate()); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testSetInvalidKiloBitrate() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $this->getFormat()->setAudioKiloBitrate(0); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testSetNegativeKiloBitrate() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $this->getFormat()->setAudioKiloBitrate(-10); } public function testGetAudioChannels() { - $this->assertInternalType('null', $this->getFormat()->getAudioChannels()); + $this->assertNull($this->getFormat()->getAudioChannels()); } public function testSetAudioChannels() @@ -83,25 +81,21 @@ abstract class AudioTestCase extends TestCase $this->assertEquals(2, $format->getAudioChannels()); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testSetInvalidChannels() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $this->getFormat()->setAudioChannels(0); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testSetNegativeChannels() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $this->getFormat()->setAudioChannels(-10); } public function testCreateProgressListener() { - $media = $this->getMock('FFMpeg\Media\MediaTypeInterface'); + $media = $this->getMockBuilder('FFMpeg\Media\MediaTypeInterface')->getMock(); $media->expects($this->any()) ->method('getPathfile') ->will($this->returnValue(__FILE__)); diff --git a/tests/Unit/Format/Video/VideoTestCase.php b/tests/Unit/Format/Video/VideoTestCase.php index a62c99f..0df42ee 100644 --- a/tests/Unit/Format/Video/VideoTestCase.php +++ b/tests/Unit/Format/Video/VideoTestCase.php @@ -14,7 +14,7 @@ abstract class VideoTestCase extends AudioTestCase public function testSupportBFrames() { - $this->assertInternalType('boolean', $this->getFormat()->supportBFrames()); + $this->assertIsBool($this->getFormat()->supportBFrames()); } public function testSetVideoCodec() @@ -29,7 +29,7 @@ abstract class VideoTestCase extends AudioTestCase public function testGetKiloBitrate() { - $this->assertInternalType('integer', $this->getFormat()->getKiloBitrate()); + $this->assertIsInt($this->getFormat()->getKiloBitrate()); } public function testSetKiloBitrate() @@ -39,11 +39,9 @@ abstract class VideoTestCase extends AudioTestCase $this->assertEquals(2560, $format->getKiloBitrate()); } - /** - * @expectedException FFMpeg\Exception\InvalidArgumentException - */ public function testSetInvalidVideoCodec() { + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $this->getFormat()->setVideoCodec('invalid-random-video-codec'); } @@ -54,7 +52,7 @@ abstract class VideoTestCase extends AudioTestCase public function testCreateProgressListener() { - $media = $this->getMock('FFMpeg\Media\MediaTypeInterface'); + $media = $this->getMockBuilder('FFMpeg\Media\MediaTypeInterface')->getMock(); $media->expects($this->any()) ->method('getPathfile') ->will($this->returnValue(__FILE__)); @@ -72,13 +70,13 @@ abstract class VideoTestCase extends AudioTestCase public function testGetPasses() { - $this->assertInternalType('integer', $this->getFormat()->getPasses()); + $this->assertIsInt($this->getFormat()->getPasses()); $this->assertGreaterThan(0, $this->getFormat()->getPasses()); } public function testGetModulus() { - $this->assertInternalType('integer', $this->getFormat()->getModulus()); + $this->assertIsInt($this->getFormat()->getModulus()); $this->assertGreaterThan(0, $this->getFormat()->getModulus()); $this->assertEquals(0, $this->getFormat()->getModulus() % 2); } diff --git a/tests/Unit/Media/AudioTest.php b/tests/Unit/Media/AudioTest.php index 5120496..ed4dadf 100644 --- a/tests/Unit/Media/AudioTest.php +++ b/tests/Unit/Media/AudioTest.php @@ -2,6 +2,7 @@ namespace Tests\FFMpeg\Unit\Media; +use FFMpeg\Exception\RuntimeException; use FFMpeg\Media\Audio; use Alchemy\BinaryDriver\Exception\ExecutionFailureException; use FFMpeg\Format\AudioInterface; @@ -29,7 +30,7 @@ class AudioTest extends AbstractStreamableTestCase $audio = new Audio(__FILE__, $driver, $ffprobe); $audio->setFiltersCollection($filters); - $filter = $this->getMock('FFMpeg\Filters\Audio\AudioFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Audio\AudioFilterInterface')->getMock(); $filters->expects($this->once()) ->method('add') @@ -50,12 +51,12 @@ class AudioTest extends AbstractStreamableTestCase $audio = new Audio(__FILE__, $driver, $ffprobe); $audio->setFiltersCollection($filters); - $filter = $this->getMock('FFMpeg\Filters\Video\VideoFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Video\VideoFilterInterface')->getMock(); $filters->expects($this->never()) ->method('add'); - $this->setExpectedException('FFMpeg\Exception\InvalidArgumentException'); + $this->expectException('\FFMpeg\Exception\InvalidArgumentException'); $audio->addFilter($filter); } @@ -65,24 +66,24 @@ class AudioTest extends AbstractStreamableTestCase $ffprobe = $this->getFFProbeMock(); $outputPathfile = '/target/file'; - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $format->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') ->will($this->returnValue($configuration)); - $failure = new ExecutionFailureException('failed to encode'); + $failure = new RuntimeException('failed to encode'); $driver->expects($this->once()) ->method('command') ->will($this->throwException($failure)); $audio = new Audio(__FILE__, $driver, $ffprobe); - $this->setExpectedException('FFMpeg\Exception\RuntimeException'); + $this->expectException('\FFMpeg\Exception\RuntimeException'); $audio->save($format, $outputPathfile); } @@ -91,12 +92,12 @@ class AudioTest extends AbstractStreamableTestCase $driver = $this->getFFMpegDriverMock(); $ffprobe = $this->getFFProbeMock(); $outputPathfile = '/target/file'; - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $format->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') @@ -104,7 +105,7 @@ class AudioTest extends AbstractStreamableTestCase $audio = new Audio(__FILE__, $driver, $ffprobe); - $filter = $this->getMock('FFMpeg\Filters\Audio\AudioFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Audio\AudioFilterInterface')->getMock(); $filter->expects($this->once()) ->method('apply') ->with($audio, $format) @@ -138,7 +139,7 @@ class AudioTest extends AbstractStreamableTestCase $driver = $this->getFFMpegDriverMock(); $ffprobe = $this->getFFProbeMock(); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') @@ -180,7 +181,7 @@ class AudioTest extends AbstractStreamableTestCase public function provideSaveData() { - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $format->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -191,7 +192,7 @@ class AudioTest extends AbstractStreamableTestCase ->method('getAudioChannels') ->will($this->returnValue(5)); - $audioFormat = $this->getMock('FFMpeg\Format\AudioInterface'); + $audioFormat = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $audioFormat->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -205,7 +206,7 @@ class AudioTest extends AbstractStreamableTestCase ->method('getAudioCodec') ->will($this->returnValue('patati-patata-audio')); - $formatExtra = $this->getMock('FFMpeg\Format\AudioInterface'); + $formatExtra = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $formatExtra->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array('extra', 'param'))); @@ -216,7 +217,7 @@ class AudioTest extends AbstractStreamableTestCase ->method('getAudioChannels') ->will($this->returnValue(5)); - $listeners = array($this->getMock('Alchemy\BinaryDriver\Listeners\ListenerInterface')); + $listeners = array($this->getMockBuilder('Alchemy\BinaryDriver\Listeners\ListenerInterface')->getMock()); $progressableFormat = $this->getMockBuilder('Tests\FFMpeg\Unit\Media\AudioProg') ->disableOriginalConstructor()->getMock(); @@ -290,7 +291,7 @@ class AudioTest extends AbstractStreamableTestCase $driver = $this->getFFMpegDriverMock(); $ffprobe = $this->getFFProbeMock(); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') @@ -317,7 +318,7 @@ class AudioTest extends AbstractStreamableTestCase $outputPathfile = '/target/file'; - $format = $this->getMock('FFMpeg\Format\AudioInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $format->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array('param'))); diff --git a/tests/Unit/Media/ClipTest.php b/tests/Unit/Media/ClipTest.php index 61e988c..ab1b656 100644 --- a/tests/Unit/Media/ClipTest.php +++ b/tests/Unit/Media/ClipTest.php @@ -35,7 +35,7 @@ class ClipTest extends AbstractMediaTestCase $outputPathfile = '/target/file'; - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('getPasses') ->will($this->returnValue(1)); diff --git a/tests/Unit/Media/ConcatTest.php b/tests/Unit/Media/ConcatTest.php index d345961..28de35e 100644 --- a/tests/Unit/Media/ConcatTest.php +++ b/tests/Unit/Media/ConcatTest.php @@ -34,7 +34,7 @@ class ConcatTest extends AbstractMediaTestCase ->disableOriginalConstructor() ->getMock(); - $filter = $this->getMock('FFMpeg\Filters\Concat\ConcatFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Concat\ConcatFilterInterface')->getMock(); $filters->expects($this->once()) ->method('add') @@ -115,7 +115,7 @@ class ConcatTest extends AbstractMediaTestCase array_push($commands, $pathfile); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') @@ -136,9 +136,9 @@ class ConcatTest extends AbstractMediaTestCase array( '-i', __FILE__, '-i', 'concat-2.mp4', - '-filter_complex', + '-filter_complex', '[0:v:0] [0:a:0] [1:v:0] [1:a:0] concat=n=2:v=1:a=1 [v] [a]', - '-map', '[v]', + '-map', '[v]', '-map', '[a]' ), ), diff --git a/tests/Unit/Media/FrameTest.php b/tests/Unit/Media/FrameTest.php index 4c96527..88c0bef 100644 --- a/tests/Unit/Media/FrameTest.php +++ b/tests/Unit/Media/FrameTest.php @@ -36,7 +36,7 @@ class FrameTest extends AbstractMediaTestCase ->disableOriginalConstructor() ->getMock(); - $filter = $this->getMock('FFMpeg\Filters\Frame\FrameFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Frame\FrameFilterInterface')->getMock(); $filters->expects($this->once()) ->method('add') @@ -78,7 +78,7 @@ class FrameTest extends AbstractMediaTestCase $frame->save($pathfile, $accurate, $base64); } } - + public function provideSaveOptions() { return array( diff --git a/tests/Unit/Media/GifTest.php b/tests/Unit/Media/GifTest.php index 4925953..a2dfe91 100644 --- a/tests/Unit/Media/GifTest.php +++ b/tests/Unit/Media/GifTest.php @@ -51,7 +51,7 @@ class GifTest extends AbstractMediaTestCase ->disableOriginalConstructor() ->getMock(); - $filter = $this->getMock('FFMpeg\Filters\Gif\GifFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Gif\GifFilterInterface')->getMock(); $filters->expects($this->once()) ->method('add') diff --git a/tests/Unit/Media/VideoTest.php b/tests/Unit/Media/VideoTest.php index 009a2f6..03c42de 100644 --- a/tests/Unit/Media/VideoTest.php +++ b/tests/Unit/Media/VideoTest.php @@ -2,6 +2,7 @@ namespace Tests\FFMpeg\Unit\Media; +use FFMpeg\Exception\RuntimeException; use FFMpeg\Media\Video; use Alchemy\BinaryDriver\Exception\ExecutionFailureException; use FFMpeg\Format\VideoInterface; @@ -29,7 +30,7 @@ class VideoTest extends AbstractStreamableTestCase $video = new Video(__FILE__, $driver, $ffprobe); $video->setFiltersCollection($filters); - $filter = $this->getMock('FFMpeg\Filters\Video\VideoFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Video\VideoFilterInterface')->getMock(); $filters->expects($this->once()) ->method('add') @@ -50,7 +51,7 @@ class VideoTest extends AbstractStreamableTestCase $video = new Video(__FILE__, $driver, $ffprobe); $video->setFiltersCollection($filters); - $filter = $this->getMock('FFMpeg\Filters\Audio\AudioFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Audio\AudioFilterInterface')->getMock(); $filters->expects($this->once()) ->method('add') @@ -79,7 +80,7 @@ class VideoTest extends AbstractStreamableTestCase $driver = $this->getFFMpegDriverMock(); $ffprobe = $this->getFFProbeMock(); $outputPathfile = '/target/file'; - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('getPasses') ->will($this->returnValue(1)); @@ -87,19 +88,19 @@ class VideoTest extends AbstractStreamableTestCase ->method('getExtraParams') ->will($this->returnValue(array())); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') ->will($this->returnValue($configuration)); - $failure = new ExecutionFailureException('failed to encode'); + $failure = new RuntimeException('failed to encode'); $driver->expects($this->once()) ->method('command') ->will($this->throwException($failure)); $video = new Video(__FILE__, $driver, $ffprobe); - $this->setExpectedException('FFMpeg\Exception\RuntimeException'); + $this->expectException('\FFMpeg\Exception\RuntimeException'); $video->save($format, $outputPathfile); } @@ -108,7 +109,7 @@ class VideoTest extends AbstractStreamableTestCase $driver = $this->getFFMpegDriverMock(); $ffprobe = $this->getFFProbeMock(); $outputPathfile = '/target/file'; - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -116,7 +117,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getPasses') ->will($this->returnValue(2)); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') @@ -124,7 +125,7 @@ class VideoTest extends AbstractStreamableTestCase $video = new Video(__FILE__, $driver, $ffprobe); - $filter = $this->getMock('FFMpeg\Filters\Video\VideoFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Video\VideoFilterInterface')->getMock(); $filter->expects($this->once()) ->method('apply') ->with($video, $format) @@ -158,7 +159,7 @@ class VideoTest extends AbstractStreamableTestCase $driver = $this->getFFMpegDriverMock(); $ffprobe = $this->getFFProbeMock(); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') @@ -233,7 +234,7 @@ class VideoTest extends AbstractStreamableTestCase public function provideSaveData() { - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -253,7 +254,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getAdditionalParameters') ->will($this->returnValue(array('foo', 'bar'))); - $format2 = $this->getMock('FFMpeg\Format\VideoInterface'); + $format2 = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format2->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -273,7 +274,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getAdditionalParameters') ->will($this->returnValue(array('foo', 'bar'))); - $audioFormat = $this->getMock('FFMpeg\Format\AudioInterface'); + $audioFormat = $this->getMockBuilder('FFMpeg\Format\AudioInterface')->getMock(); $audioFormat->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -290,7 +291,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getPasses') ->will($this->returnValue(1)); - $audioVideoFormat = $this->getMock('FFMpeg\Format\VideoInterface'); + $audioVideoFormat = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $audioVideoFormat->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -316,7 +317,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getAdditionalParameters') ->will($this->returnValue(array())); - $audioVideoFormatSinglePass = $this->getMock('FFMpeg\Format\VideoInterface'); + $audioVideoFormatSinglePass = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $audioVideoFormatSinglePass->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array())); @@ -342,7 +343,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getAdditionalParameters') ->will($this->returnValue(array())); - $formatExtra = $this->getMock('FFMpeg\Format\VideoInterface'); + $formatExtra = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $formatExtra->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array('extra', 'param'))); @@ -362,7 +363,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getAdditionalParameters') ->will($this->returnValue(array())); - $formatExtra2 = $this->getMock('FFMpeg\Format\VideoInterface'); + $formatExtra2 = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $formatExtra2->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array('extra', 'param'))); @@ -382,7 +383,7 @@ class VideoTest extends AbstractStreamableTestCase ->method('getAdditionalParameters') ->will($this->returnValue(array())); - $listeners = array($this->getMock('Alchemy\BinaryDriver\Listeners\ListenerInterface')); + $listeners = array($this->getMockBuilder('Alchemy\BinaryDriver\Listeners\ListenerInterface')->getMock()); $progressableFormat = $this->getMockBuilder('Tests\FFMpeg\Unit\Media\Prog') ->disableOriginalConstructor()->getMock(); @@ -585,7 +586,7 @@ class VideoTest extends AbstractStreamableTestCase $driver = $this->getFFMpegDriverMock(); $ffprobe = $this->getFFProbeMock(); - $configuration = $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + $configuration = $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); $driver->expects($this->any()) ->method('getConfiguration') @@ -612,7 +613,7 @@ class VideoTest extends AbstractStreamableTestCase $outputPathfile = '/target/file'; - $format = $this->getMock('FFMpeg\Format\VideoInterface'); + $format = $this->getMockBuilder('FFMpeg\Format\VideoInterface')->getMock(); $format->expects($this->any()) ->method('getExtraParams') ->will($this->returnValue(array('param'))); diff --git a/tests/Unit/Media/WaveformTest.php b/tests/Unit/Media/WaveformTest.php index 5d13c2e..4151235 100644 --- a/tests/Unit/Media/WaveformTest.php +++ b/tests/Unit/Media/WaveformTest.php @@ -25,7 +25,7 @@ class WaveformTest extends AbstractMediaTestCase ->disableOriginalConstructor() ->getMock(); - $filter = $this->getMock('FFMpeg\Filters\Waveform\WaveformFilterInterface'); + $filter = $this->getMockBuilder('FFMpeg\Filters\Waveform\WaveformFilterInterface')->getMock(); $filters->expects($this->once()) ->method('add') diff --git a/tests/Unit/TestCase.php b/tests/Unit/TestCase.php index 2d366dd..60bd004 100644 --- a/tests/Unit/TestCase.php +++ b/tests/Unit/TestCase.php @@ -2,23 +2,18 @@ namespace Tests\FFMpeg\Unit; -use PHPUnit\Framework\TestCase as BaseTestCase; +use Tests\FFMpeg\BaseTestCase; class TestCase extends BaseTestCase { - public function assertScalar($value) - { - $this->assertTrue(is_scalar($value)); - } - public function getLoggerMock() { - return $this->getMock('Psr\Log\LoggerInterface'); + return $this->getMockBuilder('Psr\Log\LoggerInterface')->getMock(); } public function getCacheMock() { - return $this->getMock('Doctrine\Common\Cache\Cache'); + return $this->getMockBuilder('Doctrine\Common\Cache\Cache')->getMock(); } public function getTimeCodeMock() @@ -86,17 +81,17 @@ class TestCase extends BaseTestCase public function getFFProbeParserMock() { - return $this->getMock('FFMpeg\FFProbe\OutputParserInterface'); + return $this->getMockBuilder('FFMpeg\FFProbe\OutputParserInterface')->getMock(); } public function getFFProbeOptionsTesterMock() { - return $this->getMock('FFMpeg\FFProbe\OptionsTesterInterface'); + return $this->getMockBuilder('FFMpeg\FFProbe\OptionsTesterInterface')->getMock(); } public function getFFProbeMapperMock() { - return $this->getMock('FFMpeg\FFProbe\MapperInterface'); + return $this->getMockBuilder('FFMpeg\FFProbe\MapperInterface')->getMock(); } public function getFFProbeOptionsTesterMockWithOptions(array $options) @@ -114,7 +109,7 @@ class TestCase extends BaseTestCase public function getConfigurationMock() { - return $this->getMock('Alchemy\BinaryDriver\ConfigurationInterface'); + return $this->getMockBuilder('Alchemy\BinaryDriver\ConfigurationInterface')->getMock(); } public function getFormatMock()