Refactor: extractSignatureBytes to use stringToByteVector and add tra… (#1230)

Signed-off-by: Mascud Muse Hassan <mascuudmusse44@gmail.com>

Author: ak004

src/sdk/main/src/Transaction.cc

@@ -80,27 +80,21 @@ namespace Hiero
 namespace
 {
 /**
- * Helper function - Extract the raw signature bytes from a protobuf SignaturePair.
+ * Extract the raw signature bytes from a protobuf SignaturePair.
  */
 std::vector<std::byte> extractSignatureBytes(const proto::SignaturePair& pair)
 {
-  const std::string* sigStr = nullptr;
   if (pair.has_ed25519())
   {
-    sigStr = &pair.ed25519();
+    return internal::Utilities::stringToByteVector(pair.ed25519());
   }
-  else if (pair.has_ecdsa_secp256k1())
-  {
-    sigStr = &pair.ecdsa_secp256k1();
-  }
-  else
+
+  if (pair.has_ecdsa_secp256k1())
   {
-    throw IllegalStateException("Unknown signature type");
+    return internal::Utilities::stringToByteVector(pair.ecdsa_secp256k1());
   }
 
-  std::vector<std::byte> sigBytes(sigStr->size());
-  std::transform(sigStr->begin(), sigStr->end(), sigBytes.begin(), [](char c) { return static_cast<std::byte>(c); });
-  return sigBytes;
+  throw IllegalStateException("Unknown signature type");
 }
 } // anonymous namespace
 

src/sdk/tests/unit/TransactionUnitTests.cc

@@ -3173,4 +3173,4 @@ TEST_F(TransactionUnitTests, RemoveSignatureFromMultiNodeTransaction)
   // Then
   // Verify that all internal signatory tracking and protobuf signature maps are completely clear
   EXPECT_EQ(finalRemoval.size(), 0);
-}
+}