diff --git a/tests/common/util/TestObject.cpp b/tests/common/util/TestObject.cpp
index fa2b3adb..55e9f844 100644
--- a/tests/common/util/TestObject.cpp
+++ b/tests/common/util/TestObject.cpp
@@ -1027,7 +1027,7 @@ createAcceptNftSellerOfferTxWithMetadata(
     // offer owner is not the nft's new owner for seller offer, we need to create other nodes for processing new owner
     finalFields.setAccountID(ripple::sfOwner, account.value());
 
-    node.emplace_back(std::move(finalFields));
+    node.emplace_back(finalFields);
     node.setFieldH256(ripple::sfLedgerIndex, ripple::uint256{offerId});
     metaArray.push_back(node);
 
diff --git a/tests/unit/etl/NFTHelpersTests.cpp b/tests/unit/etl/NFTHelpersTests.cpp
index e048b8df..01e24933 100644
--- a/tests/unit/etl/NFTHelpersTests.cpp
+++ b/tests/unit/etl/NFTHelpersTests.cpp
@@ -479,7 +479,7 @@ TEST_F(NFTHelpersTest, NFTDataFromLedgerObject)
 
     auto const nftDatas = etl::getNFTDataFromObj(
         kSEQ,
-        std::string(static_cast<char const*>(static_cast<void const*>(account.data())), ripple::AccountID::size()),
+        std::string(reinterpret_cast<char const*>(account.data()), ripple::AccountID::size()),
         std::string(static_cast<char const*>(serializerNftPage.getDataPtr()), serializerNftPage.getDataLength())
     );