diff --git a/tests/TestEntryModel.cpp b/tests/TestEntryModel.cpp index c1bedf415..8555de4c4 100644 --- a/tests/TestEntryModel.cpp +++ b/tests/TestEntryModel.cpp @@ -45,7 +45,7 @@ void TestEntryModel::test() EntryModel* model = new EntryModel(this); - new ModelTest(model, this); + ModelTest* modelTest = new ModelTest(model, this); model->setGroup(group1); @@ -87,6 +87,9 @@ void TestEntryModel::test() delete group1; delete group2; + + delete modelTest; + delete model; } QTEST_MAIN(TestEntryModel); diff --git a/tests/TestGroup.cpp b/tests/TestGroup.cpp index 8c71cd05f..2a54d5f00 100644 --- a/tests/TestGroup.cpp +++ b/tests/TestGroup.cpp @@ -17,6 +17,7 @@ #include "TestGroup.h" +#include #include #include @@ -32,10 +33,11 @@ void TestGroup::testParenting() Database* db = new Database(); Group* tmpRoot = new Group(); - Group* g1 = new Group(); - Group* g2 = new Group(); - Group* g3 = new Group(); - Group* g4 = new Group(); + QPointer g1 = new Group(); + QPointer g2 = new Group(); + QPointer g3 = new Group(); + QPointer g4 = new Group(); + g1->setParent(tmpRoot); g2->setParent(tmpRoot); g3->setParent(tmpRoot); @@ -74,12 +76,21 @@ void TestGroup::testParenting() g3->setIcon(Uuid::random()); g1->setIcon(2); QCOMPARE(spy.count(), 6); + + delete db; + + QVERIFY(g1.isNull()); + QVERIFY(g2.isNull()); + QVERIFY(g3.isNull()); + QVERIFY(g4.isNull()); + + delete tmpRoot; } void TestGroup::testSignals() { Database* db = new Database(); - Group* root = new Group(); + QPointer root = new Group(); root->setParent(db); Group* g1 = new Group(); @@ -100,16 +111,18 @@ void TestGroup::testSignals() QCOMPARE(spyRemoved.count(), 1); delete db; + + QVERIFY(root.isNull()); } void TestGroup::testEntries() { Group* group = new Group(); - Entry* entry1 = new Entry(); + QPointer entry1 = new Entry(); entry1->setGroup(group); - Entry* entry2 = new Entry(); + QPointer entry2 = new Entry(); entry2->setGroup(group); QCOMPARE(group->entries().size(), 2); @@ -117,6 +130,9 @@ void TestGroup::testEntries() QVERIFY(group->entries().at(1) == entry2); delete group; + + QVERIFY(entry1.isNull()); + QVERIFY(entry2.isNull()); } QTEST_MAIN(TestGroup); diff --git a/tests/TestGroupModel.cpp b/tests/TestGroupModel.cpp index 042a6164b..51c181f2b 100644 --- a/tests/TestGroupModel.cpp +++ b/tests/TestGroupModel.cpp @@ -60,7 +60,7 @@ void TestGroupModel::test() GroupModel* model = new GroupModel(db, this); - new ModelTest(model, this); + ModelTest* modelTest = new ModelTest(model, this); QModelIndex indexRoot = model->index(0, 0); QModelIndex index1 = model->index(0, 0, indexRoot); @@ -95,6 +95,9 @@ void TestGroupModel::test() delete groupRoot; delete db; + + delete modelTest; + delete model; } QTEST_MAIN(TestGroupModel); diff --git a/tests/TestKeePass2Reader.cpp b/tests/TestKeePass2Reader.cpp index a2f0f32d6..d94dce852 100644 --- a/tests/TestKeePass2Reader.cpp +++ b/tests/TestKeePass2Reader.cpp @@ -41,6 +41,9 @@ void TestKeePass2Reader::testNonAscii() QVERIFY(db); QVERIFY(!reader->error()); QCOMPARE(db->metadata()->name(), QString("NonAsciiTest")); + + delete db; + delete reader; } void TestKeePass2Reader::testCompressed() @@ -53,6 +56,9 @@ void TestKeePass2Reader::testCompressed() QVERIFY(db); QVERIFY(!reader->error()); QCOMPARE(db->metadata()->name(), QString("Compressed")); + + delete db; + delete reader; } QTEST_MAIN(TestKeePass2Reader);