diff options
author | Florian Bruhin <me@the-compiler.org> | 2021-01-14 11:53:09 +0100 |
---|---|---|
committer | Florian Bruhin <me@the-compiler.org> | 2021-01-14 11:53:09 +0100 |
commit | 84ad7c990b27264f9ed03ef81a47e9d4acdffaf0 (patch) | |
tree | 415f48c11d89fc5b548909770df05064cdf15f5f | |
parent | 482cf9ae047b8a08ee7ad6a1ff64feff39fe6e3c (diff) | |
download | qutebrowser-84ad7c990b27264f9ed03ef81a47e9d4acdffaf0.tar.gz qutebrowser-84ad7c990b27264f9ed03ef81a47e9d4acdffaf0.zip |
sql: Improve/fix rows_affected() tests
-rw-r--r-- | tests/unit/misc/test_sql.py | 20 |
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') |