fix: should check key status immediately after get it

3 files changed, 29 insertions, 0 deletions

diff --git a/src/test/core/GpgCoreTestBasicOpera.cpp b/src/test/core/GpgCoreTestBasicOpera.cpp
index 1c35a414..2e675766 100644
--- a/src/test/core/GpgCoreTestBasicOpera.cpp
+++ b/src/test/core/GpgCoreTestBasicOpera.cpp
@@ -42,6 +42,8 @@ namespace GpgFrontend::Test {
 TEST_F(GpgCoreTest, CoreEncryptDecrTest) {
   auto encrypt_key = GpgKeyGetter::GetInstance().GetPubkey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(encrypt_key.IsGood());
+
   auto buffer = GFBuffer(QString("Hello GpgFrontend!"));
 
   auto [err, data_object] =
@@ -152,6 +154,8 @@ TEST_F(GpgCoreTest, CoreEncryptDecrTest_KeyNotFound_ResultAnalyse) {
 TEST_F(GpgCoreTest, CoreSignVerifyNormalTest) {
   auto sign_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(sign_key.IsGood());
+
   auto sign_text = GFBuffer(QString("Hello GpgFrontend!"));
 
   auto [err, data_object] = GpgBasicOperator::GetInstance().SignSync(
@@ -177,6 +181,8 @@ TEST_F(GpgCoreTest, CoreSignVerifyNormalTest) {
 TEST_F(GpgCoreTest, CoreSignVerifyDetachTest) {
   auto sign_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(sign_key.IsGood());
+
   auto sign_text = GFBuffer(QString("Hello GpgFrontend!"));
 
   auto [err, data_object] = GpgBasicOperator::GetInstance().SignSync(
@@ -202,6 +208,8 @@ TEST_F(GpgCoreTest, CoreSignVerifyDetachTest) {
 TEST_F(GpgCoreTest, CoreSignVerifyClearTest) {
   auto sign_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(sign_key.IsGood());
+
   auto sign_text = GFBuffer(QString("Hello GpgFrontend!"));
 
   auto [err, data_object] = GpgBasicOperator::GetInstance().SignSync(
@@ -226,8 +234,10 @@ TEST_F(GpgCoreTest, CoreSignVerifyClearTest) {
 TEST_F(GpgCoreTest, CoreEncryptSignDecrVerifyTest) {
   auto encrypt_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(encrypt_key.IsGood());
   auto sign_key = GpgKeyGetter::GetInstance().GetKey(
       "8933EB283A18995F45D61DAC021D89771B680FFB");
+  ASSERT_TRUE(sign_key.IsGood());
   auto encrypt_text = GFBuffer(QString("Hello GpgFrontend!"));
 
   ASSERT_TRUE(sign_key.IsPrivateKey());
diff --git a/src/test/core/GpgCoreTestFileBasicOpera.cpp b/src/test/core/GpgCoreTestFileBasicOpera.cpp
index ce7359b2..9a0c6a34 100644
--- a/src/test/core/GpgCoreTestFileBasicOpera.cpp
+++ b/src/test/core/GpgCoreTestFileBasicOpera.cpp
@@ -42,6 +42,7 @@ namespace GpgFrontend::Test {
 TEST_F(GpgCoreTest, CoreFileEncryptDecrTest) {
   auto encrypt_key = GpgKeyGetter::GetInstance().GetPubkey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(encrypt_key.IsGood());
 
   auto buffer = GFBuffer(QString("Hello GpgFrontend!"));
   auto input_file = CreateTempFileAndWriteData(buffer);
@@ -73,6 +74,7 @@ TEST_F(GpgCoreTest, CoreFileEncryptDecrTest) {
 TEST_F(GpgCoreTest, CoreFileEncryptDecrBinaryTest) {
   auto encrypt_key = GpgKeyGetter::GetInstance().GetPubkey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(encrypt_key.IsGood());
 
   auto buffer = GFBuffer(QString("Hello GpgFrontend!"));
   auto input_file = CreateTempFileAndWriteData(buffer);
@@ -165,6 +167,8 @@ TEST_F(GpgCoreTest, CoreFileEncryptSymmetricDecrBinaryTest) {
 TEST_F(GpgCoreTest, CoreFileSignVerifyNormalTest) {
   auto sign_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(sign_key.IsGood());
+
   auto input_file = CreateTempFileAndWriteData("Hello GpgFrontend!");
   auto output_file = GetTempFilePath();
 
@@ -190,6 +194,8 @@ TEST_F(GpgCoreTest, CoreFileSignVerifyNormalTest) {
 TEST_F(GpgCoreTest, CoreFileSignVerifyNormalBinaryTest) {
   auto sign_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(sign_key.IsGood());
+
   auto input_file = CreateTempFileAndWriteData("Hello GpgFrontend!");
   auto output_file = GetTempFilePath();
 
@@ -215,8 +221,12 @@ TEST_F(GpgCoreTest, CoreFileSignVerifyNormalBinaryTest) {
 TEST_F(GpgCoreTest, CoreFileEncryptSignDecrVerifyTest) {
   auto encrypt_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(encrypt_key.IsGood());
+
   auto sign_key = GpgKeyGetter::GetInstance().GetKey(
       "8933EB283A18995F45D61DAC021D89771B680FFB");
+  ASSERT_TRUE(sign_key.IsGood());
+
   auto buffer = GFBuffer(QString("Hello GpgFrontend!"));
   auto input_file = CreateTempFileAndWriteData(buffer);
   auto output_file = GetTempFilePath();
@@ -259,8 +269,11 @@ TEST_F(GpgCoreTest, CoreFileEncryptSignDecrVerifyTest) {
 TEST_F(GpgCoreTest, CoreFileEncryptSignDecrVerifyBinaryTest) {
   auto encrypt_key = GpgKeyGetter::GetInstance().GetPubkey(
       "467F14220CE8DCF780CF4BAD8465C55B25C9B7D1");
+  ASSERT_TRUE(encrypt_key.IsGood());
   auto sign_key = GpgKeyGetter::GetInstance().GetKey(
       "8933EB283A18995F45D61DAC021D89771B680FFB");
+  ASSERT_TRUE(sign_key.IsGood());
+
   auto buffer = GFBuffer(QString("Hello GpgFrontend!"));
   auto input_file = CreateTempFileAndWriteData(buffer);
   auto output_file = GetTempFilePath();
diff --git a/src/test/core/GpgCoreTestSubkeygen.cpp b/src/test/core/GpgCoreTestSubkeygen.cpp
index 34dcb85e..7c826c24 100644
--- a/src/test/core/GpgCoreTestSubkeygen.cpp
+++ b/src/test/core/GpgCoreTestSubkeygen.cpp
@@ -40,6 +40,7 @@ namespace GpgFrontend::Test {
 TEST_F(GpgCoreTest, GenerateSubkeyRSA2048Test) {
   auto main_key = GpgKeyGetter::GetInstance().GetKey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(main_key.IsGood());
 
   auto subkeygen_info = SecureCreateSharedObject<GenKeyInfo>(true);
   subkeygen_info->SetAlgo(
@@ -83,6 +84,7 @@ TEST_F(GpgCoreTest, GenerateSubkeyRSA2048Test) {
 TEST_F(GpgCoreTest, GenerateSubkeyDSA2048Test) {
   auto main_key = GpgKeyGetter::GetInstance().GetKey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(main_key.IsGood());
 
   auto subkeygen_info = SecureCreateSharedObject<GenKeyInfo>(true);
   subkeygen_info->SetAlgo(
@@ -126,6 +128,7 @@ TEST_F(GpgCoreTest, GenerateSubkeyDSA2048Test) {
 TEST_F(GpgCoreTest, GenerateSubkeyED25519Test) {
   auto main_key = GpgKeyGetter::GetInstance().GetKey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(main_key.IsGood());
 
   auto subkeygen_info = SecureCreateSharedObject<GenKeyInfo>(true);
   subkeygen_info->SetAlgo(
@@ -169,6 +172,7 @@ TEST_F(GpgCoreTest, GenerateSubkeyED25519Test) {
 TEST_F(GpgCoreTest, GenerateSubkeyCV25519Test) {
   auto main_key = GpgKeyGetter::GetInstance().GetKey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(main_key.IsGood());
 
   auto subkeygen_info = SecureCreateSharedObject<GenKeyInfo>(true);
   subkeygen_info->SetAlgo(
@@ -212,6 +216,7 @@ TEST_F(GpgCoreTest, GenerateSubkeyCV25519Test) {
 TEST_F(GpgCoreTest, GenerateSubkeyNISTP256Test) {
   auto main_key = GpgKeyGetter::GetInstance().GetKey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(main_key.IsGood());
 
   auto subkeygen_info = SecureCreateSharedObject<GenKeyInfo>(true);
   subkeygen_info->SetAlgo(
@@ -255,6 +260,7 @@ TEST_F(GpgCoreTest, GenerateSubkeyNISTP256Test) {
 TEST_F(GpgCoreTest, GenerateSubkeyBRAINPOOLP256R1Test) {
   auto main_key = GpgKeyGetter::GetInstance().GetKey(
       "E87C6A2D8D95C818DE93B3AE6A2764F8298DEB29");
+  ASSERT_TRUE(main_key.IsGood());
 
   auto subkeygen_info = SecureCreateSharedObject<GenKeyInfo>(true);
   subkeygen_info->SetAlgo(