Fix mismatched interface

3 files changed, 34 insertions, 15 deletions

diff --git a/tests/Unit/Fetcher/FeedFetcherTest.php b/tests/Unit/Fetcher/FeedFetcherTest.php
index f6ef47602..510e5471d 100644
--- a/tests/Unit/Fetcher/FeedFetcherTest.php
+++ b/tests/Unit/Fetcher/FeedFetcherTest.php
@@ -165,10 +165,16 @@ class FeedFetcherTest extends TestCase
         $this->assertTrue($this->_fetcher->canHandle($url));
     }
 
+    /**
+     * Test if error is thrown when the feed remain the same.
+     *
+     * @expectedException \OCA\News\Fetcher\FetcherException
+     * @expectedExceptionMessage Feed was not modified since last fetch
+     */
     public function testNoFetchIfNotModified()
     {
         $this->_setUpReader($this->_url, false);;
-        $result = $this->_fetcher->fetch($this->_url, false);
+        $result = $this->_fetcher->fetch($this->_url, false, null, null, null);
         $this->assertSame([null, null], $result);
     }
 
@@ -178,7 +184,7 @@ class FeedFetcherTest extends TestCase
         $item = $this->_createItem();
         $feed = $this->_createFeed();
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        $result = $this->_fetcher->fetch($this->_url, false);
+        $result = $this->_fetcher->fetch($this->_url, false, null, null, null);
 
         $this->assertEquals([$feed, [$item]], $result);
     }
@@ -190,7 +196,7 @@ class FeedFetcherTest extends TestCase
         $item = $this->_createItem('audio/ogg');
         $feed = $this->_createFeed();
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        $result = $this->_fetcher->fetch($this->_url, false);
+        $result = $this->_fetcher->fetch($this->_url, false, null, null, null);
 
         $this->assertEquals([$feed, [$item]], $result);
     }
@@ -202,7 +208,7 @@ class FeedFetcherTest extends TestCase
         $item = $this->_createItem('video/ogg');
         $feed = $this->_createFeed();
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        $result = $this->_fetcher->fetch($this->_url, false);
+        $result = $this->_fetcher->fetch($this->_url, false, null, null, null);
 
         $this->assertEquals([$feed, [$item]], $result);
     }
@@ -214,7 +220,7 @@ class FeedFetcherTest extends TestCase
         $feed = $this->_createFeed('de-DE', true);
         $item = $this->_createItem();
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        $result = $this->_fetcher->fetch($this->_url, true);
+        $result = $this->_fetcher->fetch($this->_url, true, null, null, null);
 
         $this->assertEquals([$feed, [$item]], $result);
     }
@@ -230,7 +236,7 @@ class FeedFetcherTest extends TestCase
 
         $item = $this->_createItem();
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        $result = $this->_fetcher->fetch($this->_url, false);
+        $result = $this->_fetcher->fetch($this->_url, false, null, null, null);
 
         $this->assertEquals([$feed, [$item]], $result);
     }
@@ -241,9 +247,7 @@ class FeedFetcherTest extends TestCase
         $this->_createFeed('he-IL');
         $this->_createItem();
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        list($feed, $items) = $this->_fetcher->fetch(
-            $this->_url, false
-        );
+        list($feed, $items) = $this->_fetcher->fetch($this->_url, false, null, null, null);
         $this->assertTrue($items[0]->getRtl());
     }
 
@@ -262,7 +266,7 @@ class FeedFetcherTest extends TestCase
 
 
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        list($feed, $items) = $this->_fetcher->fetch($this->_url, false);
+        list($feed, $items) = $this->_fetcher->fetch($this->_url, false, null, null, null);
         $this->assertSame($items[0]->getPubDate(), 1522180229);
     }
 
@@ -281,7 +285,7 @@ class FeedFetcherTest extends TestCase
 
 
         $this->_mockIterator($this->_feed_mock, [$this->_item_mock]);
-        list($feed, $items) = $this->_fetcher->fetch($this->_url, false);
+        list($feed, $items) = $this->_fetcher->fetch($this->_url, false, null, null, null);
         $this->assertSame($items[0]->getPubDate(), 1519761029);
     }
 
@@ -374,6 +378,7 @@ class FeedFetcherTest extends TestCase
         $this->_response->expects($this->once())
             ->method('isModified')
             ->will($this->returnValue($modified));
+        $this->_location = $url;
 
         if (!$modified) {
             $this->_result->expects($this->never())
@@ -462,6 +467,7 @@ class FeedFetcherTest extends TestCase
 
         $feed->setTitle('&its a title');
         $feed->setLink($this->_feed_link);
+        $feed->setLocation($this->_location);
         $feed->setUrl($this->_url);
         $feed->setLastModified(3);
         $feed->setAdded($this->_time);
diff --git a/tests/Unit/Fetcher/FetcherTest.php b/tests/Unit/Fetcher/FetcherTest.php
index e8c648177..2549ff606 100644
--- a/tests/Unit/Fetcher/FetcherTest.php
+++ b/tests/Unit/Fetcher/FetcherTest.php
@@ -118,7 +118,7 @@ class FetcherTest extends TestCase
     public function testMultipleFetchersOnlyOneShouldHandle()
     {
         $url = 'hi';
-        $return = 'zeas';
+        $return = [];
         $mockFetcher = $this->getMockBuilder(IFeedFetcher::class)
             ->disableOriginalConstructor()
             ->getMock();
diff --git a/tests/Unit/Fetcher/YoutubeFetcherTest.php b/tests/Unit/Fetcher/YoutubeFetcherTest.php
index f53958b8f..80f50b9d6 100644
--- a/tests/Unit/Fetcher/YoutubeFetcherTest.php
+++ b/tests/Unit/Fetcher/YoutubeFetcherTest.php
@@ -13,6 +13,7 @@ namespace OCA\News\Tests\Unit\Fetcher;
 
 use \OCA\News\Db\Feed;
 use OCA\News\Fetcher\FeedFetcher;
+use OCA\News\Fetcher\Fetcher;
 use OCA\News\Fetcher\YoutubeFetcher;
 
 use PHPUnit\Framework\TestCase;
@@ -20,7 +21,18 @@ use PHPUnit\Framework\TestCase;
 class YoutubeFetcherTest extends TestCase
 {
 
+    /**
+     * Mocked fetcher.
+     *
+     * @var Fetcher
+     */
     private $fetcher;
+
+    /**
+     * Mocked Feed Fetcher.
+     *
+     * @var FeedFetcher
+     */
     private $feedFetcher;
 
     public function setUp() 
@@ -52,7 +64,8 @@ class YoutubeFetcherTest extends TestCase
         $transformedUrl = 'http://gdata.youtube.com/feeds/api/playlists/sobo3';
         $favicon = true;
         $modified = 3;
-        $etag = 5;
+        $user = 5;
+        $password = 5;
         $feed = new Feed();
         $feed->setUrl('http://google.de');
         $result = [$feed, []];
@@ -63,10 +76,10 @@ class YoutubeFetcherTest extends TestCase
                 $this->equalTo($transformedUrl),
                 $this->equalTo($favicon),
                 $this->equalTo($modified),
-                $this->equalTo($etag)
+                $this->equalTo($user)
             )
             ->will($this->returnValue($result));
-        $feed = $this->fetcher->fetch($url, $favicon, $modified, $etag);
+        $feed = $this->fetcher->fetch($url, $favicon, $modified, $user, $password);
 
         $this->assertEquals($url, $result[0]->getUrl());
     }