diff options
| author | Alexey Rusakov <Kitsune-Ral@users.sf.net> | 2022-09-25 21:37:49 +0200 |
|---|---|---|
| committer | Alexey Rusakov <Kitsune-Ral@users.sf.net> | 2022-09-26 10:46:34 +0200 |
| commit | 72e14cb1bdff68dfe0fb61fff0defd6c50dff43c (patch) | |
| tree | 30e197665edd678cafdd33947a7386bd248635e1 /autotests | |
| parent | 388c301222586583a0144f7229a564cf07b597d4 (diff) | |
| download | libquotient-72e14cb1bdff68dfe0fb61fff0defd6c50dff43c.tar.gz libquotient-72e14cb1bdff68dfe0fb61fff0defd6c50dff43c.zip | |
QOlmUtility::ed25519Verify: just return bool
It's too easy to incorrectly test the previous return type.
(cherry picked from commit 5904a61c59f0eef00aef07ef998658fd791ff139)
Diffstat (limited to 'autotests')
| -rw-r--r-- | autotests/testolmaccount.cpp | 3 | ||||
| -rw-r--r-- | autotests/testolmutility.cpp | 6 |
2 files changed, 3 insertions, 6 deletions
diff --git a/autotests/testolmaccount.cpp b/autotests/testolmaccount.cpp index 56532698..53a0c955 100644 --- a/autotests/testolmaccount.cpp +++ b/autotests/testolmaccount.cpp @@ -60,8 +60,7 @@ void TestOlmAccount::signatureValid() QOlmUtility utility; const auto identityKeys = olmAccount.identityKeys(); const auto ed25519Key = identityKeys.ed25519; - const auto verify = utility.ed25519Verify(ed25519Key, message, signature); - QVERIFY(verify.value_or(false)); + QVERIFY(utility.ed25519Verify(ed25519Key, message, signature)); } void TestOlmAccount::oneTimeKeysValid() diff --git a/autotests/testolmutility.cpp b/autotests/testolmutility.cpp index 64ceb3e7..4de5afdf 100644 --- a/autotests/testolmutility.cpp +++ b/autotests/testolmutility.cpp @@ -78,10 +78,8 @@ void TestOlmUtility::verifySignedOneTimeKey() delete[](reinterpret_cast<uint8_t *>(utility)); QOlmUtility utility2; - auto res2 = - utility2 - .ed25519Verify(aliceOlm.identityKeys().ed25519, msg, signatureBuf1) - .value_or(false); + auto res2 = utility2.ed25519Verify(aliceOlm.identityKeys().ed25519, msg, + signatureBuf1); //QCOMPARE(std::string(olm_utility_last_error(utility)), "SUCCESS"); QVERIFY(res2); |