Merge pull request #198 from choldgraf/adding_URL_error

Throwing an error when data8.berkeley.edu URL is passed

datascience/tables.py

@@ -117,8 +117,12 @@ def read_table(cls, filepath_or_buffer, *args, **vargs):
         # Look for .csv at the end of the path; use "," as a separator if found
         try:
             path = urllib.parse.urlparse(filepath_or_buffer).path
+            if 'data8.berkeley.edu' in filepath_or_buffer:
+                raise ValueError('data8.berkeley.edu requires authentication, '
+                                 'which is not supported.')
         except AttributeError:
             path = filepath_or_buffer
+
         try:
             if 'sep' not in vargs and path.endswith('.csv'):
                 vargs['sep'] = ','

tests/test_tables.py

@@ -1038,3 +1038,10 @@ def test_array_roundtrip(table):
 
     for (c0, c1) in zip(t.columns, table.columns):
         assert_equal(c0, c1)
+
+
+def test_url_parse():
+    """Test that Tables parses URLs correctly"""
+    with pytest.raises(ValueError):
+        url = 'https://data8.berkeley.edu/something/something/dark/side'
+        Table.read_table(url)