1berry/qtbase
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

QSqlQueryModel test: don't copy a QSqlQuery

Browse Source 
Move the query into the model, and use a local scope to make sure that
we don't use the moved-from query later.

Pick-to: 6.9
Change-Id: I9d216e770733af8b0771280276dba0775209a802
Reviewed-by: Christian Ehrlicher <ch.ehrlicher@gmx.de>
Reviewed-by: Dheerendra Purohit <dheerendra@pthinks.com>
This commit is contained in:
Volker Hilsheimer 2024-12-14 20:40:48 +01:00
parent d884abaf8b
commit af760da541
1 changed files with 7 additions and 5 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/auto/sql/models/qsqlquerymodel/tst_qsqlquerymodel.cpp
View File

@ -694,11 +694,13 @@ void tst_QSqlQueryModel::refreshQueryWithBoundValues()
QSqlDatabase db = QSqlDatabase::database(dbName);
QSqlQueryModel model;
QSqlQuery query(db);
query.prepare("SELECT name FROM " + qTableName("test", __FILE__, db) + " WHERE id = :id");
query.bindValue(":id", 1);
query.exec();
model.setQuery(query);
{
QSqlQuery query(db);
query.prepare("SELECT name FROM " + qTableName("test", __FILE__, db) + " WHERE id = :id");
query.bindValue(":id", 1);
query.exec();
model.setQuery(std::move(query));
}
QCOMPARE(model.rowCount(), 1);
QCOMPARE(model.data(model.index(0, 0)).toString(), QString("harry"));