changed env unit test based on ED's commit

PeterChen13579 · Jun 22, 2023 · 690ce23 · 690ce23
1 parent e493534
commit 690ce23
Show file tree

Hide file tree

Showing 2 changed files with 12 additions and 7 deletions.
diff --git a/src/test/rpc/AccountInfo_test.cpp b/src/test/rpc/AccountInfo_test.cpp
@@ -220,6 +220,10 @@ class AccountInfo_test : public beast::unit_test::suite
             "\"api_version\": 2, \"account\": \"" + alice.human() + "\", " +
             "\"signer_lists\": true }";
 
+        auto const withSignersAsString = std::string("{ ") +
+            "\"api_version\": 2, \"account\": \"" + alice.human() + "\", " +
+            "\"signer_lists\": asdfgh }";
+
         // Alice has no SignerList yet.
         {
             // account_info without the "signer_lists" argument.
@@ -237,6 +241,13 @@ class AccountInfo_test : public beast::unit_test::suite
             BEAST_EXPECT(signerLists.isArray());
             BEAST_EXPECT(signerLists.size() == 0);
         }
+        {
+            // expects error if string used for signer_lists
+            auto const info =
+                env.rpc("json", "account_info", withSignersAsString);
+            BEAST_EXPECT(info[jss::status] == "error");
+            BEAST_EXPECT(info[jss::error] == "invalidParams");
+        }
 
         // Give alice a SignerList.
         Account const bogie{"bogie"};

diff --git a/src/test/rpc/GatewayBalances_test.cpp b/src/test/rpc/GatewayBalances_test.cpp
@@ -153,13 +153,7 @@ class GatewayBalances_test : public beast::unit_test::suite
     {
         using namespace std::chrono_literals;
         using namespace jtx;
-        Env env{
-            *this,
-            envconfig([](std::unique_ptr<Config> c) {
-                c->loadFromString("\n[beta_rpc_api]\n1\n");
-                return c;
-            }),
-            features};
+        Env env{*this, features};
         env.close();
 
         // Gateway account and assets