fix clang-tidy issues (#1871)

Author: cindyyan317

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);
 

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())
     );