diff options
author | Janek Bevendorff <janek@jbev.net> | 2018-11-22 13:47:31 +0300 |
---|---|---|
committer | GitHub <noreply@github.com> | 2018-11-22 13:47:31 +0300 |
commit | d612cad09a4f1cd8ef9bf6be8bf90b75248ecfef (patch) | |
tree | 94fbd4d07f56f9646dc8c0dd8e7586776c2e3585 /tests/TestKdbx4.cpp | |
parent | 917c4cc18b8dbc04034cb0d43301b3f406644885 (diff) |
Refactor Database and Database widgets (#2491)
The Database, DatabaseWidget, and DatabaseTabWidget classes share many responsibilities in inconsistent ways resulting in impenetrable and unmaintainable code and a diverse set of bugs and architecture restrictions. This patch reworks the architecture, responsibilities of, and dependencies between these classes.
The core changes are:
* Move loading and saving logic from widgets into the Database class
* Get rid of the DatabaseManagerStruct and move all the information contained in it into the Database
* Let database objects keep track of modifications and dirty/clean state instead of handing this to external widgets
* Move GUI interactions for loading and saving from the DatabaseTabWidget into the DatabaseWidget (resolves #2494 as a side-effect)
* Heavily clean up DatabaseTabWidget and degrade it to a slightly glorified QTabWidget
* Use QSharedPointers for all Database objects
* Remove the modifiedImmediate signal and replace it with a markAsModified() method
* Implement proper tabName() method instead of reading back titles from GUI widgets (resolves #1389 and its duplicates #2146 #855)
* Fix unwanted AES-KDF downgrade if database uses Argon2 and has CustomData
* Improve code
This patch is also the first major step towards solving issues #476 and #2322.
Diffstat (limited to 'tests/TestKdbx4.cpp')
-rw-r--r-- | tests/TestKdbx4.cpp | 28 |
1 files changed, 16 insertions, 12 deletions
diff --git a/tests/TestKdbx4.cpp b/tests/TestKdbx4.cpp index 297cde284..a7fd52e0e 100644 --- a/tests/TestKdbx4.cpp +++ b/tests/TestKdbx4.cpp @@ -37,7 +37,7 @@ void TestKdbx4::initTestCaseImpl() m_kdbxSourceDb->changeKdf(fastKdf(KeePass2::uuidToKdf(KeePass2::KDF_ARGON2))); } -Database* TestKdbx4::readXml(const QString& path, bool strictMode, bool& hasError, QString& errorString) +QSharedPointer<Database> TestKdbx4::readXml(const QString& path, bool strictMode, bool& hasError, QString& errorString) { KdbxXmlReader reader(KeePass2::FILE_VERSION_4); reader.setStrictMode(strictMode); @@ -47,7 +47,7 @@ Database* TestKdbx4::readXml(const QString& path, bool strictMode, bool& hasErro return db; } -Database* TestKdbx4::readXml(QBuffer* buf, bool strictMode, bool& hasError, QString& errorString) +QSharedPointer<Database> TestKdbx4::readXml(QBuffer* buf, bool strictMode, bool& hasError, QString& errorString) { KdbxXmlReader reader(KeePass2::FILE_VERSION_4); reader.setStrictMode(strictMode); @@ -67,12 +67,12 @@ void TestKdbx4::writeXml(QBuffer* buf, Database* db, bool& hasError, QString& er void TestKdbx4::readKdbx(QIODevice* device, QSharedPointer<const CompositeKey> key, - QScopedPointer<Database>& db, + QSharedPointer<Database> db, bool& hasError, QString& errorString) { KeePass2Reader reader; - db.reset(reader.readDatabase(device, key)); + reader.readDatabase(device, key, db.data()); hasError = reader.hasError(); if (hasError) { errorString = reader.errorString(); @@ -82,12 +82,12 @@ void TestKdbx4::readKdbx(QIODevice* device, void TestKdbx4::readKdbx(const QString& path, QSharedPointer<const CompositeKey> key, - QScopedPointer<Database>& db, + QSharedPointer<Database> db, bool& hasError, QString& errorString) { KeePass2Reader reader; - db.reset(reader.readDatabase(path, key)); + reader.readDatabase(path, key, db.data()); hasError = reader.hasError(); if (hasError) { errorString = reader.errorString(); @@ -116,7 +116,8 @@ void TestKdbx4::testFormat400() auto key = QSharedPointer<CompositeKey>::create(); key->addKey(QSharedPointer<PasswordKey>::create("t")); KeePass2Reader reader; - QScopedPointer<Database> db(reader.readDatabase(filename, key)); + auto db = QSharedPointer<Database>::create(); + reader.readDatabase(filename, key, db.data()); QCOMPARE(reader.version(), KeePass2::FILE_VERSION_4); QVERIFY(db.data()); QVERIFY(!reader.hasError()); @@ -174,7 +175,8 @@ void TestKdbx4::testFormat400Upgrade() // read buffer back buffer.seek(0); KeePass2Reader reader; - QScopedPointer<Database> targetDb(reader.readDatabase(&buffer, key)); + auto targetDb = QSharedPointer<Database>::create(); + reader.readDatabase(&buffer, key, targetDb.data()); if (reader.hasError()) { QFAIL(qPrintable(QString("Error while reading database: %1").arg(reader.errorString()))); } @@ -292,14 +294,15 @@ void TestKdbx4::testUpgradeMasterKeyIntegrity() // paranoid check that we cannot decrypt the database without a key buffer.seek(0); KeePass2Reader reader; - QScopedPointer<Database> db2; - db2.reset(reader.readDatabase(&buffer, QSharedPointer<CompositeKey>::create())); + auto db2 = QSharedPointer<Database>::create(); + reader.readDatabase(&buffer, QSharedPointer<CompositeKey>::create(), db2.data()); QVERIFY(reader.hasError()); // check that we can read back the database with the original composite key, // i.e., no components have been lost on the way buffer.seek(0); - db2.reset(reader.readDatabase(&buffer, compositeKey)); + db2 = QSharedPointer<Database>::create(); + reader.readDatabase(&buffer, compositeKey, db2.data()); if (reader.hasError()) { QFAIL(qPrintable(reader.errorString())); } @@ -396,7 +399,8 @@ void TestKdbx4::testCustomData() // read buffer back buffer.seek(0); KeePass2Reader reader; - QSharedPointer<Database> newDb(reader.readDatabase(&buffer, QSharedPointer<CompositeKey>::create())); + auto newDb = QSharedPointer<Database>::create(); + reader.readDatabase(&buffer, QSharedPointer<CompositeKey>::create(), newDb.data()); // test all custom data are read back successfully from KDBX QCOMPARE(newDb->publicCustomData(), publicCustomData); |