BUG: Fix crash on empty table

Fixes #22

Author: davidastephens

pandas_datareader/data.py

@@ -701,9 +701,6 @@ def _option_frames_from_url(self, url):
         calls = frames[self._TABLE_LOC['calls']]
         puts = frames[self._TABLE_LOC['puts']]
 
-        if len(calls) == 0 or len(puts) == 0:
-            raise RemoteDataError('Received no data from Yahoo at url: %s' % url)
-
         calls = self._process_data(calls, 'call')
         puts = self._process_data(puts, 'put')
 

pandas_datareader/tests/test_data.py

@@ -280,6 +280,7 @@ def setUpClass(cls):
         cls.dirpath = tm.get_data_path()
         cls.html1 = os.path.join(cls.dirpath, 'yahoo_options1.html')
         cls.html2 = os.path.join(cls.dirpath, 'yahoo_options2.html')
+        cls.html3 = os.path.join(cls.dirpath, 'yahoo_options3.html') #Empty table GH#22
         cls.data1 = cls.aapl._option_frames_from_url(cls.html1)['puts']
 
     @classmethod
@@ -398,6 +399,11 @@ def test_month_year(self):
 
         self.assertTrue(len(data) > 1)
 
+    def test_empty_table(self):
+        #GH22
+        empty = self.aapl._option_frames_from_url(self.html3)['puts']
+        self.assertTrue(len(empty) == 0)
+
 
 class TestOptionsWarnings(tm.TestCase):
     @classmethod
@@ -410,7 +416,7 @@ def tearDownClass(cls):
 
     def test_options_source_warning(self):
         if not assert_produces_warning:
-            raise SkipTest("old version of pandas without "
+            raise nose.SkipTest("old version of pandas without "
                            "compat.assert_produces_warning")
         with assert_produces_warning():
             aapl = web.Options('aapl')
@@ -426,6 +432,7 @@ def test_read_yahoo(self):
         assert isinstance(gs, DataFrame)
 
     def test_read_google(self):
+        raise nose.SkipTest("Google API deprecated")
         gs = DataReader("GS", "google")
         assert isinstance(gs, DataFrame)