changeset:   82958:33bdd0a985b9
user:        Victor Stinner <victor.stinner@gmail.com>
date:        Tue Mar 26 01:14:08 2013 +0100
files:       Lib/test/test_bisect.py Lib/test/test_cmd_line.py Lib/test/test_long.py Lib/test/test_optparse.py
description:
Issue #17516: do not create useless tuple: remove dummy commas in tests


diff -r 313bcff51900 -r 33bdd0a985b9 Lib/test/test_bisect.py
--- a/Lib/test/test_bisect.py	Tue Mar 26 01:11:54 2013 +0100
+++ b/Lib/test/test_bisect.py	Tue Mar 26 01:14:08 2013 +0100
@@ -120,10 +120,10 @@
     def test_negative_lo(self):
         # Issue 3301
         mod = self.module
-        self.assertRaises(ValueError, mod.bisect_left, [1, 2, 3], 5, -1, 3),
-        self.assertRaises(ValueError, mod.bisect_right, [1, 2, 3], 5, -1, 3),
-        self.assertRaises(ValueError, mod.insort_left, [1, 2, 3], 5, -1, 3),
-        self.assertRaises(ValueError, mod.insort_right, [1, 2, 3], 5, -1, 3),
+        self.assertRaises(ValueError, mod.bisect_left, [1, 2, 3], 5, -1, 3)
+        self.assertRaises(ValueError, mod.bisect_right, [1, 2, 3], 5, -1, 3)
+        self.assertRaises(ValueError, mod.insort_left, [1, 2, 3], 5, -1, 3)
+        self.assertRaises(ValueError, mod.insort_right, [1, 2, 3], 5, -1, 3)
 
     def test_large_range(self):
         # Issue 13496
diff -r 313bcff51900 -r 33bdd0a985b9 Lib/test/test_cmd_line.py
--- a/Lib/test/test_cmd_line.py	Tue Mar 26 01:11:54 2013 +0100
+++ b/Lib/test/test_cmd_line.py	Tue Mar 26 01:14:08 2013 +0100
@@ -98,9 +98,9 @@
         assert_python_failure('-m', 'fnord43520xyz')
         # Check the runpy module also gives an error for
         # a nonexistent module
-        assert_python_failure('-m', 'runpy', 'fnord43520xyz'),
+        assert_python_failure('-m', 'runpy', 'fnord43520xyz')
         # All good if module is located and run successfully
-        assert_python_ok('-m', 'timeit', '-n', '1'),
+        assert_python_ok('-m', 'timeit', '-n', '1')
 
     def test_run_module_bug1764407(self):
         # -m and -i need to play well together
diff -r 313bcff51900 -r 33bdd0a985b9 Lib/test/test_long.py
--- a/Lib/test/test_long.py	Tue Mar 26 01:11:54 2013 +0100
+++ b/Lib/test/test_long.py	Tue Mar 26 01:14:08 2013 +0100
@@ -1086,7 +1086,7 @@
         self.assertRaises(OverflowError, (256).to_bytes, 1, 'big', signed=True)
         self.assertRaises(OverflowError, (256).to_bytes, 1, 'little', signed=False)
         self.assertRaises(OverflowError, (256).to_bytes, 1, 'little', signed=True)
-        self.assertRaises(OverflowError, (-1).to_bytes, 2, 'big', signed=False),
+        self.assertRaises(OverflowError, (-1).to_bytes, 2, 'big', signed=False)
         self.assertRaises(OverflowError, (-1).to_bytes, 2, 'little', signed=False)
         self.assertEqual((0).to_bytes(0, 'big'), b'')
         self.assertEqual((1).to_bytes(5, 'big'), b'\x00\x00\x00\x00\x01')
diff -r 313bcff51900 -r 33bdd0a985b9 Lib/test/test_optparse.py
--- a/Lib/test/test_optparse.py	Tue Mar 26 01:11:54 2013 +0100
+++ b/Lib/test/test_optparse.py	Tue Mar 26 01:14:08 2013 +0100
@@ -730,7 +730,7 @@
     def test_short_and_long_option_split(self):
         self.assertParseOK(["-a", "xyz", "--foo", "bar"],
                            {'a': 'xyz', 'boo': None, 'foo': ["bar"]},
-                           []),
+                           [])
 
     def test_short_option_split_long_option_append(self):
         self.assertParseOK(["--foo=bar", "-b", "123", "--foo", "baz"],
@@ -740,15 +740,15 @@
     def test_short_option_split_one_positional_arg(self):
         self.assertParseOK(["-a", "foo", "bar"],
                            {'a': "foo", 'boo': None, 'foo': None},
-                           ["bar"]),
+                           ["bar"])
 
     def test_short_option_consumes_separator(self):
         self.assertParseOK(["-a", "--", "foo", "bar"],
                            {'a': "--", 'boo': None, 'foo': None},
-                           ["foo", "bar"]),
+                           ["foo", "bar"])
         self.assertParseOK(["-a", "--", "--foo", "bar"],
                            {'a': "--", 'boo': None, 'foo': ["bar"]},
-                           []),
+                           [])
 
     def test_short_option_joined_and_separator(self):
         self.assertParseOK(["-ab", "--", "--foo", "bar"],