Skip to content

Commit 958a25e

Browse files
Fixed several assertTrue() that were intended to be assertEqual(). (GH-8191)
Fixed also testing the "always" warning filter. (cherry picked from commit b796e7d) Co-authored-by: Sergey Fedoseev <[email protected]>
1 parent 1b6e217 commit 958a25e

File tree

6 files changed

+13
-9
lines changed

6 files changed

+13
-9
lines changed

Lib/ctypes/test/test_as_parameter.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -24,7 +24,7 @@ def test_wchar_parm(self):
2424
f.argtypes = [c_byte, c_wchar, c_int, c_long, c_float, c_double]
2525
result = f(self.wrap(1), self.wrap("x"), self.wrap(3), self.wrap(4), self.wrap(5.0), self.wrap(6.0))
2626
self.assertEqual(result, 139)
27-
self.assertTrue(type(result), int)
27+
self.assertIs(type(result), int)
2828

2929
def test_pointers(self):
3030
f = dll._testfunc_p_p

Lib/test/test_pkg.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ def test_2(self):
138138

139139
s = """
140140
from t2 import *
141-
self.assertTrue(dir(), ['self', 'sub'])
141+
self.assertEqual(dir(), ['self', 'sub'])
142142
"""
143143
self.run_code(s)
144144

Lib/test/test_socket.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -5843,7 +5843,7 @@ def test_aes_cbc(self):
58435843
op=socket.ALG_OP_ENCRYPT, iv=iv)
58445844
enc = op.recv(msglen * multiplier)
58455845
self.assertEqual(len(enc), msglen * multiplier)
5846-
self.assertTrue(enc[:msglen], ciphertext)
5846+
self.assertEqual(enc[:msglen], ciphertext)
58475847

58485848
op, _ = algo.accept()
58495849
with op:

Lib/test/test_support.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -264,7 +264,7 @@ def test_temp_cwd(self):
264264
with support.temp_cwd(name=TESTFN):
265265
self.assertEqual(os.path.basename(os.getcwd()), TESTFN)
266266
self.assertFalse(os.path.exists(TESTFN))
267-
self.assertTrue(os.path.basename(os.getcwd()), here)
267+
self.assertEqual(os.getcwd(), here)
268268

269269

270270
def test_temp_cwd__name_none(self):

Lib/test/test_tokenize.py

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1352,7 +1352,7 @@ def mock_readline():
13521352
tokenize_module.detect_encoding = orig_detect_encoding
13531353
tokenize_module._tokenize = orig__tokenize
13541354

1355-
self.assertTrue(encoding_used, encoding)
1355+
self.assertEqual(encoding_used, encoding)
13561356

13571357
def test_oneline_defs(self):
13581358
buf = []

Lib/test/test_warnings/__init__.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -148,10 +148,14 @@ def test_always(self):
148148
self.module.resetwarnings()
149149
self.module.filterwarnings("always", category=UserWarning)
150150
message = "FilterTests.test_always"
151-
self.module.warn(message, UserWarning)
152-
self.assertTrue(message, w[-1].message)
153-
self.module.warn(message, UserWarning)
154-
self.assertTrue(w[-1].message, message)
151+
def f():
152+
self.module.warn(message, UserWarning)
153+
f()
154+
self.assertEqual(len(w), 1)
155+
self.assertEqual(w[-1].message.args[0], message)
156+
f()
157+
self.assertEqual(len(w), 2)
158+
self.assertEqual(w[-1].message.args[0], message)
155159

156160
def test_always_after_default(self):
157161
with original_warnings.catch_warnings(record=True,

0 commit comments

Comments
 (0)