X-Git-Url: https://git.openstreetmap.org./nominatim.git/blobdiff_plain/0ad1b28497b49bf8baa5cfa2ea4438c4824617a1..c06a1d007a4e67ab5caf5bf63d351f83e4e54ab6:/test/python/test_db_utils.py diff --git a/test/python/test_db_utils.py b/test/python/test_db_utils.py index e756f2c4..4a603888 100644 --- a/test/python/test_db_utils.py +++ b/test/python/test_db_utils.py @@ -5,26 +5,49 @@ import psycopg2 import pytest import nominatim.db.utils as db_utils +from nominatim.errors import UsageError -def test_execute_file_success(temp_db_conn, tmp_path): +def test_execute_file_success(dsn, temp_db_cursor, tmp_path): tmpfile = tmp_path / 'test.sql' tmpfile.write_text('CREATE TABLE test (id INT);\nINSERT INTO test VALUES(56);') - db_utils.execute_file(temp_db_conn, tmpfile) + db_utils.execute_file(dsn, tmpfile) - with temp_db_conn.cursor() as cur: - cur.execute('SELECT * FROM test') + assert temp_db_cursor.row_set('SELECT * FROM test') == {(56, )} - assert cur.rowcount == 1 - assert cur.fetchone()[0] == 56 - -def test_execute_file_bad_file(temp_db_conn, tmp_path): +def test_execute_file_bad_file(dsn, tmp_path): with pytest.raises(FileNotFoundError): - db_utils.execute_file(temp_db_conn, tmp_path / 'test2.sql') + db_utils.execute_file(dsn, tmp_path / 'test2.sql') + -def test_execute_file_bad_sql(temp_db_conn, tmp_path): +def test_execute_file_bad_sql(dsn, tmp_path): tmpfile = tmp_path / 'test.sql' tmpfile.write_text('CREATE STABLE test (id INT)') - with pytest.raises(psycopg2.ProgrammingError): - db_utils.execute_file(temp_db_conn, tmpfile) + with pytest.raises(UsageError): + db_utils.execute_file(dsn, tmpfile) + + +def test_execute_file_bad_sql_ignore_errors(dsn, tmp_path): + tmpfile = tmp_path / 'test.sql' + tmpfile.write_text('CREATE STABLE test (id INT)') + + db_utils.execute_file(dsn, tmpfile, ignore_errors=True) + + +def test_execute_file_with_pre_code(dsn, tmp_path, temp_db_cursor): + tmpfile = tmp_path / 'test.sql' + tmpfile.write_text('INSERT INTO test VALUES(4)') + + db_utils.execute_file(dsn, tmpfile, pre_code='CREATE TABLE test (id INT)') + + assert temp_db_cursor.row_set('SELECT * FROM test') == {(4, )} + + +def test_execute_file_with_post_code(dsn, tmp_path, temp_db_cursor): + tmpfile = tmp_path / 'test.sql' + tmpfile.write_text('CREATE TABLE test (id INT)') + + db_utils.execute_file(dsn, tmpfile, post_code='INSERT INTO test VALUES(23)') + + assert temp_db_cursor.row_set('SELECT * FROM test') == {(23, )}