Skip to content

gh-79009: sqlite3.iterdump now correctly handles tables with autoincrement #9621

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
erlend-aasland merged 15 commits into from
Jun 19, 2022
Merged

gh-79009: sqlite3.iterdump now correctly handles tables with autoincrement #9621

Changes from 1 commit
Commits
Show all changes
15 commits
Select commit Hold shift + click to select a range
9bf9eb7
bpo-34828 sqlite3.iterdump now correctly handles tables with autoincr…
itssme Sep 28, 2018
5d7b0ae
bpo-34828: added entry in NEWS.d/next/Library and in ACKS
itssme Sep 28, 2018
1ac20b0
Merge remote-tracking branch 'origin/master' into fix-issue-34828
itssme Jul 3, 2021
365a2f1
bpo-34828 Removed redundant code and made some small formatting chang…
itssme Jul 3, 2021
7c5901b
Merge branch 'main' into fix-issue-34828
itssme Oct 15, 2021
b259219
bpo-34828 formatting changes as well as readability improvements in s…
itssme Oct 15, 2021
3812bf6
Merge branch 'main' into fix-issue-34828
itssme May 22, 2022
daed6e5
bpo-34828: reformatted some tests in test_sqlite3/test.dump.py as wel…
itssme May 22, 2022
d5daa91
Merge branch 'main' into fix-issue-34828
erlend-aasland Jun 9, 2022
4e573e9
bpo-34828: minified test case for some sqlite3 dump command tests, fi…
itssme Jun 17, 2022
64f4101
Merge branch 'main' into fix-issue-34828
erlend-aasland Jun 19, 2022
1abc7d0
Clean up dump.py
erlend-aasland Jun 19, 2022
6bfac7e
Cleanup: don't use abbreviated link in NEWS entry
erlend-aasland Jun 19, 2022
103c50f
Cleanup: tests
erlend-aasland Jun 19, 2022
53076e9
Fix 1abc7d05b7a605224611a09eaa74e50ef1cc38e1
erlend-aasland Jun 19, 2022
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
bpo-34828: minified test case for some sqlite3 dump command tests, fi… 
…xed wrong use of double quotes and single quotes in sqlite3 statements
  • Loading branch information
@itssme
itssme committed Jun 17, 2022
commit 4e573e9c12c32cedbe64a2be5cfdc9a4d1e97a36
32 changes: 14 additions & 18 deletions Lib/test/test_sqlite3/test_dump.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -53,11 +53,9 @@ def test_table_dump(self):

def test_dump_autoincrement(self):
expected_sqls = [
"CREATE TABLE \"posts\" (id int primary key);",
"INSERT INTO \"posts\" VALUES(0);",
"CREATE TABLE \"tags\" (id integer primary key autoincrement, tag varchar(256), post int references posts);",
"INSERT INTO \"tags\" VALUES(NULL,'test',0);",
"CREATE TABLE \"tags2\" (id integer primary key autoincrement, tag varchar(256), post int references posts);"
"CREATE TABLE \"t1\" (id integer primary key autoincrement);",
"INSERT INTO \"t1\" VALUES(NULL);",
"CREATE TABLE \"t2\" (id integer primary key autoincrement);"
]

for sql_statement in expected_sqls:
Expand All @@ -66,11 +64,11 @@ def test_dump_autoincrement(self):
actual_sqls = [sql_statement for sql_statement in iterdump]

# the NULL value should now be automatically be set to 1
expected_sqls[3] = expected_sqls[3].replace("NULL", "1")
expected_sqls[1] = expected_sqls[1].replace("NULL", "1")
expected_sqls.insert(0, "BEGIN TRANSACTION;")
expected_sqls.extend([
"DELETE FROM \"sqlite_sequence\";",
"INSERT INTO \"sqlite_sequence\" VALUES('tags',1);",
"INSERT INTO \"sqlite_sequence\" VALUES('t1',1);",
"COMMIT;"
])

Expand All @@ -79,15 +77,13 @@ def test_dump_autoincrement(self):
def test_dump_autoincrement_create_new_db(self):
old_db = [
"BEGIN TRANSACTION ;",
"CREATE TABLE \"posts\" (id int primary key);",
"INSERT INTO \"posts\" VALUES(0);",
"CREATE TABLE \"tags\" (id integer primary key autoincrement, tag varchar(256), post int references posts);",
"CREATE TABLE \"tags2\" (id integer primary key autoincrement, tag varchar(256), post int references posts);"
"CREATE TABLE \"t1\" (id integer primary key autoincrement);",
"CREATE TABLE \"t2\" (id integer primary key autoincrement);"
]
for i in range(1, 10):
old_db.append("INSERT INTO \"tags\" VALUES(NULL,'test{0}',0);".format(i))
old_db.append("INSERT INTO \"t1\" VALUES(NULL);".format(i))
for i in range(1, 5):
old_db.append("INSERT INTO \"tags2\" VALUES(NULL,'test{0}',0);".format(i))
old_db.append("INSERT INTO \"t2\" VALUES(NULL);".format(i))
old_db.append("COMMIT;")

self.cu.executescript("".join(old_db))
Expand All @@ -97,12 +93,12 @@ def test_dump_autoincrement_create_new_db(self):
cx2.executescript(query)
cu2 = cx2.cursor()

self.assertEqual(cu2.execute("SELECT \"seq\""
self.assertEqual(cu2.execute("SELECT \"seq\" "
"FROM \"sqlite_sequence\""
"WHERE \"name\" == \"tags\"").fetchall()[0][0], 9)
self.assertEqual(cu2.execute("SELECT \"seq\" FROM"
"\"sqlite_sequence\""
"WHERE \"name\" == \"tags2\"").fetchall()[0][0], 4)
"WHERE \"name\" == 't1'").fetchall()[0][0], 9)
self.assertEqual(cu2.execute("SELECT \"seq\" "
"FROM \"sqlite_sequence\""
"WHERE \"name\" == 't2'").fetchall()[0][0], 4)

def test_unorderable_row(self):
# iterdump() should be able to cope with unorderable row types (issue #15545)
Expand Down