STYLE: Specify bare exceptions in pandas/tests

.pep8speaks.yml

@@ -13,7 +13,6 @@ pycodestyle:
         - W503,  # line break before binary operator
         - W504,  # line break after binary operator
         - E402,  # module level import not at top of file
-        - E722,  # do not use bare except
         - E731,  # do not assign a lambda expression, use a def
         - C406,  # Unnecessary list literal - rewrite as a dict literal.
         - C408,  # Unnecessary dict call - rewrite as a literal.

doc/source/conf.py

@@ -565,7 +565,7 @@ def linkcode_resolve(domain, info):
     for part in fullname.split('.'):
         try:
             obj = getattr(obj, part)
-        except:
+        except AttributeError:
             return None
 
     try:
@@ -574,14 +574,14 @@ def linkcode_resolve(domain, info):
             fn = inspect.getsourcefile(inspect.unwrap(obj))
         else:
             fn = inspect.getsourcefile(obj)
-    except:
+    except TypeError:
         fn = None
     if not fn:
         return None
 
     try:
         source, lineno = inspect.getsourcelines(obj)
-    except:
+    except OSError:
         lineno = None
 
     if lineno:

pandas/conftest.py

@@ -1,14 +1,14 @@
 import importlib
 import os
 
-import hypothesis
-from hypothesis import strategies as st
 import numpy as np
 import pytest
 
 from pandas.compat import PY3
 import pandas.util._test_decorators as td
 
+import hypothesis
+from hypothesis import strategies as st
 import pandas as pd
 
 hypothesis.settings.register_profile(

pandas/tests/indexing/common.py

@@ -156,7 +156,7 @@ def get_result(self, obj, method, key, axis):
         with catch_warnings(record=True):
             try:
                 xp = getattr(obj, method).__getitem__(_axify(obj, key, axis))
-            except:
+            except AttributeError:
                 xp = getattr(obj, method).__getitem__(key)
 
         return xp
@@ -219,7 +219,7 @@ def _print(result, error=None):
 
                 try:
                     xp = self.get_result(obj, method2, k2, a)
-                except:
+                except Exception:
                     result = 'no comp'
                     _print(result)
                     return

pandas/tests/io/formats/test_format.py

@@ -70,7 +70,7 @@ def has_horizontally_truncated_repr(df):
     try:  # Check header row
         fst_line = np.array(repr(df).splitlines()[0].split())
         cand_col = np.where(fst_line == '...')[0][0]
-    except:
+    except IndexError:
         return False
     # Make sure each row has this ... in the same place
     r = repr(df)
@@ -452,7 +452,7 @@ def test_to_string_repr_unicode(self):
         for line in rs[1:]:
             try:
                 line = line.decode(get_option("display.encoding"))
-            except:
+            except AttributeError:
                 pass
             if not line.startswith('dtype:'):
                 assert len(line) == line_len

pandas/tests/io/test_pytables.py

@@ -51,15 +51,15 @@ def safe_remove(path):
     if path is not None:
         try:
             os.remove(path)
-        except:
+        except OSError:
             pass
 
 
 def safe_close(store):
     try:
         if store is not None:
             store.close()
-    except:
+    except IOError:
         pass
 
 
@@ -117,7 +117,7 @@ def _maybe_remove(store, key):
     no content from previous tests using the same table name."""
     try:
         store.remove(key)
-    except:
+    except (ValueError, KeyError):
         pass
 
 
@@ -4605,7 +4605,7 @@ def do_copy(f, new_f=None, keys=None,
                     safe_close(tstore)
                     try:
                         os.close(fd)
-                    except:
+                    except (OSError, ValueError):
                         pass
                     safe_remove(new_f)
 

pandas/tests/io/test_sql.py

@@ -18,28 +18,30 @@
 """
 
 from __future__ import print_function
-import pytest
-import sqlite3
-import csv
 
+import csv
+from datetime import date, datetime, time
+import sqlite3
 import warnings
+
 import numpy as np
-import pandas as pd
+import pytest
 
-from datetime import datetime, date, time
+import pandas.compat as compat
+from pandas.compat import PY36, lrange, range, string_types
 
 from pandas.core.dtypes.common import (
-    is_object_dtype, is_datetime64_dtype,
-    is_datetime64tz_dtype)
-from pandas import DataFrame, Series, Index, MultiIndex, isna, concat
-from pandas import date_range, to_datetime, to_timedelta, Timestamp
-import pandas.compat as compat
-from pandas.compat import range, lrange, string_types, PY36
+    is_datetime64_dtype, is_datetime64tz_dtype, is_object_dtype)
 
-import pandas.io.sql as sql
-from pandas.io.sql import read_sql_table, read_sql_query
+import pandas as pd
+from pandas import (
+    DataFrame, Index, MultiIndex, Series, Timestamp, concat, date_range, isna,
+    to_datetime, to_timedelta)
 import pandas.util.testing as tm
+import pymysql
 
+import pandas.io.sql as sql
+from pandas.io.sql import read_sql_query, read_sql_table
 
 try:
     import sqlalchemy
@@ -1833,7 +1835,7 @@ def test_read_procedure(self):
         try:
             r1 = connection.execute(proc)  # noqa
             trans.commit()
-        except:
+        except pymysql.Error:
             trans.rollback()
             raise
 
@@ -2418,7 +2420,7 @@ def setup_class(cls):
             # No real user should allow root access with a blank password.
             pymysql.connect(host='localhost', user='root', passwd='',
                             db='pandas_nosetest')
-        except:
+        except pymysql.Error:
             pass
         else:
             return
@@ -2445,7 +2447,7 @@ def setup_method(self, request, datapath):
             # No real user should allow root access with a blank password.
             self.conn = pymysql.connect(host='localhost', user='root',
                                         passwd='', db='pandas_nosetest')
-        except:
+        except pymysql.Error:
             pass
         else:
             return

pandas/tests/test_multilevel.py

@@ -1378,7 +1378,7 @@ def f():
 
         try:
             df = f()
-        except:
+        except ValueError:
             pass
         assert (df['foo', 'one'] == 0).all()
 

pandas/tests/test_nanops.py

@@ -141,12 +141,12 @@ def _coerce_tds(targ, res):
             if axis != 0 and hasattr(
                     targ, 'shape') and targ.ndim and targ.shape != res.shape:
                 res = np.split(res, [targ.shape[0]], axis=0)[0]
-        except:
+        except (ValueError, IndexError):
             targ, res = _coerce_tds(targ, res)
 
         try:
             tm.assert_almost_equal(targ, res, check_dtype=check_dtype)
-        except:
+        except AssertionError:
 
             # handle timedelta dtypes
             if hasattr(targ, 'dtype') and targ.dtype == 'm8[ns]':
@@ -167,11 +167,11 @@ def _coerce_tds(targ, res):
                 else:
                     try:
                         res = res.astype('c16')
-                    except:
+                    except RuntimeError:
                         res = res.astype('f8')
                     try:
                         targ = targ.astype('c16')
-                    except:
+                    except RuntimeError:
                         targ = targ.astype('f8')
             # there should never be a case where numpy returns an object
             # but nanops doesn't, so make that an exception

pandas/tests/test_panel.py

@@ -335,13 +335,13 @@ def check_op(op, name):
         for op in ops:
             try:
                 check_op(getattr(operator, op), op)
-            except:
+            except AttributeError:
                 pprint_thing("Failing operation: %r" % op)
                 raise
         if compat.PY3:
             try:
                 check_op(operator.truediv, 'div')
-            except:
+            except AttributeError:
                 pprint_thing("Failing operation: %r" % 'div')
                 raise
 

pandas/tests/test_strings.py

@@ -2630,7 +2630,7 @@ def test_slice(self):
                 expected = Series([s[start:stop:step] if not isna(s) else NA
                                    for s in values])
                 tm.assert_series_equal(result, expected)
-            except:
+            except IndexError:
                 print('failed on %s:%s:%s' % (start, stop, step))
                 raise
 

setup.cfg

@@ -17,7 +17,6 @@ ignore =
     W503,  # line break before binary operator
     W504,  # line break after binary operator
     E402,  # module level import not at top of file
-    E722,  # do not use bare except
     E731,  # do not assign a lambda expression, use a def
     C406,  # Unnecessary list literal - rewrite as a dict literal.
     C408,  # Unnecessary dict call - rewrite as a literal.
@@ -206,7 +205,6 @@ skip=
     pandas/tests/io/test_feather.py,
     pandas/tests/io/test_s3.py,
     pandas/tests/io/test_html.py,
-    pandas/tests/io/test_sql.py,
     pandas/tests/io/test_packers.py,
     pandas/tests/io/test_stata.py,
     pandas/tests/io/conftest.py,