Skip to content

bpo-45243: Use connection limits to simplify sqlite3 tests #29356

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
serhiy-storchaka merged 13 commits into from
Nov 5, 2021
Merged

bpo-45243: Use connection limits to simplify sqlite3 tests #29356

Show file tree
Hide file tree
Changes from 1 commit
Commits
Show all changes
13 commits
Select commit Hold shift + click to select a range
8cdb980
bpo-45243: Use sqlite3 connection limit context manager in test suite
Nov 2, 2021
d68d6d8
Add test for too many SQL variables
Nov 2, 2021
22adb06
Adjust comment
Nov 2, 2021
cb56249
Address review
Nov 2, 2021
3a75c2d
Address review: test lim-1 and lim
Nov 4, 2021
a6da634
Address review: make it clear which function is failing
Nov 4, 2021
3f1ee2a
Merge remote-tracking branch 'upstream/main' into sqlite-use-limits-i…
Nov 4, 2021
b98d623
Update Lib/test/test_sqlite3/test_dbapi.py
Nov 5, 2021
9521fc1
Simplify test_func_too_many_args further
Nov 5, 2021
c53ffd7
Add single-argument test
Nov 5, 2021
6197b3a
Simplify sql statements in test_execute_too_many_params
Nov 5, 2021
032bfd5
Use abs iso. round
Nov 5, 2021
c257c65
Make sure we test against correct limit in test_execute_too_many_params
Nov 5, 2021
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Use abs iso. round 
build-in round is missing if SQLITE_OMIT_FLOATING_POINT is defined
  • Loading branch information
Erlend E. Aasland
Erlend E. Aasland committed Nov 5, 2021
commit 032bfd5a9941d1672f830d5c9580bda226ab0895
2 changes: 1 addition & 1 deletion Lib/test/test_sqlite3/test_dbapi.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -669,7 +669,7 @@ def test_execute_too_many_params(self):
category = sqlite.SQLITE_LIMIT_VARIABLE_NUMBER
msg = "too many SQL variables"
with cx_limit(self.cx, category=category, limit=1):
Comment thread
erlend-aasland marked this conversation as resolved.
self.cu.execute("select round(?)", (1.1,))
self.cu.execute("select abs(?)", (-1,))
with self.assertRaisesRegex(sqlite.OperationalError, msg):
self.cu.execute("select max(?, ?)", (1, 2))
Comment thread
erlend-aasland marked this conversation as resolved.
Outdated

Expand Down
2 changes: 1 addition & 1 deletion Lib/test/test_sqlite3/test_userfunctions.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -228,7 +228,7 @@ def test_func_too_many_args(self):
category = sqlite.SQLITE_LIMIT_FUNCTION_ARG
msg = "too many arguments on function"
with cx_limit(self.con, category=category, limit=1):
self.con.execute("select round(1.1)");
self.con.execute("select abs(-1)");
with self.assertRaisesRegex(sqlite.OperationalError, msg):
self.con.execute("select max(1, 2)");

Expand Down