Merge pull request #4573 from jtratner/fix-url-parse-call

BUG: Fix incorrect urlparse in _is_s3_url

Author: hayd

pandas/io/common.py

@@ -61,7 +61,7 @@ def _is_url(url):
 def _is_s3_url(url):
     """Check for an s3 url"""
     try:
-        return urlparse.urlparse(url).scheme == 's3'
+        return parse_url(url).scheme == 's3'
     except:
         return False
 

pandas/io/tests/test_data.py

@@ -332,6 +332,10 @@ def test_get_put_data_warning(self):
 
 
 class TestDataReader(unittest.TestCase):
+    def test_is_s3_url(self):
+        from pandas.io.common import _is_s3_url
+        self.assert_(_is_s3_url("s3://pandas/somethingelse.com"))
+
     @network
     def test_read_yahoo(self):
         gs = DataReader("GS", "yahoo")