Uh oh!
There was an error while loading.Please reload this page.
- Notifications
You must be signed in to change notification settings - Fork32k
Closed
Description
Lib/test/test_sqlite3/test_types.py has two bigmem tests:
test_too_large_string; andtest_too_large_blob.
Those are skipped unless-M is passed to the test runner so nobody was running those tests untilI set up a bigmem buildbot. Running tests on the buildbot revealed two failures:
======================================================================ERROR: test_too_large_blob (test.test_sqlite3.test_types.SqliteTypeTests.test_too_large_blob)----------------------------------------------------------------------Traceback (most recent call last): File "R:\buildarea\3.x.ambv-bb-win11.bigmem\build\Lib\test\support\__init__.py", line 967, in wrapper return f(self, maxsize) ^^^^^^^^^^^^^^^^ File "R:\buildarea\3.x.ambv-bb-win11.bigmem\build\Lib\test\test_sqlite3\test_types.py", line 121, in test_too_large_blob self.cur.execute("insert into test(s) values (?)", (b'x'*(2**31-1),))sqlite3.DataError: string or blob too bigStdout: ... expected peak memory use: 6.0GStderr:R:\buildarea\3.x.ambv-bb-win11.bigmem\build\Lib\test\support\__init__.py:910: RuntimeWarning: /proc not available for stats: [Errno 2] No such file or directory: '/proc/10708/statm' warnings.warn('/proc not available for stats: {}'.format(e),======================================================================ERROR: test_too_large_string (test.test_sqlite3.test_types.SqliteTypeTests.test_too_large_string)----------------------------------------------------------------------Traceback (most recent call last): File "R:\buildarea\3.x.ambv-bb-win11.bigmem\build\Lib\test\support\__init__.py", line 967, in wrapper return f(self, maxsize) ^^^^^^^^^^^^^^^^ File "R:\buildarea\3.x.ambv-bb-win11.bigmem\build\Lib\test\test_sqlite3\test_types.py", line 110, in test_too_large_string self.cur.execute("insert into test(s) values (?)", ('x'*(2**31-1),))sqlite3.DataError: string or blob too bigStdout: ... expected peak memory use: 8.0G----------------------------------------------------------------------Thewith self.assertRaises() in those tests should catchsqlite.DataError instead of the exceptions currently listed.