Merge pull request #10609 from sinhrks/fama

TST: test_read_famafrench fails with HTTP 404

Author: sinhrks

doc/source/whatsnew/v0.17.0.txt

@@ -382,3 +382,5 @@ Bug Fixes
 
 
 - Bug in operator equal on Index not being consistent with Series (:issue:`9947`)
+
+- Reading "famafrench" data via ``DataReader`` results in HTTP 404 error because of the website url is changed (:issue:`10591`).

pandas/io/data.py

@@ -504,7 +504,7 @@ def fetch_data(url, name):
 
 def get_data_famafrench(name):
     # path of zip files
-    zip_file_path = '{0}/{1}.zip'.format(_FAMAFRENCH_URL, name)
+    zip_file_path = '{0}/{1}_TXT.zip'.format(_FAMAFRENCH_URL, name)
 
     with urlopen(zip_file_path) as url:
         raw = url.read()

pandas/io/tests/test_data.py

@@ -481,8 +481,6 @@ def test_read_famafrench(self):
         for name in ("F-F_Research_Data_Factors",
                      "F-F_Research_Data_Factors_weekly", "6_Portfolios_2x3",
                      "F-F_ST_Reversal_Factor", "F-F_Momentum_Factor"):
-            raise nose.SkipTest('getting 404 errors as of 7/15/15')
-
             ff = DataReader(name, "famafrench")
             self.assertTrue(ff is not None)
             self.assertIsInstance(ff, dict)