]> git.ipfire.org Git - thirdparty/Python/cpython.git/commitdiff
gh-108963: using random to generate unique string in sys.intern test (#109491)
authorAN Long <aisk@users.noreply.github.com>
Mon, 2 Oct 2023 11:07:56 +0000 (19:07 +0800)
committerGitHub <noreply@github.com>
Mon, 2 Oct 2023 11:07:56 +0000 (13:07 +0200)
Lib/test/test_sys.py

index 16050171ad139d352a1cf07b5dbee04d0bf3d4d5..ae241d7a502749aabef1f2fd84da219a8cc3b937 100644 (file)
@@ -5,6 +5,7 @@ import gc
 import locale
 import operator
 import os
+import random
 import struct
 import subprocess
 import sys
@@ -30,10 +31,6 @@ def requires_subinterpreters(meth):
                            'subinterpreters required')(meth)
 
 
-# count the number of test runs, used to create unique
-# strings to intern in test_intern()
-INTERN_NUMRUNS = 0
-
 DICT_KEY_STRUCT_FORMAT = 'n2BI2n'
 
 class DisplayHookTest(unittest.TestCase):
@@ -696,10 +693,8 @@ class SysModuleTest(unittest.TestCase):
         self.assertEqual(sys.__stdout__.encoding, sys.__stderr__.encoding)
 
     def test_intern(self):
-        global INTERN_NUMRUNS
-        INTERN_NUMRUNS += 1
         self.assertRaises(TypeError, sys.intern)
-        s = "never interned before" + str(INTERN_NUMRUNS)
+        s = "never interned before" + str(random.randrange(0, 10**9))
         self.assertTrue(sys.intern(s) is s)
         s2 = s.swapcase().swapcase()
         self.assertTrue(sys.intern(s2) is s)
@@ -717,9 +712,7 @@ class SysModuleTest(unittest.TestCase):
 
     @requires_subinterpreters
     def test_subinterp_intern_dynamically_allocated(self):
-        global INTERN_NUMRUNS
-        INTERN_NUMRUNS += 1
-        s = "never interned before" + str(INTERN_NUMRUNS)
+        s = "never interned before" + str(random.randrange(0, 10**9))
         t = sys.intern(s)
         self.assertIs(t, s)