From 1e359f312e85f7aa9153a8e60cc049c2434d5133 Mon Sep 17 00:00:00 2001
From: abel <abarmoa@gmail.com>
Date: Thu, 4 Jan 2024 13:01:07 -0300
Subject: [PATCH] (fix) Removed support for MsgUpdateParams as requested in the
 PR review

---
 ...MsgChangeAdmin.py => 75_MsgChangeAdmin.py} |  0
 examples/chain_client/75_MsgUpdateParams.py   | 37 -------------------
 pyinjective/composer.py                       | 20 +---------
 tests/test_composer.py                        | 15 --------
 4 files changed, 1 insertion(+), 71 deletions(-)
 rename examples/chain_client/{76_MsgChangeAdmin.py => 75_MsgChangeAdmin.py} (100%)
 delete mode 100644 examples/chain_client/75_MsgUpdateParams.py

diff --git a/examples/chain_client/76_MsgChangeAdmin.py b/examples/chain_client/75_MsgChangeAdmin.py
similarity index 100%
rename from examples/chain_client/76_MsgChangeAdmin.py
rename to examples/chain_client/75_MsgChangeAdmin.py
diff --git a/examples/chain_client/75_MsgUpdateParams.py b/examples/chain_client/75_MsgUpdateParams.py
deleted file mode 100644
index 0d53231b..00000000
--- a/examples/chain_client/75_MsgUpdateParams.py
+++ /dev/null
@@ -1,37 +0,0 @@
-import asyncio
-
-from pyinjective.composer import Composer as ProtoMsgComposer
-from pyinjective.core.broadcaster import MsgBroadcasterWithPk
-from pyinjective.core.network import Network
-from pyinjective.wallet import PrivateKey
-
-
-async def main() -> None:
-    # select network: local, testnet, mainnet
-    network = Network.testnet()
-    composer = ProtoMsgComposer(network=network.string())
-    private_key_in_hexa = "f9db9bf330e23cb7839039e944adef6e9df447b90b503d5b4464c90bea9022f3"
-
-    message_broadcaster = MsgBroadcasterWithPk.new_using_simulation(
-        network=network,
-        private_key=private_key_in_hexa,
-    )
-
-    priv_key = PrivateKey.from_hex(private_key_in_hexa)
-    pub_key = priv_key.to_public_key()
-    address = pub_key.to_address()
-
-    message = composer.msg_update_params(
-        authority=address.to_acc_bech32(),
-        denom="factory/inj1hkhdaj2a2clmq5jq6mspsggqs32vynpk228q3r/inj_test",
-        amount=1000,
-    )
-
-    # broadcast the transaction
-    result = await message_broadcaster.broadcast([message])
-    print("---Transaction Response---")
-    print(result)
-
-
-if __name__ == "__main__":
-    asyncio.get_event_loop().run_until_complete(main())
diff --git a/pyinjective/composer.py b/pyinjective/composer.py
index e7d2deff..4c35b07e 100644
--- a/pyinjective/composer.py
+++ b/pyinjective/composer.py
@@ -28,10 +28,7 @@
 from pyinjective.proto.injective.oracle.v1beta1 import tx_pb2 as injective_oracle_tx_pb
 from pyinjective.proto.injective.peggy.v1 import msgs_pb2 as injective_peggy_tx_pb
 from pyinjective.proto.injective.stream.v1beta1 import query_pb2 as chain_stream_query
-from pyinjective.proto.injective.tokenfactory.v1beta1 import (
-    params_pb2 as token_factory_params_pb,
-    tx_pb2 as token_factory_tx_pb,
-)
+from pyinjective.proto.injective.tokenfactory.v1beta1 import tx_pb2 as token_factory_tx_pb
 
 REQUEST_TO_RESPONSE_TYPE_MAP = {
     "MsgCreateSpotLimitOrder": injective_exchange_tx_pb.MsgCreateSpotLimitOrderResponse,
@@ -1023,21 +1020,6 @@ def msg_set_denom_metadata(
         )
         return token_factory_tx_pb.MsgSetDenomMetadata(sender=sender, metadata=metadata)
 
-    def msg_update_params(
-        self,
-        authority: str,
-        denom: str,
-        amount: int,
-    ) -> token_factory_tx_pb.MsgUpdateParams:
-        coin = self.Coin(amount=amount, denom=denom)
-        params = token_factory_params_pb.Params(
-            denom_creation_fee=[coin],
-        )
-        return token_factory_tx_pb.MsgUpdateParams(
-            authority=authority,
-            params=params,
-        )
-
     def msg_change_admin(
         self,
         sender: str,
diff --git a/tests/test_composer.py b/tests/test_composer.py
index 392d0e7a..05810389 100644
--- a/tests/test_composer.py
+++ b/tests/test_composer.py
@@ -347,21 +347,6 @@ def test_msg_set_denom_metadata(self, basic_composer: Composer):
         assert message.metadata.uri == uri
         assert message.metadata.uri_hash == uri_hash
 
-    def test_msg_update_params(self, basic_composer: Composer):
-        authority = "inj1apmvarl2xyv6kecx2ukkeymddw3we4zkygjyc0"
-        amount = 1000
-        denom = "factory/inj1hkhdaj2a2clmq5jq6mspsggqs32vynpk228q3r/inj_test"
-
-        message = basic_composer.msg_update_params(
-            authority=authority,
-            denom=denom,
-            amount=amount,
-        )
-
-        assert message.authority == authority
-        assert message.params.denom_creation_fee[0].amount == str(amount)
-        assert message.params.denom_creation_fee[0].denom == denom
-
     def test_msg_change_admin(self, basic_composer):
         sender = "inj1apmvarl2xyv6kecx2ukkeymddw3we4zkygjyc0"
         denom = "factory/inj1hkhdaj2a2clmq5jq6mspsggqs32vynpk228q3r/inj_test"