Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 1 | import unittest |
Brett Cannon | 977eb02 | 2008-03-19 17:37:43 +0000 | [diff] [blame] | 2 | import sys |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 3 | from test.test_support import check_py3k_warnings, CleanImport, run_unittest |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 4 | import warnings |
Victor Stinner | ccd6270 | 2015-09-03 10:46:17 +0200 | [diff] [blame] | 5 | from test import test_support |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 6 | |
Brett Cannon | 977eb02 | 2008-03-19 17:37:43 +0000 | [diff] [blame] | 7 | if not sys.py3kwarning: |
Benjamin Peterson | bec087f | 2009-03-26 21:10:30 +0000 | [diff] [blame] | 8 | raise unittest.SkipTest('%s must be run with the -3 flag' % __name__) |
Brett Cannon | 977eb02 | 2008-03-19 17:37:43 +0000 | [diff] [blame] | 9 | |
Florent Xicluna | 4d42f2b | 2010-03-09 19:57:01 +0000 | [diff] [blame] | 10 | try: |
| 11 | from test.test_support import __warningregistry__ as _registry |
| 12 | except ImportError: |
| 13 | def check_deprecated_module(module_name): |
| 14 | return False |
| 15 | else: |
| 16 | past_warnings = _registry.keys() |
| 17 | del _registry |
| 18 | def check_deprecated_module(module_name): |
| 19 | """Lookup the past warnings for module already loaded using |
| 20 | test_support.import_module(..., deprecated=True) |
| 21 | """ |
| 22 | return any(module_name in msg and ' removed' in msg |
| 23 | and issubclass(cls, DeprecationWarning) |
| 24 | and (' module' in msg or ' package' in msg) |
| 25 | for (msg, cls, line) in past_warnings) |
| 26 | |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 27 | def reset_module_registry(module): |
| 28 | try: |
| 29 | registry = module.__warningregistry__ |
| 30 | except AttributeError: |
| 31 | pass |
| 32 | else: |
| 33 | registry.clear() |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 34 | |
| 35 | class TestPy3KWarnings(unittest.TestCase): |
| 36 | |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 37 | def assertWarning(self, _, warning, expected_message): |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 38 | self.assertEqual(str(warning.message), expected_message) |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 39 | |
Benjamin Peterson | 1bf4765 | 2009-07-02 17:06:17 +0000 | [diff] [blame] | 40 | def assertNoWarning(self, _, recorder): |
| 41 | self.assertEqual(len(recorder.warnings), 0) |
| 42 | |
Benjamin Peterson | 2fe3ef8 | 2008-06-08 02:05:33 +0000 | [diff] [blame] | 43 | def test_backquote(self): |
| 44 | expected = 'backquote not supported in 3.x; use repr()' |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 45 | with check_py3k_warnings((expected, SyntaxWarning)): |
Benjamin Peterson | 2fe3ef8 | 2008-06-08 02:05:33 +0000 | [diff] [blame] | 46 | exec "`2`" in {} |
Benjamin Peterson | 2fe3ef8 | 2008-06-08 02:05:33 +0000 | [diff] [blame] | 47 | |
Benjamin Peterson | 99a5023 | 2009-11-19 22:54:57 +0000 | [diff] [blame] | 48 | def test_paren_arg_names(self): |
| 49 | expected = 'parenthesized argument names are invalid in 3.x' |
| 50 | def check(s): |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 51 | with check_py3k_warnings((expected, SyntaxWarning)): |
| 52 | exec s in {} |
| 53 | check("def f((x)): pass") |
| 54 | check("def f((((x))), (y)): pass") |
| 55 | check("def f((x), (((y))), m=32): pass") |
| 56 | # Something like def f((a, (b))): pass will raise the tuple |
| 57 | # unpacking warning. |
Benjamin Peterson | 99a5023 | 2009-11-19 22:54:57 +0000 | [diff] [blame] | 58 | |
Benjamin Peterson | 399b1fe | 2008-10-25 02:53:28 +0000 | [diff] [blame] | 59 | def test_forbidden_names(self): |
Benjamin Peterson | d5efd20 | 2008-06-08 22:52:37 +0000 | [diff] [blame] | 60 | # So we don't screw up our globals |
| 61 | def safe_exec(expr): |
| 62 | def f(**kwargs): pass |
| 63 | exec expr in {'f' : f} |
| 64 | |
Benjamin Peterson | 399b1fe | 2008-10-25 02:53:28 +0000 | [diff] [blame] | 65 | tests = [("True", "assignment to True or False is forbidden in 3.x"), |
| 66 | ("False", "assignment to True or False is forbidden in 3.x"), |
| 67 | ("nonlocal", "nonlocal is a keyword in 3.x")] |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 68 | with check_py3k_warnings(('', SyntaxWarning)) as w: |
Benjamin Peterson | 399b1fe | 2008-10-25 02:53:28 +0000 | [diff] [blame] | 69 | for keyword, expected in tests: |
| 70 | safe_exec("{0} = False".format(keyword)) |
| 71 | self.assertWarning(None, w, expected) |
| 72 | w.reset() |
| 73 | try: |
| 74 | safe_exec("obj.{0} = True".format(keyword)) |
| 75 | except NameError: |
| 76 | pass |
| 77 | self.assertWarning(None, w, expected) |
| 78 | w.reset() |
| 79 | safe_exec("def {0}(): pass".format(keyword)) |
| 80 | self.assertWarning(None, w, expected) |
| 81 | w.reset() |
| 82 | safe_exec("class {0}: pass".format(keyword)) |
| 83 | self.assertWarning(None, w, expected) |
| 84 | w.reset() |
| 85 | safe_exec("def f({0}=43): pass".format(keyword)) |
| 86 | self.assertWarning(None, w, expected) |
| 87 | w.reset() |
Benjamin Peterson | d5efd20 | 2008-06-08 22:52:37 +0000 | [diff] [blame] | 88 | |
| 89 | |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 90 | def test_type_inequality_comparisons(self): |
Georg Brandl | d5b635f | 2008-03-25 08:29:14 +0000 | [diff] [blame] | 91 | expected = 'type inequality comparisons not supported in 3.x' |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 92 | with check_py3k_warnings() as w: |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 93 | self.assertWarning(int < str, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 94 | w.reset() |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 95 | self.assertWarning(type < object, w, expected) |
| 96 | |
| 97 | def test_object_inequality_comparisons(self): |
Georg Brandl | d5b635f | 2008-03-25 08:29:14 +0000 | [diff] [blame] | 98 | expected = 'comparing unequal types not supported in 3.x' |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 99 | with check_py3k_warnings() as w: |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 100 | self.assertWarning(str < [], w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 101 | w.reset() |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 102 | self.assertWarning(object() < (1, 2), w, expected) |
| 103 | |
| 104 | def test_dict_inequality_comparisons(self): |
Georg Brandl | d5b635f | 2008-03-25 08:29:14 +0000 | [diff] [blame] | 105 | expected = 'dict inequality comparisons not supported in 3.x' |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 106 | with check_py3k_warnings() as w: |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 107 | self.assertWarning({} < {2:3}, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 108 | w.reset() |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 109 | self.assertWarning({} <= {}, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 110 | w.reset() |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 111 | self.assertWarning({} > {2:3}, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 112 | w.reset() |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 113 | self.assertWarning({2:3} >= {}, w, expected) |
| 114 | |
| 115 | def test_cell_inequality_comparisons(self): |
Georg Brandl | d5b635f | 2008-03-25 08:29:14 +0000 | [diff] [blame] | 116 | expected = 'cell comparisons not supported in 3.x' |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 117 | def f(x): |
| 118 | def g(): |
| 119 | return x |
| 120 | return g |
| 121 | cell0, = f(0).func_closure |
| 122 | cell1, = f(1).func_closure |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 123 | with check_py3k_warnings() as w: |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 124 | self.assertWarning(cell0 == cell1, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 125 | w.reset() |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 126 | self.assertWarning(cell0 < cell1, w, expected) |
| 127 | |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 128 | def test_code_inequality_comparisons(self): |
Georg Brandl | d5b635f | 2008-03-25 08:29:14 +0000 | [diff] [blame] | 129 | expected = 'code inequality comparisons not supported in 3.x' |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 130 | def f(x): |
| 131 | pass |
| 132 | def g(x): |
| 133 | pass |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 134 | with check_py3k_warnings() as w: |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 135 | self.assertWarning(f.func_code < g.func_code, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 136 | w.reset() |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 137 | self.assertWarning(f.func_code <= g.func_code, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 138 | w.reset() |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 139 | self.assertWarning(f.func_code >= g.func_code, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 140 | w.reset() |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 141 | self.assertWarning(f.func_code > g.func_code, w, expected) |
| 142 | |
| 143 | def test_builtin_function_or_method_comparisons(self): |
| 144 | expected = ('builtin_function_or_method ' |
Benjamin Peterson | 1bf4765 | 2009-07-02 17:06:17 +0000 | [diff] [blame] | 145 | 'order comparisons not supported in 3.x') |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 146 | func = eval |
| 147 | meth = {}.get |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 148 | with check_py3k_warnings() as w: |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 149 | self.assertWarning(func < meth, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 150 | w.reset() |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 151 | self.assertWarning(func > meth, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 152 | w.reset() |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 153 | self.assertWarning(meth <= func, w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 154 | w.reset() |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 155 | self.assertWarning(meth >= func, w, expected) |
Benjamin Peterson | 1bf4765 | 2009-07-02 17:06:17 +0000 | [diff] [blame] | 156 | w.reset() |
| 157 | self.assertNoWarning(meth == func, w) |
| 158 | self.assertNoWarning(meth != func, w) |
| 159 | lam = lambda x: x |
| 160 | self.assertNoWarning(lam == func, w) |
| 161 | self.assertNoWarning(lam != func, w) |
Steven Bethard | 6a644f9 | 2008-03-18 22:08:20 +0000 | [diff] [blame] | 162 | |
Benjamin Peterson | f09925d | 2008-12-22 20:16:25 +0000 | [diff] [blame] | 163 | def test_frame_attributes(self): |
| 164 | template = "%s has been removed in 3.x" |
| 165 | f = sys._getframe(0) |
| 166 | for attr in ("f_exc_traceback", "f_exc_value", "f_exc_type"): |
| 167 | expected = template % attr |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 168 | with check_py3k_warnings() as w: |
Benjamin Peterson | f09925d | 2008-12-22 20:16:25 +0000 | [diff] [blame] | 169 | self.assertWarning(getattr(f, attr), w, expected) |
| 170 | w.reset() |
| 171 | self.assertWarning(setattr(f, attr, None), w, expected) |
| 172 | |
Raymond Hettinger | 0538786 | 2008-03-19 17:45:19 +0000 | [diff] [blame] | 173 | def test_sort_cmp_arg(self): |
Georg Brandl | d5b635f | 2008-03-25 08:29:14 +0000 | [diff] [blame] | 174 | expected = "the cmp argument is not supported in 3.x" |
Raymond Hettinger | 0538786 | 2008-03-19 17:45:19 +0000 | [diff] [blame] | 175 | lst = range(5) |
| 176 | cmp = lambda x,y: -1 |
| 177 | |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 178 | with check_py3k_warnings() as w: |
Raymond Hettinger | 0538786 | 2008-03-19 17:45:19 +0000 | [diff] [blame] | 179 | self.assertWarning(lst.sort(cmp=cmp), w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 180 | w.reset() |
Raymond Hettinger | 0538786 | 2008-03-19 17:45:19 +0000 | [diff] [blame] | 181 | self.assertWarning(sorted(lst, cmp=cmp), w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 182 | w.reset() |
Raymond Hettinger | 0538786 | 2008-03-19 17:45:19 +0000 | [diff] [blame] | 183 | self.assertWarning(lst.sort(cmp), w, expected) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 184 | w.reset() |
Raymond Hettinger | 0538786 | 2008-03-19 17:45:19 +0000 | [diff] [blame] | 185 | self.assertWarning(sorted(lst, cmp), w, expected) |
| 186 | |
Georg Brandl | 5a44424 | 2008-03-21 20:11:46 +0000 | [diff] [blame] | 187 | def test_sys_exc_clear(self): |
Georg Brandl | d5b635f | 2008-03-25 08:29:14 +0000 | [diff] [blame] | 188 | expected = 'sys.exc_clear() not supported in 3.x; use except clauses' |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 189 | with check_py3k_warnings() as w: |
Georg Brandl | 5a44424 | 2008-03-21 20:11:46 +0000 | [diff] [blame] | 190 | self.assertWarning(sys.exc_clear(), w, expected) |
| 191 | |
Georg Brandl | 07e5681 | 2008-03-21 20:21:46 +0000 | [diff] [blame] | 192 | def test_methods_members(self): |
| 193 | expected = '__members__ and __methods__ not supported in 3.x' |
| 194 | class C: |
| 195 | __methods__ = ['a'] |
| 196 | __members__ = ['b'] |
| 197 | c = C() |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 198 | with check_py3k_warnings() as w: |
Georg Brandl | 07e5681 | 2008-03-21 20:21:46 +0000 | [diff] [blame] | 199 | self.assertWarning(dir(c), w, expected) |
| 200 | |
Georg Brandl | 65bb42d | 2008-03-21 20:38:24 +0000 | [diff] [blame] | 201 | def test_softspace(self): |
| 202 | expected = 'file.softspace not supported in 3.x' |
| 203 | with file(__file__) as f: |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 204 | with check_py3k_warnings() as w: |
Georg Brandl | 65bb42d | 2008-03-21 20:38:24 +0000 | [diff] [blame] | 205 | self.assertWarning(f.softspace, w, expected) |
| 206 | def set(): |
| 207 | f.softspace = 0 |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 208 | with check_py3k_warnings() as w: |
Georg Brandl | 65bb42d | 2008-03-21 20:38:24 +0000 | [diff] [blame] | 209 | self.assertWarning(set(), w, expected) |
| 210 | |
Benjamin Peterson | 712ee92 | 2008-08-24 18:10:20 +0000 | [diff] [blame] | 211 | def test_slice_methods(self): |
| 212 | class Spam(object): |
| 213 | def __getslice__(self, i, j): pass |
| 214 | def __setslice__(self, i, j, what): pass |
| 215 | def __delslice__(self, i, j): pass |
| 216 | class Egg: |
| 217 | def __getslice__(self, i, h): pass |
| 218 | def __setslice__(self, i, j, what): pass |
| 219 | def __delslice__(self, i, j): pass |
| 220 | |
| 221 | expected = "in 3.x, __{0}slice__ has been removed; use __{0}item__" |
| 222 | |
| 223 | for obj in (Spam(), Egg()): |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 224 | with check_py3k_warnings() as w: |
Benjamin Peterson | 712ee92 | 2008-08-24 18:10:20 +0000 | [diff] [blame] | 225 | self.assertWarning(obj[1:2], w, expected.format('get')) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 226 | w.reset() |
Benjamin Peterson | 712ee92 | 2008-08-24 18:10:20 +0000 | [diff] [blame] | 227 | del obj[3:4] |
| 228 | self.assertWarning(None, w, expected.format('del')) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 229 | w.reset() |
Benjamin Peterson | 712ee92 | 2008-08-24 18:10:20 +0000 | [diff] [blame] | 230 | obj[4:5] = "eggs" |
| 231 | self.assertWarning(None, w, expected.format('set')) |
| 232 | |
Benjamin Peterson | f4fcdb6 | 2008-06-08 23:00:00 +0000 | [diff] [blame] | 233 | def test_tuple_parameter_unpacking(self): |
| 234 | expected = "tuple parameter unpacking has been removed in 3.x" |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 235 | with check_py3k_warnings((expected, SyntaxWarning)): |
Benjamin Peterson | f4fcdb6 | 2008-06-08 23:00:00 +0000 | [diff] [blame] | 236 | exec "def f((a, b)): pass" |
Benjamin Peterson | f4fcdb6 | 2008-06-08 23:00:00 +0000 | [diff] [blame] | 237 | |
Georg Brandl | 80055f6 | 2008-03-25 07:56:27 +0000 | [diff] [blame] | 238 | def test_buffer(self): |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 239 | expected = 'buffer() not supported in 3.x' |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 240 | with check_py3k_warnings() as w: |
Georg Brandl | 80055f6 | 2008-03-25 07:56:27 +0000 | [diff] [blame] | 241 | self.assertWarning(buffer('a'), w, expected) |
| 242 | |
Georg Brandl | a9916b5 | 2008-05-17 22:11:54 +0000 | [diff] [blame] | 243 | def test_file_xreadlines(self): |
| 244 | expected = ("f.xreadlines() not supported in 3.x, " |
| 245 | "try 'for line in f' instead") |
| 246 | with file(__file__) as f: |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 247 | with check_py3k_warnings() as w: |
Georg Brandl | a9916b5 | 2008-05-17 22:11:54 +0000 | [diff] [blame] | 248 | self.assertWarning(f.xreadlines(), w, expected) |
| 249 | |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 250 | def test_hash_inheritance(self): |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 251 | with check_py3k_warnings() as w: |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 252 | # With object as the base class |
| 253 | class WarnOnlyCmp(object): |
| 254 | def __cmp__(self, other): pass |
Mark Dickinson | ec27d91 | 2010-06-05 13:18:33 +0000 | [diff] [blame] | 255 | self.assertEqual(len(w.warnings), 0) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 256 | w.reset() |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 257 | class WarnOnlyEq(object): |
| 258 | def __eq__(self, other): pass |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 259 | self.assertEqual(len(w.warnings), 1) |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 260 | self.assertWarning(None, w, |
| 261 | "Overriding __eq__ blocks inheritance of __hash__ in 3.x") |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 262 | w.reset() |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 263 | class WarnCmpAndEq(object): |
| 264 | def __cmp__(self, other): pass |
| 265 | def __eq__(self, other): pass |
Mark Dickinson | ec27d91 | 2010-06-05 13:18:33 +0000 | [diff] [blame] | 266 | self.assertEqual(len(w.warnings), 1) |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 267 | self.assertWarning(None, w, |
| 268 | "Overriding __eq__ blocks inheritance of __hash__ in 3.x") |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 269 | w.reset() |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 270 | class NoWarningOnlyHash(object): |
| 271 | def __hash__(self): pass |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 272 | self.assertEqual(len(w.warnings), 0) |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 273 | # With an intermediate class in the heirarchy |
| 274 | class DefinesAllThree(object): |
| 275 | def __cmp__(self, other): pass |
| 276 | def __eq__(self, other): pass |
| 277 | def __hash__(self): pass |
| 278 | class WarnOnlyCmp(DefinesAllThree): |
| 279 | def __cmp__(self, other): pass |
Mark Dickinson | ec27d91 | 2010-06-05 13:18:33 +0000 | [diff] [blame] | 280 | self.assertEqual(len(w.warnings), 0) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 281 | w.reset() |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 282 | class WarnOnlyEq(DefinesAllThree): |
| 283 | def __eq__(self, other): pass |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 284 | self.assertEqual(len(w.warnings), 1) |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 285 | self.assertWarning(None, w, |
| 286 | "Overriding __eq__ blocks inheritance of __hash__ in 3.x") |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 287 | w.reset() |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 288 | class WarnCmpAndEq(DefinesAllThree): |
| 289 | def __cmp__(self, other): pass |
| 290 | def __eq__(self, other): pass |
Mark Dickinson | ec27d91 | 2010-06-05 13:18:33 +0000 | [diff] [blame] | 291 | self.assertEqual(len(w.warnings), 1) |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 292 | self.assertWarning(None, w, |
| 293 | "Overriding __eq__ blocks inheritance of __hash__ in 3.x") |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 294 | w.reset() |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 295 | class NoWarningOnlyHash(DefinesAllThree): |
| 296 | def __hash__(self): pass |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 297 | self.assertEqual(len(w.warnings), 0) |
Nick Coghlan | 48361f5 | 2008-08-11 15:45:58 +0000 | [diff] [blame] | 298 | |
Alexandre Vassalotti | 0fe7991 | 2009-07-05 04:22:40 +0000 | [diff] [blame] | 299 | def test_operator(self): |
| 300 | from operator import isCallable, sequenceIncludes |
| 301 | |
Alexandre Vassalotti | 16a0247 | 2009-07-05 04:25:46 +0000 | [diff] [blame] | 302 | callable_warn = ("operator.isCallable() is not supported in 3.x. " |
| 303 | "Use hasattr(obj, '__call__').") |
Alexandre Vassalotti | 0fe7991 | 2009-07-05 04:22:40 +0000 | [diff] [blame] | 304 | seq_warn = ("operator.sequenceIncludes() is not supported " |
| 305 | "in 3.x. Use operator.contains().") |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 306 | with check_py3k_warnings() as w: |
Alexandre Vassalotti | 0fe7991 | 2009-07-05 04:22:40 +0000 | [diff] [blame] | 307 | self.assertWarning(isCallable(self), w, callable_warn) |
| 308 | w.reset() |
| 309 | self.assertWarning(sequenceIncludes(range(3), 2), w, seq_warn) |
| 310 | |
Serhiy Storchaka | 79fa98a | 2014-06-01 22:13:39 +0300 | [diff] [blame] | 311 | def test_nonascii_bytes_literals(self): |
| 312 | expected = "non-ascii bytes literals not supported in 3.x" |
| 313 | with check_py3k_warnings((expected, SyntaxWarning)): |
| 314 | exec "b'\xbd'" |
| 315 | |
Georg Brandl | 07e5681 | 2008-03-21 20:21:46 +0000 | [diff] [blame] | 316 | |
Brett Cannon | e5d2cba | 2008-05-06 23:23:34 +0000 | [diff] [blame] | 317 | class TestStdlibRemovals(unittest.TestCase): |
| 318 | |
Brett Cannon | 3c75914 | 2008-05-09 05:25:37 +0000 | [diff] [blame] | 319 | # test.testall not tested as it executes all unit tests as an |
| 320 | # import side-effect. |
Brett Cannon | 4c1f881 | 2008-05-10 02:27:04 +0000 | [diff] [blame] | 321 | all_platforms = ('audiodev', 'imputil', 'mutex', 'user', 'new', 'rexec', |
Brett Cannon | 1e8fba7 | 2008-07-18 19:30:22 +0000 | [diff] [blame] | 322 | 'Bastion', 'compiler', 'dircache', 'mimetools', |
| 323 | 'fpformat', 'ihooks', 'mhlib', 'statvfs', 'htmllib', |
| 324 | 'sgmllib', 'rfc822', 'sunaudio') |
Brett Cannon | 54c77aa | 2008-05-14 21:08:41 +0000 | [diff] [blame] | 325 | inclusive_platforms = {'irix' : ('pure', 'AL', 'al', 'CD', 'cd', 'cddb', |
Brett Cannon | 044616a | 2008-05-15 02:33:55 +0000 | [diff] [blame] | 326 | 'cdplayer', 'CL', 'cl', 'DEVICE', 'GL', |
Brett Cannon | 75ba465 | 2008-05-15 03:23:17 +0000 | [diff] [blame] | 327 | 'gl', 'ERRNO', 'FILE', 'FL', 'flp', 'fl', |
Brett Cannon | d8c41ec | 2008-05-15 03:41:55 +0000 | [diff] [blame] | 328 | 'fm', 'GET', 'GLWS', 'imgfile', 'IN', |
Brett Cannon | cd2de08 | 2008-05-15 03:51:21 +0000 | [diff] [blame] | 329 | 'IOCTL', 'jpeg', 'panel', 'panelparser', |
Brett Cannon | 74a596c | 2008-05-15 04:17:35 +0000 | [diff] [blame] | 330 | 'readcd', 'SV', 'torgb', 'WAIT'), |
Benjamin Peterson | 2368193 | 2008-05-12 21:42:13 +0000 | [diff] [blame] | 331 | 'darwin' : ('autoGIL', 'Carbon', 'OSATerminology', |
Ronald Oussoren | 934f4e1 | 2010-06-22 09:18:28 +0000 | [diff] [blame] | 332 | 'icglue', 'Nav', |
| 333 | # MacOS should (and does) give a Py3kWarning, but one of the |
| 334 | # earlier tests already imports the MacOS extension which causes |
| 335 | # this test to fail. Disabling the test for 'MacOS' avoids this |
| 336 | # spurious test failure. |
| 337 | #'MacOS', |
| 338 | 'aepack', |
Brett Cannon | ea785fb | 2008-05-14 01:09:40 +0000 | [diff] [blame] | 339 | 'aetools', 'aetypes', 'applesingle', |
| 340 | 'appletrawmain', 'appletrunner', |
| 341 | 'argvemulator', 'bgenlocations', |
Benjamin Peterson | 2368193 | 2008-05-12 21:42:13 +0000 | [diff] [blame] | 342 | 'EasyDialogs', 'macerrors', 'macostools', |
| 343 | 'findertools', 'FrameWork', 'ic', |
| 344 | 'gensuitemodule', 'icopen', 'macresource', |
| 345 | 'MiniAEFrame', 'pimp', 'PixMapWrapper', |
Brett Cannon | ea785fb | 2008-05-14 01:09:40 +0000 | [diff] [blame] | 346 | 'terminalcommand', 'videoreader', |
| 347 | '_builtinSuites', 'CodeWarrior', |
| 348 | 'Explorer', 'Finder', 'Netscape', |
| 349 | 'StdSuites', 'SystemEvents', 'Terminal', |
| 350 | 'cfmfile', 'bundlebuilder', 'buildtools', |
Benjamin Peterson | a6864e0 | 2008-07-14 17:42:17 +0000 | [diff] [blame] | 351 | 'ColorPicker', 'Audio_mac'), |
Brett Cannon | 2224817 | 2008-05-16 00:10:24 +0000 | [diff] [blame] | 352 | 'sunos5' : ('sunaudiodev', 'SUNAUDIODEV'), |
| 353 | } |
Brett Cannon | ac861b5 | 2008-05-12 03:45:59 +0000 | [diff] [blame] | 354 | optional_modules = ('bsddb185', 'Canvas', 'dl', 'linuxaudiodev', 'imageop', |
Antoine Pitrou | 8015725 | 2010-01-08 19:20:17 +0000 | [diff] [blame] | 355 | 'sv', 'bsddb', 'dbhash') |
Brett Cannon | e5d2cba | 2008-05-06 23:23:34 +0000 | [diff] [blame] | 356 | |
Brett Cannon | 9ac3974 | 2008-05-09 22:51:58 +0000 | [diff] [blame] | 357 | def check_removal(self, module_name, optional=False): |
Brett Cannon | e5d2cba | 2008-05-06 23:23:34 +0000 | [diff] [blame] | 358 | """Make sure the specified module, when imported, raises a |
| 359 | DeprecationWarning and specifies itself in the message.""" |
Victor Stinner | ccd6270 | 2015-09-03 10:46:17 +0200 | [diff] [blame] | 360 | if module_name in sys.modules: |
| 361 | mod = sys.modules[module_name] |
| 362 | filename = getattr(mod, '__file__', '') |
| 363 | mod = None |
| 364 | # the module is not implemented in C? |
| 365 | if not filename.endswith(('.py', '.pyc', '.pyo')): |
| 366 | # Issue #23375: If the module was already loaded, reimporting |
| 367 | # the module will not emit again the warning. The warning is |
| 368 | # emited when the module is loaded, but C modules cannot |
| 369 | # unloaded. |
| 370 | if test_support.verbose: |
| 371 | print("Cannot test the Python 3 DeprecationWarning of the " |
| 372 | "%s module, the C module is already loaded" |
| 373 | % module_name) |
| 374 | return |
Florent Xicluna | 4d42f2b | 2010-03-09 19:57:01 +0000 | [diff] [blame] | 375 | with CleanImport(module_name), warnings.catch_warnings(): |
| 376 | warnings.filterwarnings("error", ".+ (module|package) .+ removed", |
| 377 | DeprecationWarning, __name__) |
| 378 | warnings.filterwarnings("error", ".+ removed .+ (module|package)", |
Benjamin Peterson | a6864e0 | 2008-07-14 17:42:17 +0000 | [diff] [blame] | 379 | DeprecationWarning, __name__) |
| 380 | try: |
| 381 | __import__(module_name, level=0) |
| 382 | except DeprecationWarning as exc: |
Ezio Melotti | aa98058 | 2010-01-23 23:04:36 +0000 | [diff] [blame] | 383 | self.assertIn(module_name, exc.args[0], |
| 384 | "%s warning didn't contain module name" |
| 385 | % module_name) |
Benjamin Peterson | a6864e0 | 2008-07-14 17:42:17 +0000 | [diff] [blame] | 386 | except ImportError: |
| 387 | if not optional: |
| 388 | self.fail("Non-optional module {0} raised an " |
| 389 | "ImportError.".format(module_name)) |
| 390 | else: |
Florent Xicluna | 4d42f2b | 2010-03-09 19:57:01 +0000 | [diff] [blame] | 391 | # For extension modules, check the __warningregistry__. |
| 392 | # They won't rerun their init code even with CleanImport. |
| 393 | if not check_deprecated_module(module_name): |
| 394 | self.fail("DeprecationWarning not raised for {0}" |
| 395 | .format(module_name)) |
Brett Cannon | e5d2cba | 2008-05-06 23:23:34 +0000 | [diff] [blame] | 396 | |
| 397 | def test_platform_independent_removals(self): |
| 398 | # Make sure that the modules that are available on all platforms raise |
| 399 | # the proper DeprecationWarning. |
| 400 | for module_name in self.all_platforms: |
| 401 | self.check_removal(module_name) |
| 402 | |
Brett Cannon | 9ac3974 | 2008-05-09 22:51:58 +0000 | [diff] [blame] | 403 | def test_platform_specific_removals(self): |
| 404 | # Test the removal of platform-specific modules. |
| 405 | for module_name in self.inclusive_platforms.get(sys.platform, []): |
| 406 | self.check_removal(module_name, optional=True) |
| 407 | |
Brett Cannon | 768d44f | 2008-05-10 02:47:54 +0000 | [diff] [blame] | 408 | def test_optional_module_removals(self): |
| 409 | # Test the removal of modules that may or may not be built. |
| 410 | for module_name in self.optional_modules: |
| 411 | self.check_removal(module_name, optional=True) |
| 412 | |
Benjamin Peterson | 0893a0a | 2008-05-09 00:27:01 +0000 | [diff] [blame] | 413 | def test_os_path_walk(self): |
| 414 | msg = "In 3.x, os.path.walk is removed in favor of os.walk." |
| 415 | def dumbo(where, names, args): pass |
| 416 | for path_mod in ("ntpath", "macpath", "os2emxpath", "posixpath"): |
| 417 | mod = __import__(path_mod) |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 418 | reset_module_registry(mod) |
Florent Xicluna | 945a8ba | 2010-03-17 19:15:56 +0000 | [diff] [blame] | 419 | with check_py3k_warnings() as w: |
Benjamin Peterson | 1d31023 | 2008-05-27 01:42:29 +0000 | [diff] [blame] | 420 | mod.walk("crashers", dumbo, None) |
Ezio Melotti | 2623a37 | 2010-11-21 13:34:58 +0000 | [diff] [blame] | 421 | self.assertEqual(str(w.message), msg) |
Benjamin Peterson | 0893a0a | 2008-05-09 00:27:01 +0000 | [diff] [blame] | 422 | |
Benjamin Peterson | 541f7da | 2008-08-18 02:12:23 +0000 | [diff] [blame] | 423 | def test_reduce_move(self): |
| 424 | from operator import add |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 425 | # reduce tests may have already triggered this warning |
Florent Xicluna | 4d42f2b | 2010-03-09 19:57:01 +0000 | [diff] [blame] | 426 | reset_module_registry(unittest.case) |
Brett Cannon | 672237d | 2008-09-09 00:49:16 +0000 | [diff] [blame] | 427 | with warnings.catch_warnings(): |
Benjamin Peterson | 541f7da | 2008-08-18 02:12:23 +0000 | [diff] [blame] | 428 | warnings.filterwarnings("error", "reduce") |
| 429 | self.assertRaises(DeprecationWarning, reduce, add, range(10)) |
| 430 | |
Brett Cannon | abb34fe | 2008-05-29 05:08:50 +0000 | [diff] [blame] | 431 | def test_mutablestring_removal(self): |
| 432 | # UserString.MutableString has been removed in 3.0. |
| 433 | import UserString |
Nick Coghlan | d2e0938 | 2008-09-11 12:11:06 +0000 | [diff] [blame] | 434 | # UserString tests may have already triggered this warning |
| 435 | reset_module_registry(UserString) |
Brett Cannon | 672237d | 2008-09-09 00:49:16 +0000 | [diff] [blame] | 436 | with warnings.catch_warnings(): |
Brett Cannon | abb34fe | 2008-05-29 05:08:50 +0000 | [diff] [blame] | 437 | warnings.filterwarnings("error", ".*MutableString", |
| 438 | DeprecationWarning) |
| 439 | self.assertRaises(DeprecationWarning, UserString.MutableString) |
| 440 | |
Brett Cannon | e5d2cba | 2008-05-06 23:23:34 +0000 | [diff] [blame] | 441 | |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 442 | def test_main(): |
Florent Xicluna | 4d42f2b | 2010-03-09 19:57:01 +0000 | [diff] [blame] | 443 | run_unittest(TestPy3KWarnings, |
| 444 | TestStdlibRemovals) |
Steven Bethard | ae42f33 | 2008-03-18 17:26:10 +0000 | [diff] [blame] | 445 | |
| 446 | if __name__ == '__main__': |
| 447 | test_main() |