summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorFlorian Bruhin <me@the-compiler.org>2021-01-14 11:53:09 +0100
committerFlorian Bruhin <me@the-compiler.org>2021-01-14 11:53:09 +0100
commit84ad7c990b27264f9ed03ef81a47e9d4acdffaf0 (patch)
tree415f48c11d89fc5b548909770df05064cdf15f5f
parent482cf9ae047b8a08ee7ad6a1ff64feff39fe6e3c (diff)
downloadqutebrowser-84ad7c990b27264f9ed03ef81a47e9d4acdffaf0.tar.gz
qutebrowser-84ad7c990b27264f9ed03ef81a47e9d4acdffaf0.zip
sql: Improve/fix rows_affected() tests
-rw-r--r--tests/unit/misc/test_sql.py20
1 files changed, 17 insertions, 3 deletions
diff --git a/tests/unit/misc/test_sql.py b/tests/unit/misc/test_sql.py
index ab34727a9..d7c6fd4d5 100644
--- a/tests/unit/misc/test_sql.py
+++ b/tests/unit/misc/test_sql.py
@@ -376,10 +376,24 @@ class TestSqlQuery:
match='No result for single-result query'):
q.value()
- def test_num_rows_affected(self):
- q = sql.Query('SELECT 0')
+ def test_num_rows_affected_not_active(self):
+ with pytest.raises(AssertionError):
+ q = sql.Query('SELECT 0')
+ q.rows_affected()
+
+ def test_num_rows_affected_select(self):
+ with pytest.raises(AssertionError):
+ q = sql.Query('SELECT 0')
+ q.run()
+ q.rows_affected()
+
+ @pytest.mark.parametrize('condition', [0, 1])
+ def test_num_rows_affected(self, condition):
+ table = sql.SqlTable('Foo', ['name'])
+ table.insert({'name': 'helloworld'})
+ q = sql.Query(f'DELETE FROM Foo WHERE {condition}')
q.run()
- assert q.rows_affected() == 0
+ assert q.rows_affected() == condition
def test_bound_values(self):
q = sql.Query('SELECT :answer')