TST: show deprecation config options as FutureWarnings

STYLE: clean some pytest / linting warnings

xref #19004, dateutil deprecation warnings

Author: jreback

appveyor.yml

@@ -15,6 +15,7 @@ environment:
     # See: http://stackoverflow.com/a/13751649/163740
     CMD_IN_ENV: "cmd /E:ON /V:ON /C .\\ci\\run_with_env.cmd"
     clone_folder: C:\projects\pandas
+    PANDAS_TESTING_MODE: "deprecate"
 
   matrix:
 

doc/source/whatsnew/v0.23.0.txt

@@ -207,6 +207,7 @@ Other API Changes
 - :func:`wide_to_long` previously kept numeric-like suffixes as ``object`` dtype. Now they are cast to numeric if possible (:issue:`17627`)
 - In :func:`read_excel`, the ``comment`` argument is now exposed as a named parameter (:issue:`18735`)
 - Rearranged the order of keyword arguments in :func:`read_excel()` to align with :func:`read_csv()` (:issue:`16672`)
+- The options ``html.border`` and ``mode.use_inf_as_null`` were deprecated in prior versions, these will now show ``FutureWarning`` rather than a ``DeprecationWarning`` (:issue:`19003`)
 
 .. _whatsnew_0230.deprecations:
 

pandas/_libs/tslibs/timezones.pyx

@@ -3,6 +3,7 @@
 
 cimport cython
 from cython cimport Py_ssize_t
+from warnings import catch_warnings
 
 # dateutil compat
 from dateutil.tz import (
@@ -100,7 +101,9 @@ cpdef inline object maybe_get_tz(object tz):
             tz = _dateutil_tzlocal()
         elif tz.startswith('dateutil/'):
             zone = tz[9:]
-            tz = dateutil_gettz(zone)
+            with catch_warnings(record=True):
+                tz = dateutil_gettz(zone)
+
             # On Python 3 on Windows, the filename is not always set correctly.
             if isinstance(tz, _dateutil_tzfile) and '.tar.gz' in tz._filename:
                 tz._filename = zone

pandas/core/config.py

@@ -613,7 +613,7 @@ def _warn_if_deprecated(key):
     if d:
         if d.msg:
             print(d.msg)
-            warnings.warn(d.msg, DeprecationWarning)
+            warnings.warn(d.msg, FutureWarning)
         else:
             msg = "'{key}' is deprecated".format(key=key)
             if d.removal_ver:
@@ -624,7 +624,7 @@ def _warn_if_deprecated(key):
             else:
                 msg += ', please refrain from using it.'
 
-            warnings.warn(msg, DeprecationWarning)
+            warnings.warn(msg, FutureWarning)
         return True
     return False
 

pandas/tests/frame/test_constructors.py

@@ -328,13 +328,13 @@ def test_constructor_error_msgs(self):
 
         # wrong size axis labels
         with tm.assert_raises_regex(ValueError, "Shape of passed values "
-                                    "is \(3, 2\), indices "
-                                    "imply \(3, 1\)"):
+                                    r"is \(3, 2\), indices "
+                                    r"imply \(3, 1\)"):
             DataFrame(np.random.rand(2, 3), columns=['A', 'B', 'C'], index=[1])
 
         with tm.assert_raises_regex(ValueError, "Shape of passed values "
-                                    "is \(3, 2\), indices "
-                                    "imply \(2, 2\)"):
+                                    r"is \(3, 2\), indices "
+                                    r"imply \(2, 2\)"):
             DataFrame(np.random.rand(2, 3), columns=['A', 'B'], index=[1, 2])
 
         with tm.assert_raises_regex(ValueError, "If using all scalar "

pandas/tests/frame/test_query_eval.py

@@ -1040,6 +1040,6 @@ def test_invalid_type_for_operator_raises(self, parser, engine):
         ops = '+', '-', '*', '/'
         for op in ops:
             with tm.assert_raises_regex(TypeError,
-                                        "unsupported operand type\(s\) "
+                                        r"unsupported operand type\(s\) "
                                         "for .+: '.+' and '.+'"):
                 df.eval('a {0} b'.format(op), engine=engine, parser=parser)

pandas/tests/groupby/test_nth.py

@@ -175,7 +175,7 @@ def test_nth(self):
         df = DataFrame([[1, np.nan], [1, 4], [5, 6]], columns=['A', 'B'])
         g = df.groupby('A')
         # PR 17493, related to issue 11038
-        # test Series.nth with True for dropna produces DeprecationWarning
+        # test Series.nth with True for dropna produces FutureWarning
         with assert_produces_warning(FutureWarning):
             result = g.B.nth(0, dropna=True)
         expected = g.B.first()

pandas/tests/groupby/test_transform.py

@@ -261,7 +261,7 @@ def test_transform_casting(self):
         9   B-053  b76cd912ff "2014-10-08 19:17:48"
         10  B-065  b76cd912ff "2014-10-08 19:21:38"
         """
-        df = pd.read_csv(StringIO(data), sep='\s+',
+        df = pd.read_csv(StringIO(data), sep=r'\s+',
                          index_col=[0], parse_dates=['DATETIME'])
 
         result = df.groupby('ID3')['DATETIME'].transform(lambda x: x.diff())

pandas/tests/indexes/datetimes/test_date_range.py

@@ -402,7 +402,7 @@ def test_daterange_bug_456(self):
         assert isinstance(result, DatetimeIndex)
 
     def test_error_with_zero_monthends(self):
-        msg = 'Offset <0 \* MonthEnds> did not increment date'
+        msg = r'Offset <0 \* MonthEnds> did not increment date'
         with tm.assert_raises_regex(ValueError, msg):
             date_range('1/1/2000', '1/1/2001', freq=MonthEnd(0))
 

pandas/tests/indexes/datetimes/test_tools.py

@@ -709,7 +709,7 @@ def test_dataframe(self, cache):
                          'day': [4, 5]})
 
         msg = ("cannot assemble the datetimes: time data .+ does not "
-               "match format '%Y%m%d' \(match\)")
+               r"match format '%Y%m%d' \(match\)")
         with tm.assert_raises_regex(ValueError, msg):
             to_datetime(df2, cache=cache)
         result = to_datetime(df2, errors='coerce', cache=cache)

pandas/tests/indexes/interval/test_interval.py

@@ -175,7 +175,7 @@ def test_constructors_empty(self, data, closed):
     def test_constructors_errors(self):
 
         # scalar
-        msg = ('IntervalIndex\(...\) must be called with a collection of '
+        msg = (r'IntervalIndex\(...\) must be called with a collection of '
                'some kind, 5 was passed')
         with tm.assert_raises_regex(TypeError, msg):
             IntervalIndex(5)

pandas/tests/indexing/test_multiindex.py

@@ -299,7 +299,7 @@ def test_getitem_partial_int(self):
         # missing item:
         with tm.assert_raises_regex(KeyError, '1'):
             df[1]
-        with tm.assert_raises_regex(KeyError, "'\[1\] not in index'"):
+        with tm.assert_raises_regex(KeyError, r"'\[1\] not in index'"):
             df[[1]]
 
     def test_loc_multiindex_indexer_none(self):

pandas/tests/io/formats/test_to_html.py

@@ -1411,8 +1411,9 @@ def test_to_html_border_zero(self):
         result = df.to_html(border=0)
         assert 'border="0"' in result
 
+    @tm.capture_stdout
     def test_display_option_warning(self):
-        with tm.assert_produces_warning(DeprecationWarning,
+        with tm.assert_produces_warning(FutureWarning,
                                         check_stacklevel=False):
             pd.options.html.border
 

pandas/tests/io/formats/test_to_latex.py

@@ -424,11 +424,11 @@ def test_to_latex_longtable(self, frame):
 
         df = DataFrame({'a': [1, 2]})
         with1column_result = df.to_latex(index=False, longtable=True)
-        assert "\multicolumn{1}" in with1column_result
+        assert r"\multicolumn{1}" in with1column_result
 
         df = DataFrame({'a': [1, 2], 'b': [3, 4], 'c': [5, 6]})
         with3columns_result = df.to_latex(index=False, longtable=True)
-        assert "\multicolumn{3}" in with3columns_result
+        assert r"\multicolumn{3}" in with3columns_result
 
     def test_to_latex_escape_special_chars(self):
         special_characters = ['&', '%', '$', '#', '_', '{', '}', '~', '^',

pandas/tests/reshape/merge/test_merge_asof.py

@@ -976,7 +976,7 @@ def test_on_float_by_int(self):
 
     def test_merge_datatype_error(self):
         """ Tests merge datatype mismatch error """
-        msg = 'merge keys \[0\] object and int64, must be the same type'
+        msg = r'merge keys \[0\] object and int64, must be the same type'
 
         left = pd.DataFrame({'left_val': [1, 5, 10],
                              'a': ['a', 'b', 'c']})

pandas/tests/reshape/test_concat.py

@@ -178,9 +178,9 @@ def test_concatlike_same_dtypes(self):
             tm.assert_series_equal(res, exp, check_index_type=True)
 
             # cannot append non-index
-            msg = ('cannot concatenate object of type \"(.+?)\";'
+            msg = (r'cannot concatenate object of type \"(.+?)\";'
                    ' only pd.Series, pd.DataFrame, and pd.Panel'
-                   ' \(deprecated\) objs are valid')
+                   r' \(deprecated\) objs are valid')
             with tm.assert_raises_regex(TypeError, msg):
                 pd.Series(vals1).append(vals2)
 

pandas/tests/scalar/test_interval.py

@@ -122,7 +122,7 @@ def test_math_add(self, interval):
         actual += 1
         assert expected == actual
 
-        msg = "unsupported operand type\(s\) for \+"
+        msg = r"unsupported operand type\(s\) for \+"
         with tm.assert_raises_regex(TypeError, msg):
             interval + Interval(1, 2)
 
@@ -138,7 +138,7 @@ def test_math_sub(self, interval):
         actual -= 1
         assert expected == actual
 
-        msg = "unsupported operand type\(s\) for -"
+        msg = r"unsupported operand type\(s\) for -"
         with tm.assert_raises_regex(TypeError, msg):
             interval - Interval(1, 2)
 
@@ -158,11 +158,11 @@ def test_math_mult(self, interval):
         actual *= 2
         assert expected == actual
 
-        msg = "unsupported operand type\(s\) for \*"
+        msg = r"unsupported operand type\(s\) for \*"
         with tm.assert_raises_regex(TypeError, msg):
             interval * Interval(1, 2)
 
-        msg = "can\'t multiply sequence by non-int"
+        msg = r"can\'t multiply sequence by non-int"
         with tm.assert_raises_regex(TypeError, msg):
             interval * 'foo'
 
@@ -175,7 +175,7 @@ def test_math_div(self, interval):
         actual /= 2.0
         assert expected == actual
 
-        msg = "unsupported operand type\(s\) for /"
+        msg = r"unsupported operand type\(s\) for /"
         with tm.assert_raises_regex(TypeError, msg):
             interval / Interval(1, 2)
 

pandas/tests/series/test_missing.py

@@ -480,12 +480,9 @@ def test_isna_for_inf(self):
     def test_isnull_for_inf_deprecated(self):
         # gh-17115
         s = Series(['a', np.inf, np.nan, 1.0])
-        with tm.assert_produces_warning(DeprecationWarning,
-                                        check_stacklevel=False):
-            pd.set_option('mode.use_inf_as_null', True)
+        with pd.option_context('mode.use_inf_as_null', True):
             r = s.isna()
             dr = s.dropna()
-            pd.reset_option('mode.use_inf_as_null')
 
         e = Series([False, True, True, False])
         de = Series(['a', 1.0], index=[0, 3])

pandas/tests/test_algos.py

@@ -788,10 +788,10 @@ def test_duplicated_with_nas(self):
                   2, 4, 1, 5, 6]),
         np.array([1.1, 2.2, 1.1, np.nan, 3.3,
                   2.2, 4.4, 1.1, np.nan, 6.6]),
-        pytest.mark.xfail(reason="Complex bug. GH 16399")(
-            np.array([1 + 1j, 2 + 2j, 1 + 1j, 5 + 5j, 3 + 3j,
-                      2 + 2j, 4 + 4j, 1 + 1j, 5 + 5j, 6 + 6j])
-        ),
+        pytest.param(np.array([1 + 1j, 2 + 2j, 1 + 1j, 5 + 5j, 3 + 3j,
+                               2 + 2j, 4 + 4j, 1 + 1j, 5 + 5j, 6 + 6j]),
+                     marks=pytest.mark.xfail(reason="Complex bug. GH 16399")
+                     ),
         np.array(['a', 'b', 'a', 'e', 'c',
                   'b', 'd', 'a', 'e', 'f'], dtype=object),
         np.array([1, 2**63, 1, 3**5, 10, 2**63, 39, 1, 3**5, 7],

pandas/tests/test_multilevel.py

@@ -2289,7 +2289,7 @@ def test_reset_index_multiindex_columns(self):
 
         # gh-16120: already existing column
         with tm.assert_raises_regex(ValueError,
-                                    ("cannot insert \('A', ''\), "
+                                    (r"cannot insert \('A', ''\), "
                                      "already exists")):
             df.rename_axis('A').reset_index()