test: refactor tests for remote URLs support

Refs #55

Author: char0n

tests/MovieTest.php

@@ -32,12 +32,7 @@ public static function setUpBeforeClass(): void
         self::$moviePath   = dirname(__FILE__).DIRECTORY_SEPARATOR.'data'.DIRECTORY_SEPARATOR.'test.mp4';
         self::$audioPath   = dirname(__FILE__).DIRECTORY_SEPARATOR.'data'.DIRECTORY_SEPARATOR.'test.wav';
         self::$noMediaPath = dirname(__FILE__).DIRECTORY_SEPARATOR.'data'.DIRECTORY_SEPARATOR.'test1.txt';
-        self::$movieUrl = array(
-            'base' => 'https://github.com/char0n/ffmpeg-php/blob/master/tests/data/',
-            'fileName' => 'test',
-            'fileExtension' => '.mp4',
-            'query' => '?raw=true'
-        );
+        self::$movieUrl = 'https://github.com/char0n/ffmpeg-php/blob/master/tests/data/test.mp4?raw=true';
     }
 
     public static function tearDownAfterClass(): void
@@ -67,20 +62,18 @@ public function testFileDoesNotExistException()
         new Movie(uniqid('test', true));
     }
 
-    public function testFileFromUrlDoesNotExistException()
+    public function testRemoteUrlMovieFile()
+    {
+        $movie = new Movie(self::$movieUrl);
+        $this->assertInstanceOf(Movie::class, $movie);
+    }
+
+    public function testRemoteUrlMovieFileDoesNotExistException()
     {
         $this->expectException(\UnexpectedValueException::class);
         $this->expectExceptionCode(334561);
 
-        $nonExistingUrlFile = sprintf(
-            '%s%s%s%s',
-            self::$movieUrl['base'],
-            uniqid('test', true),
-            self::$movieUrl['fileExtension'],
-            self::$movieUrl['query']
-        );
-
-        new Movie($nonExistingUrlFile);
+        new Movie('https://github.com/char0n/test.mp4');
     }
 
     public function testPersistentResourceSimulation()

tests/OutputProviders/FFMpegProviderTest.php

@@ -20,12 +20,7 @@ public static function setUpBeforeClass(): void
     {
         $path = dirname(__FILE__).DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'data'.DIRECTORY_SEPARATOR;
         self::$moviePath = realpath($path.'test.mp4');
-        self::$movieUrl = array(
-            'base' => 'https://github.com/char0n/ffmpeg-php/blob/master/tests/data/',
-            'fileName' => 'test',
-            'fileExtension' => '.mp4',
-            'query' => '?raw=true'
-        );
+        self::$movieUrl = 'https://github.com/char0n/ffmpeg-php/blob/master/tests/data/test.mp4?raw=true';
     }
 
     public static function tearDownAfterClass(): void
@@ -60,29 +55,22 @@ public function testGetOutputFileDoesNotExist()
         $provider->getOutput();
     }
 
-    public function testGetOutputUrl()
+    public function testGetOutputFromRemoteUrl()
     {
         $provider = new FFMpegProvider();
-        $provider->setMovieFile(implode(self::$movieUrl));
+        $provider->setMovieFile(self::$movieUrl);
         $output = $provider->getOutput();
 
         $this->assertEquals(1, preg_match('/FFmpeg version/i', $output));
     }
 
-    public function testGetOutputUrlFileDoesNotExist()
+    public function testGetOutputFromRemoteUrlDoesNotExist()
     {
         $this->expectException(\UnexpectedValueException::class);
         $this->expectExceptionCode(334561);
 
         $provider = new FFMpegProvider();
-        $nonExistingUrlFile = sprintf(
-            '%s%s%s%s',
-            self::$movieUrl['base'],
-            uniqid('test', true),
-            self::$movieUrl['fileExtension'],
-            self::$movieUrl['query']
-        );
-        $provider->setMovieFile($nonExistingUrlFile);
+        $provider->setMovieFile('https://github.com/char0n/test.mp4');
         $provider->getOutput();
     }
 

tests/OutputProviders/FFProbeProviderTest.php

@@ -20,12 +20,7 @@ public static function setUpBeforeClass(): void
     {
         $path = dirname(__FILE__).DIRECTORY_SEPARATOR.'..'.DIRECTORY_SEPARATOR.'data'.DIRECTORY_SEPARATOR;
         self::$moviePath = realpath($path.'test.mp4');
-        self::$movieUrl = array(
-            'base' => 'https://github.com/char0n/ffmpeg-php/blob/master/tests/data/',
-            'fileName' => 'test',
-            'fileExtension' => '.mp4',
-            'query' => '?raw=true'
-        );
+        self::$movieUrl = 'https://github.com/char0n/ffmpeg-php/blob/master/tests/data/test.mp4?raw=true';
     }
 
     public static function tearDownAfterClass(): void
@@ -60,29 +55,22 @@ public function testGetOutputFileDoesNotExist()
         $provider->getOutput();
     }
 
-    public function testGetOutputUrl()
+    public function testGetOutputFromRemoteUrl()
     {
         $provider = new FFProbeProvider();
-        $provider->setMovieFile(implode(self::$movieUrl));
+        $provider->setMovieFile(self::$movieUrl);
         $output = $provider->getOutput();
 
         $this->assertEquals(1, preg_match('/FFprobe version/i', $output));
     }
 
-    public function testGetOutputUrlFileDoesNotExist()
+    public function testGetOutputFromRemoteUrlDoesNotExist()
     {
         $this->expectException(\UnexpectedValueException::class);
         $this->expectExceptionCode(334561);
 
         $provider = new FFProbeProvider();
-        $nonExistingUrlFile = sprintf(
-            '%s%s%s%s',
-            self::$movieUrl['base'],
-            uniqid('test', true),
-            self::$movieUrl['fileExtension'],
-            self::$movieUrl['query']
-        );
-        $provider->setMovieFile($nonExistingUrlFile);
+        $provider->setMovieFile('https://github.com/char0n/test.mp4');
         $provider->getOutput();
     }