Apply suggestions from code review

Co-authored-by: Ed Hennis <[email protected]>
gregtatcam · Oct 24, 2024 · 5aa36e4 · 5aa36e4
1 parent e8201f3
commit 5aa36e4
Showing 1 changed file with 4 additions and 1 deletion.
diff --git a/src/test/app/MPToken_test.cpp b/src/test/app/MPToken_test.cpp
@@ -671,8 +671,8 @@ class MPToken_test : public beast::unit_test::suite
 
             Json::Value jv;
             jv[jss::secret] = alice.name();
-            jv[jss::tx_json][jss::Fee] = to_string(env.current()->fees().base);
             jv[jss::tx_json] = pay(alice, carol, mpt);
+            jv[jss::tx_json][jss::Fee] = to_string(env.current()->fees().base);
             auto const jrr = env.rpc("json", "submit", to_string(jv));
             BEAST_EXPECT(jrr[jss::result][jss::engine_result] == "temDISABLED");
         }
@@ -1376,6 +1376,9 @@ class MPToken_test : public beast::unit_test::suite
                 jv1[jss::tx_json] = jv;
                 jrr = env.rpc("json", "submit", to_string(jv1));
                 BEAST_EXPECT(jrr[jss::result][jss::error] == "invalidParams");
+
+                jrr = env.rpc("json", "sign", to_string(jv1));
+                BEAST_EXPECT(jrr[jss::result][jss::error] == "invalidParams");
             };
             // All transactions with sfAmount, which don't support MPT
             // and transactions with amount fields, which can't be MPT