diff options
author | Miss Islington (bot) <31488909+miss-islington@users.noreply.github.com> | 2021-06-23 10:03:00 (GMT) |
---|---|---|
committer | GitHub <noreply@github.com> | 2021-06-23 10:03:00 (GMT) |
commit | fcde2c6a8c99a56576b25733d5cc60bce6d51f46 (patch) | |
tree | 5b369bec44a2bd6534d5ace1e51d3d1fe21fe8f5 | |
parent | ef89b2bf42650accd460973f2ecb4f8d661fa5c4 (diff) | |
download | cpython-fcde2c6a8c99a56576b25733d5cc60bce6d51f46.zip cpython-fcde2c6a8c99a56576b25733d5cc60bce6d51f46.tar.gz cpython-fcde2c6a8c99a56576b25733d5cc60bce6d51f46.tar.bz2 |
bpo-28395: Remove unnecessary semicolons in tests (GH-26868)
(cherry picked from commit 5a3108044d2e5b694da2d1f4176c9bbaef15c142)
Co-authored-by: Dong-hee Na <donghee.na@python.org>
-rw-r--r-- | Lib/test/test_capi.py | 4 | ||||
-rw-r--r-- | Lib/test/test_codeop.py | 30 | ||||
-rw-r--r-- | Lib/test/test_csv.py | 2 | ||||
-rw-r--r-- | Lib/test/test_deque.py | 7 | ||||
-rw-r--r-- | Lib/test/test_float.py | 8 | ||||
-rw-r--r-- | Lib/test/test_heapq.py | 4 | ||||
-rw-r--r-- | Lib/test/test_import/__init__.py | 4 | ||||
-rw-r--r-- | Lib/test/test_set.py | 4 |
8 files changed, 32 insertions, 31 deletions
diff --git a/Lib/test/test_capi.py b/Lib/test/test_capi.py index 5150d57..b21c3ac 100644 --- a/Lib/test/test_capi.py +++ b/Lib/test/test_capi.py @@ -553,11 +553,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 66caf5a..98f4be2 100644 --- a/Lib/test/test_codeop.py +++ b/Lib/test/test_codeop.py @@ -178,21 +178,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("{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("{") + 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 d421be0..2dab17d 100644 --- a/Lib/test/test_csv.py +++ b/Lib/test/test_csv.py @@ -156,7 +156,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 c0f7138..67c0ace 100644 --- a/Lib/test/test_deque.py +++ b/Lib/test/test_deque.py @@ -148,7 +148,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))) @@ -562,8 +563,8 @@ class TestBasic(unittest.TestCase): support.unlink(support.TESTFN) 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 9651281..4f66b41 100644 --- a/Lib/test/test_float.py +++ b/Lib/test/test_float.py @@ -1162,10 +1162,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 6902573..2cca9a0 100644 --- a/Lib/test/test_heapq.py +++ b/Lib/test/test_heapq.py @@ -145,11 +145,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 35f1484..538b22d 100644 --- a/Lib/test/test_import/__init__.py +++ b/Lib/test/test_import/__init__.py @@ -1084,7 +1084,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) @@ -1093,7 +1093,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 e4766ab..faf25a7 100644 --- a/Lib/test/test_set.py +++ b/Lib/test/test_set.py @@ -375,8 +375,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)) |