Merge pull request #62 from davidastephens/coverage

TST: pragma: no cover where appropriate

Author: davidastephens

pandas_datareader/tests/test_data.py

@@ -12,15 +12,15 @@
 from pandas.util.testing import assert_frame_equal, assert_series_equal
 try:
     from pandas.util.testing import assert_produces_warning
-except ImportError:
+except ImportError: # pragma: no cover
     assert_produces_warning = None
 
 import pandas.util.testing as tm
 from numpy.testing import assert_array_equal
 
 try:
     from urllib.error import HTTPError
-except ImportError:
+except ImportError: # pragma: no cover
     from urllib2 import HTTPError
 
 import pandas_datareader.data as web
@@ -30,7 +30,7 @@
 def _skip_if_no_lxml():
     try:
         import lxml
-    except ImportError:
+    except ImportError: # pragma: no cover
         raise nose.SkipTest("no lxml")
 
 
@@ -50,7 +50,7 @@ class TestGoogle(tm.TestCase):
     def setUpClass(cls):
         super(TestGoogle, cls).setUpClass()
         cls.locales = tm.get_locales(prefix='en_US')
-        if not cls.locales:
+        if not cls.locales: # pragma: no cover
             raise nose.SkipTest("US English locale not available for testing")
 
     @classmethod
@@ -92,7 +92,7 @@ def test_get_multi1(self):
             if (hasattr(pan, 'Close') and hasattr(pan.Close, 'GOOG') and
                 hasattr(pan.Close, 'AAPL')):
                 self.assertEqual(ts[0].dayofyear, 3)
-            else:
+            else: # pragma: no cover
                 self.assertRaises(AttributeError, lambda: pan.Close)
 
     def test_get_multi_invalid(self):
@@ -173,14 +173,14 @@ def test_get_quote_stringlist(self):
         df = web.get_quote_yahoo(['GOOG', 'AAPL', 'GOOG'])
         assert_series_equal(df.ix[0], df.ix[2])
 
-    def test_get_components_dow_jones(self):
+    def test_get_components_dow_jones(self): # pragma: no cover
         raise nose.SkipTest('unreliable test, receive partial components back for dow_jones')
 
         df = web.get_components_yahoo('^DJI') #Dow Jones
         assert isinstance(df, pd.DataFrame)
         self.assertEqual(len(df), 30)
 
-    def test_get_components_dax(self):
+    def test_get_components_dax(self): # pragma: no cover
         raise nose.SkipTest('unreliable test, receive partial components back for dax')
 
         df = web.get_components_yahoo('^GDAXI') #DAX
@@ -189,7 +189,7 @@ def test_get_components_dax(self):
         self.assertEqual(df[df.name.str.contains('adidas', case=False)].index,
                          'ADS.DE')
 
-    def test_get_components_nasdaq_100(self):
+    def test_get_components_nasdaq_100(self): # pragma: no cover
         # as of 7/12/13 the conditional will test false because the link is invalid
         raise nose.SkipTest('unreliable test, receive partial components back for nasdaq_100')
 
@@ -298,8 +298,7 @@ def setUpClass(cls):
         today = datetime.today()
         cls.year = today.year
         cls.month = today.month + 1
-        if cls.month > 12:
-            cls.year = cls.year + 1
+        if cls.month > 12: # pragma: no cover
             cls.month = 1
         cls.expiry = datetime(cls.year, cls.month, 1)
         cls.dirpath = tm.get_data_path()
@@ -320,57 +319,57 @@ def test_get_options_data(self):
 
         try:
             options = self.aapl.get_options_data(expiry=self.expiry)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(options) > 1)
 
     def test_get_near_stock_price(self):
         try:
             options = self.aapl.get_near_stock_price(call=True, put=True,
                                                      expiry=self.expiry)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(options) > 1)
 
     def test_get_call_data(self):
         try:
             calls = self.aapl.get_call_data(expiry=self.expiry)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(calls) > 1)
 
     def test_get_put_data(self):
         try:
             puts = self.aapl.get_put_data(expiry=self.expiry)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(puts) > 1)
 
     def test_get_expiry_dates(self):
         try:
             dates, _ = self.aapl._get_expiry_dates_and_links()
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(dates) > 1)
 
     def test_get_all_data(self):
         try:
             data = self.aapl.get_all_data(put=True)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(data) > 1)
 
     def test_get_data_with_list(self):
         try:
             data = self.aapl.get_call_data(expiry=self.aapl.expiry_dates)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(data) > 1)
 
     def test_get_all_data_calls_only(self):
         try:
             data = self.aapl.get_all_data(call=True, put=False)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assertTrue(len(data) > 1)
 
@@ -381,7 +380,7 @@ def test_get_underlying_price(self):
             url = options_object._yahoo_url_from_expiry(options_object.expiry_dates[0])
             root = options_object._parse_url(url)
             quote_price = options_object._underlying_price_from_root(root)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
         self.assert_(isinstance(quote_price, float))
 
@@ -419,7 +418,7 @@ def test_sample_page_chg_float(self):
     def test_month_year(self):
         try:
             data = self.aapl.get_call_data(month=self.month, year=self.year)
-        except RemoteDataError as e:
+        except RemoteDataError as e: # pragma: no cover
             raise nose.SkipTest(e)
 
         self.assertTrue(len(data) > 1)
@@ -440,7 +439,7 @@ def tearDownClass(cls):
         super(TestOptionsWarnings, cls).tearDownClass()
 
     def test_options_source_warning(self):
-        if not assert_produces_warning:
+        if not assert_produces_warning: # pragma: no cover
             raise nose.SkipTest("old version of pandas without "
                            "compat.assert_produces_warning")
         with assert_produces_warning():
@@ -499,7 +498,7 @@ def test_fred_nan(self):
         df = web.DataReader("DFII5", "fred", start, end)
         assert pd.isnull(df.ix['2010-01-01'][0])
 
-    def test_fred_parts(self):
+    def test_fred_parts(self): # pragma: no cover
         raise nose.SkipTest('buggy as of 2/18/14; maybe a data revision?')
 
         start = datetime(2010, 1, 1)
@@ -524,9 +523,8 @@ def test_invalid_series(self):
         name = "NOT A REAL SERIES"
         self.assertRaises(Exception, web.get_data_fred, name)
 
-    def test_fred_multi(self):
+    def test_fred_multi(self): # pragma: no cover
         raise nose.SkipTest('buggy as of 2/18/14; maybe a data revision?')
-
         names = ['CPIAUCSL', 'CPALTT01USQ661S', 'CPILFESL']
         start = datetime(2010, 1, 1)
         end = datetime(2013, 1, 27)
@@ -546,4 +544,4 @@ def test_fred_multi_bad_series(self):
 
 if __name__ == '__main__':
     nose.runmodule(argv=[__file__, '-vvs', '-x', '--pdb', '--pdb-failure'],
-                   exit=False)
+                   exit=False) # pragma: no cover

pandas_datareader/tests/test_wb.py

@@ -8,7 +8,7 @@
 
 try:
     from pandas.compat import u
-except ImportError:
+except ImportError: # pragma: no cover
     try:
         unicode # python 2
         def u(s):
@@ -81,7 +81,7 @@ def test_wdi_download_w_retired_indicator(self):
         # if it ever gets here, it means WB unretired the indicator.
         # even if they dropped it completely, it would still get caught above
         # or the WB API changed somehow in a really unexpected way.
-        if len(result) > 0:
+        if len(result) > 0: # pragma: no cover
             raise nose.SkipTest("Invalid results")
 
     def test_wdi_download_w_crash_inducing_countrycode(self):
@@ -97,7 +97,7 @@ def test_wdi_download_w_crash_inducing_countrycode(self):
 
         # if it ever gets here, it means the country code XXX got used by WB
         # or the WB API changed somehow in a really unexpected way.
-        if len(result) > 0:
+        if len(result) > 0: # pragma: no cover
             raise nose.SkipTest("Invalid results")
 
     def test_wdi_get_countries(self):
@@ -109,4 +109,4 @@ def test_wdi_get_countries(self):
 
 if __name__ == '__main__':
     nose.runmodule(argv=[__file__, '-vvs', '-x', '--pdb', '--pdb-failure'],
-                   exit=False)
+                   exit=False) # pragma: no cover