Bug fixes + more tests

composer.json

@@ -36,8 +36,12 @@
   },
   "autoload": {
     "psr-4": {
-      "vipnytt\\": "src/",
-      "vipnytt\\test\\": "test/"
+      "vipnytt\\": "src/"
+    }
+  },
+  "autoload-dev": {
+    "psr-4": {
+      "vipnytt\\SitemapParser\\Tests\\": "tests/"
     }
   }
 }

src/SitemapParser.php

@@ -146,9 +146,7 @@ public function parse($url, $urlContent = null)
             $response = gzdecode($response);
         }
         $sitemapJson = $this->generateXMLObject($response);
-        if ($sitemapJson === false && empty($response)) {
-            throw new SitemapParserException("The XML found on the given URL doesn't appear to be valid according to simplexml_load_string/libxml");
-        } elseif ($sitemapJson === false) {
+        if ($sitemapJson === false) {
             $this->parseString($response);
             return;
         }
@@ -188,9 +186,6 @@ protected function getContent()
             }
             $client = new GuzzleHttp\Client();
             $res = $client->request('GET', $this->currentURL, $this->config['guzzle']);
-            if ($res->getStatusCode() < 200 || $res->getStatusCode() >= 400) {
-                throw new SitemapParserException('The server responds with a bad status code: ' . $res->getStatusCode());
-            }
             return $res->getBody();
         } catch (GuzzleHttp\Exception\TransferException $e) {
             throw new SitemapParserException($e->getMessage());

tests/DownloadTest.php

@@ -44,6 +44,9 @@ function generateDataForTest()
             ],
             [
                 'http://php.net/sitemap.xml',
+            ],
+            [
+                'https://www.yahoo.com/news/sitemaps/news-sitemap_index_US_en-US.xml.gz',
             ]
         ];
     }

tests/ExceptionEncodingTest.php

@@ -0,0 +1,18 @@
+<?php
+namespace vipnytt\SitemapParser\Tests;
+
+use vipnytt\SitemapParser;
+
+class ExceptionEncodingTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * Test if exception is thrown when extension `simpleXML` is not loaded
+     */
+    public function testExceptionEncoding()
+    {
+        if (!mb_internal_encoding('UTF-8')) {
+            $this->expectException('\vipnytt\SitemapParser\Exceptions\SitemapParserException');
+            new SitemapParser('SitemapParser');
+        }
+    }
+}

tests/ExceptionMBStringTest.php

@@ -0,0 +1,18 @@
+<?php
+namespace vipnytt\SitemapParser\Tests;
+
+use vipnytt\SitemapParser;
+
+class ExceptionMBStringTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * Test if exception is thrown when extension `simpleXML` is not loaded
+     */
+    public function testExceptionMBString()
+    {
+        if (!extension_loaded('mbstring')) {
+            $this->expectException('\vipnytt\SitemapParser\Exceptions\SitemapParserException');
+            new SitemapParser('SitemapParser');
+        }
+    }
+}

tests/ExceptionSimpleXMLTest.php

@@ -0,0 +1,18 @@
+<?php
+namespace vipnytt\SitemapParser\Tests;
+
+use vipnytt\SitemapParser;
+
+class ExceptionSimpleXMLTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * Test if exception is thrown when extension `simpleXML` is not loaded
+     */
+    public function testExceptionSimpleXML()
+    {
+        if (!extension_loaded('simplexml')) {
+            $this->expectException('\vipnytt\SitemapParser\Exceptions\SitemapParserException');
+            new SitemapParser('SitemapParser');
+        }
+    }
+}

tests/InvalidURLTest.php

@@ -0,0 +1,38 @@
+<?php
+namespace vipnytt\SitemapParser\Tests;
+
+use vipnytt\SitemapParser;
+
+class InvalidURLTest extends \PHPUnit_Framework_TestCase
+{
+    /**
+     * @dataProvider generateDataForTest
+     * @param string $url URL
+     */
+    public function testInvalidURL($url)
+    {
+        $this->expectException('\vipnytt\SitemapParser\Exceptions\SitemapParserException');
+        $parser = new SitemapParser('SitemapParser');
+        $this->assertInstanceOf('vipnytt\SitemapParser', $parser);
+        $parser->parse($url);
+    }
+
+    /**
+     * Generate test data
+     * @return array
+     */
+    public function generateDataForTest()
+    {
+        return [
+            [
+                'htt://www.example.c/',
+            ],
+            [
+                'http:/www.example.com/',
+            ],
+            [
+                'https//www.example.com/',
+            ]
+        ];
+    }
+}