Use TESTFN iso. hard-coded test file paths in sqlite3 tests

Current code:

cpython/Lib/test/test_sqlite3/test_transactions.py

Lines 28 to 54 in 5bcf33d

    
           def get_db_path(): 
        
               return "sqlite_testdb" 
        
           class TransactionTests(unittest.TestCase): 
        
               def setUp(self): 
        
                   try: 
        
                       os.remove(get_db_path()) 
        
                   except OSError: 
        
                       pass 
        
                   self.con1 = sqlite.connect(get_db_path(), timeout=0.1) 
        
                   self.cur1 = self.con1.cursor() 
        
                   self.con2 = sqlite.connect(get_db_path(), timeout=0.1) 
        
                   self.cur2 = self.con2.cursor() 
        
               def tearDown(self): 
        
                   self.cur1.close() 
        
                   self.con1.close() 
        
                   self.cur2.close() 
        
                   self.con2.close() 
        
                   try: 
        
                       os.unlink(get_db_path()) 
        
                   except OSError: 
        
                       pass

Suggesting to use the test.support helpers TESTFN and unlink.

We can also swap the timeout with one of the timeout constants in test.support.

pythongh-93795: Use test.support TESTFN/unlink in sqlite3 tests

gh-93795: Use test.support TESTFN/unlink in sqlite3 tests (#93796)

[3.11] pythongh-93795: Use test.support TESTFN/unlink in sqlite3 tests (

[3.10] pythongh-93795: Use test.support TESTFN/unlink in sqlite3 tests (

[3.11] gh-93795: Use test.support TESTFN/unlink in sqlite3 tests (GH-…

[3.10] gh-93795: Use test.support TESTFN/unlink in sqlite3 tests (GH-…

Revert 30935 defer bpo45162 to 312 (#94285)

Navigation Menu

Repository navigation

Use TESTFN iso. hard-coded test file paths in sqlite3 tests #93795

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

	def get_db_path():
	return "sqlite_testdb"

	class TransactionTests(unittest.TestCase):
	def setUp(self):
	try:
	os.remove(get_db_path())
	except OSError:
	pass

	self.con1 = sqlite.connect(get_db_path(), timeout=0.1)
	self.cur1 = self.con1.cursor()

	self.con2 = sqlite.connect(get_db_path(), timeout=0.1)
	self.cur2 = self.con2.cursor()

	def tearDown(self):
	self.cur1.close()
	self.con1.close()

	self.cur2.close()
	self.con2.close()

	try:
	os.unlink(get_db_path())
	except OSError:
	pass

Use TESTFN iso. hard-coded test file paths in sqlite3 tests #93795

Description

Activity

erlend-aasland commented on Jun 14, 2022

Metadata

Metadata

Assignees

Labels

Projects

Milestone

Relationships

Development

Issue actions