io/packers: Just let python raise itself if the file was not found

As opposed to checking if it exists first.

Author: Sam Spilsbury

pandas/io/packers.py

@@ -176,14 +176,8 @@ def read(fh):
 
     # see if we have an actual file
     if isinstance(path_or_buf, str):
-        try:
-            exists = os.path.exists(path_or_buf)
-        except (TypeError, ValueError):
-            exists = False
-
-        if exists:
-            with open(path_or_buf, 'rb') as fh:
-                return read(fh)
+        with open(path_or_buf, 'rb') as fh:
+            return read(fh)
 
     if isinstance(path_or_buf, bytes):
         # treat as a binary-like

pandas/tests/io/test_common.py

@@ -140,7 +140,7 @@ def test_iterator(self):
         (pd.read_stata, 'os', FileNotFoundError, 'dta'),
         (pd.read_sas, 'os', FileNotFoundError, 'sas7bdat'),
         (pd.read_json, 'os', ValueError, 'json'),
-        (pd.read_msgpack, 'os', ValueError, 'mp'),
+        (pd.read_msgpack, 'os', FileNotFoundError, 'mp'),
         (pd.read_pickle, 'os', FileNotFoundError, 'pickle'),
     ])
     def test_read_non_existant(self, reader, module, error_class, fn_ext):
@@ -169,7 +169,7 @@ def test_read_non_existant(self, reader, module, error_class, fn_ext):
         (pd.read_stata, 'os', FileNotFoundError, 'dta'),
         (pd.read_sas, 'os', FileNotFoundError, 'sas7bdat'),
         (pd.read_json, 'os', ValueError, 'json'),
-        (pd.read_msgpack, 'os', ValueError, 'mp'),
+        (pd.read_msgpack, 'os', FileNotFoundError, 'mp'),
         (pd.read_pickle, 'os', FileNotFoundError, 'pickle'),
     ])
     def test_read_expands_user_home_dir(self, reader, module,