X-Git-Url: https://git.immae.eu/?a=blobdiff_plain;f=tests%2Fhttp%2FMetadataRetrieverTest.php;h=3c9eaa0e706a40803bee68a0a952028bd7c44e1c;hb=8d8fa898ab09da87d6c5b333ce6c5b7e8d1506d0;hp=2a1838e81cb1ad59e5f7b62a8326127219469cc2;hpb=4cf3564d28dc8e4d08a3e64f09ad045ffbde97ae;p=github%2Fshaarli%2FShaarli.git diff --git a/tests/http/MetadataRetrieverTest.php b/tests/http/MetadataRetrieverTest.php index 2a1838e8..3c9eaa0e 100644 --- a/tests/http/MetadataRetrieverTest.php +++ b/tests/http/MetadataRetrieverTest.php @@ -38,6 +38,7 @@ class MetadataRetrieverTest extends TestCase $remoteTitle = 'Remote Title '; $remoteDesc = 'Sometimes the meta description is relevant.'; $remoteTags = 'abc def'; + $remoteCharset = 'utf-8'; $expectedResult = [ 'title' => $remoteTitle, @@ -45,11 +46,28 @@ class MetadataRetrieverTest extends TestCase 'tags' => $remoteTags, ]; + $this->httpAccess + ->expects(static::once()) + ->method('getCurlHeaderCallback') + ->willReturnCallback( + function (&$charset) use ( + $remoteCharset + ): callable { + return function () use ( + &$charset, + $remoteCharset + ): void { + $charset = $remoteCharset; + }; + } + ) + ; $this->httpAccess ->expects(static::once()) ->method('getCurlDownloadCallback') ->willReturnCallback( function (&$charset, &$title, &$description, &$tags) use ( + $remoteCharset, $remoteTitle, $remoteDesc, $remoteTags @@ -59,11 +77,13 @@ class MetadataRetrieverTest extends TestCase &$title, &$description, &$tags, + $remoteCharset, $remoteTitle, $remoteDesc, $remoteTags ): void { - $charset = 'ISO-8859-1'; + static::assertSame($remoteCharset, $charset); + $title = $remoteTitle; $description = $remoteDesc; $tags = $remoteTags; @@ -75,8 +95,9 @@ class MetadataRetrieverTest extends TestCase ->expects(static::once()) ->method('getHttpResponse') ->with($url, 30, 4194304) - ->willReturnCallback(function($url, $timeout, $maxBytes, $callback): void { - $callback(); + ->willReturnCallback(function($url, $timeout, $maxBytes, $headerCallback, $dlCallback): void { + $headerCallback(); + $dlCallback(); }) ; @@ -102,8 +123,17 @@ class MetadataRetrieverTest extends TestCase ->expects(static::once()) ->method('getCurlDownloadCallback') ->willReturnCallback( - function (&$charset, &$title, &$description, &$tags): callable { - return function () use (&$charset, &$title, &$description, &$tags): void {}; + function (): callable { + return function (): void {}; + } + ) + ; + $this->httpAccess + ->expects(static::once()) + ->method('getCurlHeaderCallback') + ->willReturnCallback( + function (): callable { + return function (): void {}; } ) ; @@ -111,8 +141,9 @@ class MetadataRetrieverTest extends TestCase ->expects(static::once()) ->method('getHttpResponse') ->with($url, 30, 4194304) - ->willReturnCallback(function($url, $timeout, $maxBytes, $callback): void { - $callback(); + ->willReturnCallback(function($url, $timeout, $maxBytes, $headerCallback, $dlCallback): void { + $headerCallback(); + $dlCallback(); }) ;