Remove tx_history in API v2

Author: Bronek

src/ripple/rpc/impl/Handler.cpp

@@ -152,7 +152,7 @@ Handler const handlerArray[]{
     {"stop", byRef(&doStop), Role::ADMIN, NO_CONDITION},
     {"transaction_entry", byRef(&doTransactionEntry), Role::USER, NO_CONDITION},
     {"tx", byRef(&doTxJson), Role::USER, NEEDS_NETWORK_CONNECTION},
-    {"tx_history", byRef(&doTxHistory), Role::USER, NO_CONDITION},
+    {"tx_history", byRef(&doTxHistory), Role::USER, NO_CONDITION, 1, 1},
     {"tx_reduce_relay", byRef(&doTxReduceRelay), Role::USER, NO_CONDITION},
     {"unl_list", byRef(&doUnlList), Role::ADMIN, NO_CONDITION},
     {"validation_create",

src/test/rpc/TransactionHistory_test.cpp

@@ -54,6 +54,22 @@ class TransactionHistory_test : public beast::unit_test::suite
         }
     }
 
+    void
+    testCommandRetired()
+    {
+        testcase("Command retired from API v2");
+        using namespace test::jtx;
+        Env env{*this, envconfig(no_admin)};
+
+        Json::Value params{Json::objectValue};
+        params[jss::api_version] = 2;
+        auto const result =
+            env.client().invoke("tx_history", params)[jss::result];
+        std::cout << result << '\n';
+        BEAST_EXPECT(result[jss::error] == "unknownCmd");
+        BEAST_EXPECT(result[jss::status] == "error");
+    }
+
     void
     testRequest()
     {
@@ -148,6 +164,7 @@ class TransactionHistory_test : public beast::unit_test::suite
     {
         testBadInput();
         testRequest();
+        testCommandRetired();
     }
 };