summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMiss Islington (bot) <31488909+miss-islington@users.noreply.github.com>2021-06-23 10:03:00 (GMT)
committerGitHub <noreply@github.com>2021-06-23 10:03:00 (GMT)
commitfcde2c6a8c99a56576b25733d5cc60bce6d51f46 (patch)
tree5b369bec44a2bd6534d5ace1e51d3d1fe21fe8f5
parentef89b2bf42650accd460973f2ecb4f8d661fa5c4 (diff)
downloadcpython-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.py4
-rw-r--r--Lib/test/test_codeop.py30
-rw-r--r--Lib/test/test_csv.py2
-rw-r--r--Lib/test/test_deque.py7
-rw-r--r--Lib/test/test_float.py8
-rw-r--r--Lib/test/test_heapq.py4
-rw-r--r--Lib/test/test_import/__init__.py4
-rw-r--r--Lib/test/test_set.py4
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))