From 5a3108044d2e5b694da2d1f4176c9bbaef15c142 Mon Sep 17 00:00:00 2001 From: Dong-hee Na Date: Wed, 23 Jun 2021 18:01:06 +0900 Subject: [PATCH] bpo-28395: Remove unnecessary semicolons in tests (GH-26868) --- Lib/test/test_capi.py | 4 ++-- Lib/test/test_codeop.py | 28 ++++++++++++++-------------- Lib/test/test_csv.py | 2 +- Lib/test/test_deque.py | 7 ++++--- Lib/test/test_float.py | 8 ++++---- Lib/test/test_heapq.py | 4 ++-- Lib/test/test_import/__init__.py | 4 ++-- Lib/test/test_set.py | 4 ++-- 8 files changed, 31 insertions(+), 30 deletions(-) diff --git a/Lib/test/test_capi.py b/Lib/test/test_capi.py index f4b7b8c13b7..169e7acbf92 100644 --- a/Lib/test/test_capi.py +++ b/Lib/test/test_capi.py @@ -635,11 +635,11 @@ class TestPendingCalls(unittest.TestCase): #unsuccessful. while True: if _testcapi._pending_threadfunc(callback): - break; + break def pendingcalls_wait(self, l, n, context = None): #now, stick around until l[0] has grown to 10 - count = 0; + count = 0 while len(l) != n: #this busy loop is where we expect to be interrupted to #run our callbacks. Note that callbacks are only run on the diff --git a/Lib/test/test_codeop.py b/Lib/test/test_codeop.py index 07b46afbe1b..17376c7ed75 100644 --- a/Lib/test/test_codeop.py +++ b/Lib/test/test_codeop.py @@ -182,21 +182,21 @@ class CodeopTests(unittest.TestCase): ai("from a import (b,c") ai("from a import (b,c,") - ai("["); - ai("[a"); - ai("[a,"); - ai("[a,b"); - ai("[a,b,"); + ai("[") + ai("[a") + ai("[a,") + ai("[a,b") + ai("[a,b,") - ai("{"); - ai("{a"); - ai("{a:"); - ai("{a:b"); - ai("{a:b,"); - ai("{a:b,c"); - ai("{a:b,c:"); - ai("{a:b,c:d"); - ai("{a:b,c:d,"); + ai("{") + ai("{a") + ai("{a:") + ai("{a:b") + ai("{a:b,") + ai("{a:b,c") + ai("{a:b,c:") + ai("{a:b,c:d") + ai("{a:b,c:d,") ai("a(") ai("a(b") diff --git a/Lib/test/test_csv.py b/Lib/test/test_csv.py index a1e050acd2a..225f5c72890 100644 --- a/Lib/test/test_csv.py +++ b/Lib/test/test_csv.py @@ -157,7 +157,7 @@ class Test_Csv(unittest.TestCase): self._write_error_test(OSError, BadIterable()) class BadList: def __len__(self): - return 10; + return 10 def __getitem__(self, i): if i > 2: raise OSError diff --git a/Lib/test/test_deque.py b/Lib/test/test_deque.py index f1a79373dec..ffe0e204a0e 100644 --- a/Lib/test/test_deque.py +++ b/Lib/test/test_deque.py @@ -129,7 +129,8 @@ class TestBasic(unittest.TestCase): self.assertEqual(d.count(None), 16) def test_comparisons(self): - d = deque('xabc'); d.popleft() + d = deque('xabc') + d.popleft() for e in [d, deque('abc'), deque('ab'), deque(), list(d)]: self.assertEqual(d==e, type(d)==type(e) and list(d)==list(e)) self.assertEqual(d!=e, not(type(d)==type(e) and list(d)==list(e))) @@ -529,8 +530,8 @@ class TestBasic(unittest.TestCase): self.assertEqual(repr(d)[-20:], '7, 198, 199, [...]])') def test_init(self): - self.assertRaises(TypeError, deque, 'abc', 2, 3); - self.assertRaises(TypeError, deque, 1); + self.assertRaises(TypeError, deque, 'abc', 2, 3) + self.assertRaises(TypeError, deque, 1) def test_hash(self): self.assertRaises(TypeError, hash, deque('abc')) diff --git a/Lib/test/test_float.py b/Lib/test/test_float.py index f0ed40f7c94..38a17cedd64 100644 --- a/Lib/test/test_float.py +++ b/Lib/test/test_float.py @@ -1184,10 +1184,10 @@ class HexFloatTestCase(unittest.TestCase): def test_from_hex(self): - MIN = self.MIN; - MAX = self.MAX; - TINY = self.TINY; - EPS = self.EPS; + MIN = self.MIN + MAX = self.MAX + TINY = self.TINY + EPS = self.EPS # two spellings of infinity, with optional signs; case-insensitive self.identical(fromHex('inf'), INF) diff --git a/Lib/test/test_heapq.py b/Lib/test/test_heapq.py index ebbc6274570..cb1e4505b02 100644 --- a/Lib/test/test_heapq.py +++ b/Lib/test/test_heapq.py @@ -146,11 +146,11 @@ class TestHeap: self.assertEqual(type(h[0]), int) self.assertEqual(type(x), float) - h = [10]; + h = [10] x = self.module.heappushpop(h, 9) self.assertEqual((h, x), ([10], 9)) - h = [10]; + h = [10] x = self.module.heappushpop(h, 11) self.assertEqual((h, x), ([11], 10)) diff --git a/Lib/test/test_import/__init__.py b/Lib/test/test_import/__init__.py index 8fe3e1d4bf3..97447619b90 100644 --- a/Lib/test/test_import/__init__.py +++ b/Lib/test/test_import/__init__.py @@ -1086,7 +1086,7 @@ class GetSourcefileTests(unittest.TestCase): # Given a valid bytecode path, return the path to the corresponding # source file if it exists. with mock.patch('importlib._bootstrap_external._path_isfile') as _path_isfile: - _path_isfile.return_value = True; + _path_isfile.return_value = True path = TESTFN + '.pyc' expect = TESTFN + '.py' self.assertEqual(_get_sourcefile(path), expect) @@ -1095,7 +1095,7 @@ class GetSourcefileTests(unittest.TestCase): # Given a valid bytecode path without a corresponding source path, # return the original bytecode path. with mock.patch('importlib._bootstrap_external._path_isfile') as _path_isfile: - _path_isfile.return_value = False; + _path_isfile.return_value = False path = TESTFN + '.pyc' self.assertEqual(_get_sourcefile(path), path) diff --git a/Lib/test/test_set.py b/Lib/test/test_set.py index e45f018d2da..b1fab0f6207 100644 --- a/Lib/test/test_set.py +++ b/Lib/test/test_set.py @@ -362,8 +362,8 @@ class TestSet(TestJointOps, unittest.TestCase): self.assertEqual(s, set(self.word)) s.__init__(self.otherword) self.assertEqual(s, set(self.otherword)) - self.assertRaises(TypeError, s.__init__, s, 2); - self.assertRaises(TypeError, s.__init__, 1); + self.assertRaises(TypeError, s.__init__, s, 2) + self.assertRaises(TypeError, s.__init__, 1) def test_constructor_identity(self): s = self.thetype(range(3))