diff options
Diffstat (limited to 'tests')
-rw-r--r-- | tests/unit/utility/FeedFetcherTest.php | 19 |
1 files changed, 10 insertions, 9 deletions
diff --git a/tests/unit/utility/FeedFetcherTest.php b/tests/unit/utility/FeedFetcherTest.php index ba2e5c191..3794eb1da 100644 --- a/tests/unit/utility/FeedFetcherTest.php +++ b/tests/unit/utility/FeedFetcherTest.php @@ -213,7 +213,7 @@ class FeedFetcherTest extends \OCA\AppFramework\Utility\TestUtility { } - private function createFeed($hasFavicon=false, $hasWebFavicon=false) { + private function createFeed($hasFeedFavicon=false, $hasWebFavicon=false) { $this->expectCore('get_title', $this->feedTitle); $this->expectCore('get_link', $this->feedLink); @@ -224,14 +224,6 @@ class FeedFetcherTest extends \OCA\AppFramework\Utility\TestUtility { $feed->setUrlHash(md5($this->url)); $feed->setAdded($this->time); - if($hasFavicon) { - $this->expectCore('get_image_url', $this->feedImage); - $feed->setFaviconLink($this->feedImage); - } else { - $feed->setFaviconLink(null); - $this->expectCore('get_image_url', null); - } - if($hasWebFavicon) { $this->faviconFetcher->expects($this->once()) ->method('fetch') @@ -240,6 +232,15 @@ class FeedFetcherTest extends \OCA\AppFramework\Utility\TestUtility { $feed->setFaviconLink($this->webFavicon); } + if($hasFeedFavicon) { + $this->expectCore('get_image_url', $this->feedImage); + $feed->setFaviconLink($this->feedImage); + } elseif(!$hasWebFavicon) { + $feed->setFaviconLink(null); + $this->expectCore('get_image_url', null); + } + + return $feed; } |