diff --git a/src/test/app/AMM_test.cpp b/src/test/app/AMM_test.cpp index 5de08a113..ad1e761d9 100644 --- a/src/test/app/AMM_test.cpp +++ b/src/test/app/AMM_test.cpp @@ -3302,6 +3302,13 @@ private: ter(tecNO_PERMISSION)); }); + // Can't pay into AMM with Remit. + testAMM([&](AMM& ammAlice, Env& env) { + env(remit::remit(carol, ammAlice.ammAccount()), + remit::amts({XRP(1)}), + ter(tecNO_PERMISSION)); + }); + // Pay amounts close to one side of the pool testAMM( [&](AMM& ammAlice, Env& env) { diff --git a/src/test/jtx/impl/remit.cpp b/src/test/jtx/impl/remit.cpp index bcee071ce..f18e084a8 100644 --- a/src/test/jtx/impl/remit.cpp +++ b/src/test/jtx/impl/remit.cpp @@ -27,15 +27,15 @@ namespace remit { Json::Value remit( - jtx::Account const& account, - jtx::Account const& dest, + AccountID const& account, + AccountID const& dest, std::optional const& dstTag) { using namespace jtx; Json::Value jv; jv[jss::TransactionType] = jss::Remit; - jv[jss::Account] = account.human(); - jv[jss::Destination] = dest.human(); + jv[jss::Account] = to_string(account); + jv[jss::Destination] = to_string(dest); if (dstTag) jv[sfDestinationTag.jsonName] = *dstTag; return jv; diff --git a/src/test/jtx/remit.h b/src/test/jtx/remit.h index 4db4b24c2..c6196d8b7 100644 --- a/src/test/jtx/remit.h +++ b/src/test/jtx/remit.h @@ -32,8 +32,8 @@ namespace remit { Json::Value remit( - jtx::Account const& account, - jtx::Account const& dest, + AccountID const& account, + AccountID const& dest, std::optional const& dstTag = std::nullopt); /** Sets the optional Amount on a JTx. */ @@ -125,4 +125,4 @@ public: } // namespace test } // namespace ripple -#endif // RIPPLE_TEST_JTX_REMIT_H_INCLUDED \ No newline at end of file +#endif // RIPPLE_TEST_JTX_REMIT_H_INCLUDED