[Python-checkins] bpo-46198: rename duplicate tests and remove unused code (GH-30297)
JelleZijlstra
webhook-mailer at python.org
Thu Mar 10 11:20:24 EST 2022
https://github.com/python/cpython/commit/6c83c8e6b56b57a8a794e7b6c07837be4ce3bb97
commit: 6c83c8e6b56b57a8a794e7b6c07837be4ce3bb97
branch: main
author: Nikita Sobolev <mail at sobolevn.me>
committer: JelleZijlstra <jelle.zijlstra at gmail.com>
date: 2022-03-10T08:20:11-08:00
summary:
bpo-46198: rename duplicate tests and remove unused code (GH-30297)
files:
M Lib/test/datetimetester.py
M Lib/test/support/__init__.py
M Lib/test/test_asyncio/test_sslproto.py
M Lib/test/test_codecs.py
M Lib/test/test_compile.py
M Lib/test/test_dict.py
M Lib/test/test_email/test__header_value_parser.py
M Lib/test/test_enum.py
M Lib/test/test_pty.py
M Lib/test/test_tabnanny.py
diff --git a/Lib/test/datetimetester.py b/Lib/test/datetimetester.py
index 810478c7db2be..df3764d61b186 100644
--- a/Lib/test/datetimetester.py
+++ b/Lib/test/datetimetester.py
@@ -1864,8 +1864,6 @@ def test_fromisoformat_fails(self):
def test_fromisoformat_fails_typeerror(self):
# Test that fromisoformat fails when passed the wrong type
- import io
-
bad_types = [b'2009-03-01', None, io.StringIO('2009-03-01')]
for bad_type in bad_types:
with self.assertRaises(TypeError):
@@ -3988,8 +3986,6 @@ def test_fromisoformat_fails(self):
def test_fromisoformat_fails_typeerror(self):
# Test the fromisoformat fails when passed the wrong type
- import io
-
bad_types = [b'12:30:45', None, io.StringIO('12:30:45')]
for bad_type in bad_types:
diff --git a/Lib/test/support/__init__.py b/Lib/test/support/__init__.py
index 865fb976f532e..c7bee8be662cc 100644
--- a/Lib/test/support/__init__.py
+++ b/Lib/test/support/__init__.py
@@ -1442,9 +1442,6 @@ def __init__(self, link=None):
self._platform_specific()
- def _platform_specific(self):
- pass
-
if sys.platform == "win32":
def _platform_specific(self):
import glob
@@ -1472,6 +1469,9 @@ def _platform_specific(self):
self._env["PYTHONHOME"] = os.path.dirname(self.real)
if sysconfig.is_python_build(True):
self._env["PYTHONPATH"] = STDLIB_DIR
+ else:
+ def _platform_specific(self):
+ pass
def __enter__(self):
os.symlink(self.real, self.link)
diff --git a/Lib/test/test_asyncio/test_sslproto.py b/Lib/test/test_asyncio/test_sslproto.py
index 6e8de7c84c0aa..4095b4dfa84eb 100644
--- a/Lib/test/test_asyncio/test_sslproto.py
+++ b/Lib/test/test_asyncio/test_sslproto.py
@@ -2,7 +2,6 @@
import logging
import socket
-from test import support
import unittest
import weakref
from unittest import mock
diff --git a/Lib/test/test_codecs.py b/Lib/test/test_codecs.py
index 8118ec6f4727b..5853e08819715 100644
--- a/Lib/test/test_codecs.py
+++ b/Lib/test/test_codecs.py
@@ -36,7 +36,7 @@ def check(input, expect):
# On small versions of Windows like Windows IoT or Windows Nano Server not all codepages are present
def is_code_page_present(cp):
from ctypes import POINTER, WINFUNCTYPE, WinDLL
- from ctypes.wintypes import BOOL, UINT, BYTE, WCHAR, UINT, DWORD
+ from ctypes.wintypes import BOOL, BYTE, WCHAR, UINT, DWORD
MAX_LEADBYTES = 12 # 5 ranges, 2 bytes ea., 0 term.
MAX_DEFAULTCHAR = 2 # single or double byte
diff --git a/Lib/test/test_compile.py b/Lib/test/test_compile.py
index 79046b8615e3e..8499d2855bdd4 100644
--- a/Lib/test/test_compile.py
+++ b/Lib/test/test_compile.py
@@ -1189,7 +1189,7 @@ def test_func_kwargs(self):
kwargs = (f'a{i}=x' for i in range(self.N))
self.check_stack_size("f(" + ", ".join(kwargs) + ")")
- def test_func_args(self):
+ def test_meth_args(self):
self.check_stack_size("o.m(" + "x, " * self.N + ")")
def test_meth_kwargs(self):
diff --git a/Lib/test/test_dict.py b/Lib/test/test_dict.py
index e60ae4309cbc1..5b8baaf9e6e28 100644
--- a/Lib/test/test_dict.py
+++ b/Lib/test/test_dict.py
@@ -1455,7 +1455,7 @@ def test_dict_items_result_gc(self):
self.assertTrue(gc.is_tracked(next(it)))
@support.cpython_only
- def test_dict_items_result_gc(self):
+ def test_dict_items_result_gc_reversed(self):
# Same as test_dict_items_result_gc above, but reversed.
it = reversed({None: []}.items())
gc.collect()
diff --git a/Lib/test/test_email/test__header_value_parser.py b/Lib/test/test_email/test__header_value_parser.py
index 1bdcfa129b4c8..854f2ff009c61 100644
--- a/Lib/test/test_email/test__header_value_parser.py
+++ b/Lib/test/test_email/test__header_value_parser.py
@@ -395,7 +395,7 @@ def test_get_unstructured_without_trailing_whitespace_hang_case(self):
[errors.InvalidHeaderDefect],
'')
- def test_get_unstructured_invalid_ew(self):
+ def test_get_unstructured_invalid_ew2(self):
self._test_get_x(self._get_unst,
'=?utf-8?q?=somevalue?=',
'=?utf-8?q?=somevalue?=',
diff --git a/Lib/test/test_enum.py b/Lib/test/test_enum.py
index b8a7914355c53..f2572b2ac351a 100644
--- a/Lib/test/test_enum.py
+++ b/Lib/test/test_enum.py
@@ -3708,7 +3708,7 @@ class Sillier(IntEnum):
triple = 3
value = 4
-class TestHelpers(unittest.TestCase):
+class TestInternals(unittest.TestCase):
sunder_names = '_bad_', '_good_', '_what_ho_'
dunder_names = '__mal__', '__bien__', '__que_que__'
diff --git a/Lib/test/test_pty.py b/Lib/test/test_pty.py
index 0781cde1e1582..fa0dbcc16f3ce 100644
--- a/Lib/test/test_pty.py
+++ b/Lib/test/test_pty.py
@@ -17,7 +17,6 @@
import unittest
import struct
-import tty
import fcntl
import warnings
diff --git a/Lib/test/test_tabnanny.py b/Lib/test/test_tabnanny.py
index 59fdfc5573d37..e0a82e95c486b 100644
--- a/Lib/test/test_tabnanny.py
+++ b/Lib/test/test_tabnanny.py
@@ -4,7 +4,6 @@
* errored : Whitespace related problems present in file.
"""
from unittest import TestCase, mock
-from unittest import mock
import errno
import os
import tabnanny
More information about the Python-checkins
mailing list