Merge pull request #90 from alexdebril/issue/66

Fix #66
alexdebril · Jun 3, 2017 · 8c529b9 · 8c529b9
2 parents 19deb31 + c044c36
commit 8c529b9
Show file tree

Hide file tree

Showing 2 changed files with 20 additions and 1 deletion.
diff --git a/src/FeedIo/Standard/Rss.php b/src/FeedIo/Standard/Rss.php
@@ -42,6 +42,8 @@ class Rss extends StandardAbstract
      */
     const DATE_NODE_TAGNAME = 'pubDate';
 
+    protected $mandatoryFields = ['channel'];
+
     /**
      * Formats the document according to the standard's specification
      * @param  \DOMDocument $document

diff --git a/tests/FeedIo/ParserTest.php b/tests/FeedIo/ParserTest.php
@@ -12,13 +12,14 @@
 
 use FeedIo\Feed\Item;
 use FeedIo\Rule\DateTimeBuilder;
+use FeedIo\Standard\Rss;
 use Psr\Log\NullLogger;
 
 class ParserTest extends \PHPUnit_Framework_TestCase
 {
 
     /**
-     * @var \FeedIo\ParserAbstract
+     * @var \FeedIo\Parser
      */
     protected $object;
 
@@ -150,6 +151,22 @@ public function testAddResetFilters()
         $this->assertAttributeCount(0, 'filters', $this->object);
     }
 
+    /**
+     * @expectedException \FeedIo\Parser\MissingFieldsException
+     */
+    public function testParseEmptyRssFeed()
+    {
+        $rss = <<<RSS
+<rss version="2.0"></rss>
+RSS;
+        $document = new \DOMDocument();
+        $document->loadXML($rss);
+        $parser = new Parser(new Rss(
+            new DateTimeBuilder()
+        ), new NullLogger());
+        $parser->parse($document, new Feed());
+    }
+
     /**
      * @param  boolean                 $returnValue
      * @return \FeedIo\FilterInterface