tst_QConcatenateTablesProxyModel: extend shouldMergeRoleNames() with removal
We should also test removal, not just addition. Amends 5ffb9d7ae6d60fb370b79f8222dab7d7e628fa4f. Pick-to: 6.9 Change-Id: I0842a6be6230362e2095f6c8b2b1ced230826c5e Reviewed-by: Volker Hilsheimer <volker.hilsheimer@qt.io>
This commit is contained in:
parent
26ba78900d
commit
e15ef96bcb
@ -980,6 +980,13 @@ void tst_QConcatenateTablesProxyModel::shouldMergeRoleNames()
|
||||
QCOMPARE(roleNames[Qt::DisplayRole], "display");
|
||||
QCOMPARE(roleNames[Qt::UserRole], "user");
|
||||
QCOMPARE(roleNames[Qt::UserRole + 1], "user+1");
|
||||
|
||||
// When removing a source model
|
||||
pm.removeSourceModel(&mod2);
|
||||
|
||||
// Then the role names should be updated
|
||||
const auto roleNamesAfterMod2Removal = pm.roleNames();
|
||||
QVERIFY(!roleNamesAfterMod2Removal.contains(Qt::UserRole + 1));
|
||||
}
|
||||
|
||||
QTEST_GUILESS_MAIN(tst_QConcatenateTablesProxyModel)
|
||||
|
Loading…
x
Reference in New Issue
Block a user