From bce40b3a584a0fabcb53fcfe4c260dc373bef913 Mon Sep 17 00:00:00 2001
From: ALPAC-4 <81249838+ALPAC-4@users.noreply.github.com>
Date: Mon, 23 Dec 2024 13:17:34 +0900
Subject: [PATCH] fix: deprecate `finalize_arb` and make new version of it
 (#172)

* fix: deprecate `finalize_arb` and make new version of it

* fix: fix balance condition

* chore: make precompile

* fix: remove `generate_finalize_token_withdrawal_msg` function

* chore: make precompile
---
 precompile/binaries/stdlib/minitswap.mv       | Bin 21531 -> 21291 bytes
 .../initia_stdlib/sources/minitswap.move      | 126 +++++-------------
 2 files changed, 31 insertions(+), 95 deletions(-)

diff --git a/precompile/binaries/stdlib/minitswap.mv b/precompile/binaries/stdlib/minitswap.mv
index a835ec55040fab7ac7986a3280db512901fdea4a..3fc4a17ddfcc8fea4de96855f2adf8c7ceb62c64 100644
GIT binary patch
delta 3964
zcmZu!X>e4>5uToR?wi@SJ5MXEgtTZEt*+Gp61I5~7CJ!U5FpOjU<i?f0D%w~foy|q
z31P%xa}F3^G2jCq*w|RU@D=-4exxepB$fP0r4lDUlBzgWcBK+uNt~Xy0y|Zq#7w`L
zp09iQn%>8shs&3s`C`#unMH`hfy|+E{tntzdN`zCV{e1;Ie#*0ok;vLWq&d!Y3^?x
zqtOGcr)}r#!iy$<v+J`};Y-h)YOeME4dh!Oqm-PYL{c(HUcEZ9LK|WKfJA}-`6p}y
z0`LW_vkA~Ip}CK01VUfICIB1A*U*pK#y7B4G5-q&DhP=GK#9Or?{6SH0y+C>i3b$O
z7~L(9=>Sa(<MaT%`yK-BK`PYAaT-_CA-XS4KtD`JARp%=v|UX{v(tk#p}roX9cuaw
z1bfG5J|5JMXBSTdoa7|E4_RWTsX&3bXXyQ?202S%6tyA`Qy)*n&Sf_rp)1wqqx5Da
z_%YgEgI|x+kp}?3jVGvAjNd<|!6$i=7LTjzo~Om?&;?q27(cCxwDqV$eu_Q>#}x52
z5+@Y#3_S&>74a-R3u?YZ6KYB-tHE>BPAVflPn%F`<OO;T9tpT!q$7_h`Q$QvB3O8d
z>N?7OnVtun^((XkuaNf&wNZ`uSMY4Wbd{DW?APe?@PZ;H=t}{}>r_c2Z_t+&oAD;S
z60pBThp(#jx9KRnrig3wFia@o9XehGuPf?ZD&Y-9yho3~n~Hd!z7?GQfL;scpU{43
zm|fdePg!001Df1iIi<3)etZ33{f@QX+C*skx}EbAp~29OwRT=2Btl1^x;3vgbQJ3G
z?+DZvH{>?v&dzNpXe^jrFa~Jn+mcU0@**T(gqa1&SD-q7Mt-$D!>;ybc*%EQbwTp|
ziI2>2O^!L`32JhU8cYLQw@sU5Db2K_8aMgGTTYrz{GNa8GPY;1d$_xQpu1;4ep@)Z
zBeJP~uzR3?Z{LRQT|FCy>#d&df$qWHzS!oz!M<JHyZbiu_w;Vq-ZwPVy}3{8=^e6n
z@7U0D&+fh<v!{Eg@0OeNtwTEoC$<*dO<fbWhBj_9dv^?N-!U}t$Envq{vr9k_B+d)
zxR5#yu^f{9$J*z$FI&+*w{=zfyos8U)|^gshIwNb^JYc)O3*1uZ|Sy3hH%XxLU*)i
zzF3vg*E(7x?tp0W%ee09ek7CThGRs+!CJYTKdUxY;aH9t=gbX>PNvQ>zd|0Fm4Zrn
zc~-^9Z2|bUdW%#MP8}M};${2@iqfjm(JWS#=;S)nm@aPTMl=@V7BiStrdKhFsx0KT
zYct@662LXw(!@&22rvq!K6zCUEnkD=&=~mDj4rl_WeI1t7x8la8o$=#+;m7CN$D*P
zaNDwlqyD#elcxz@Z@Rd{HT)SkYKoTY{o-U@*o-71Q*%*zI2^V@!V!yDnA^<e+%^4r
z`GjA<W^!~lbG>-Wo$^CJF>(_gDpp5q!)K(9rt}dM%3a;nxIpcNXtNv?zU#Vhp2J<v
z82&Mr<#?VK^>0Ru<Rpg4uitT6`FbSxgk0j^!fBW?<(w3w^oKyuiFZ{hJ5NO;6I@3l
zSEV^PfJRo|RUG-FDY>^HV{Hid_sYu+r7%nWqM?rYenB1hQ@GVKEW?eJp&xNGY&$yg
zH#cTrSZ-}BUeS%h!Egv8D7b|7;OD*fi(YgkaEuyU+{l6HIQOAkg26%@D!S41hS+4%
z>$lkK0D5qX=o9_&w~cKTn;gfn;$pLdK96&Z2o@Hbnd2C7u@#XJ9oe@bbNlR_BLj$%
zpJwQOv5gZHLX|#>zP@I{q@eF4hSk+km5s=+iC*O2g8UlJeK$^Sq0M-1q{w2z32aNm
zqA<0%a(Q6sMJ^RBTVrB7*Q_w|=wWoIEe0_nbh<+os{K$M1Ujk}HDfqTQ-h%fu7(md
zq}5QWh8#6S)KI1dR}CV@NYsmYg><Jpf9tfiZgs0TCPtb;oX$ZF#{gMGlTy(I(GpYs
z7k8pg?sc8Su0W-sz{dxq*0i{;Jch2Uh=~F8i5|7pm!$?fmM6kESH`e<SqwATW)^yD
zw;XM%wtIj(xS5f!H~lPsPXGrh35U_cL$o4TI4IA}*{tx*EIyV4g>Nc+MJaq!;oCgK
z>^OH=$gvQAdUJr!aWf-(oBwjlUH^+;LGM;3Jp?7nq)`kfKO6*=^87s*)KxJYs^$Kc
zw6!-l`k;KNB{L-x!zzR7P*ev8)PgE~XtYv(){<D$pu#Cy=r`(Xf_ZjK?DKEM=*4Kg
z0gP!<9b>AXTY=gJL)96Yhv{B|`b}In+Wa|OGiKczNIni@An$7{W6ccH-EYApIxMfW
z<wr&w%WuVdhxC0&waE`!lS)CZ6y!=lt`y`-L5>Q3*_x<ozyxpt1@{E=q=GIcBjXXq
z@a_)YojG{-mbNLW!(il~My|0AH3}QcnFY+IAs#ZO+@}n9QtoT3_?}|ps94dotWwQN
zDeiC<fMfZY1V;(g^3%37j7u@MuC<#1BY7$Yvq=`G_bcQxa9a@VnhU<rSD*!Tq-Mo#
z<X;||RA+`*FCUnjjMgyI&lZ`vXdek!W#V~VsKLCzK#P%T=MJFkTKUP`*hp;*$?0mC
zp>A6j^QO&{V3gnmjYf69j6=-HOebPJHw|wjcx@hrBK4L<;<dp$g=H|4M%UuY5iJs{
zP$zsrQhEWSdY&qvqd;$HENaX5Llz_h1^*v~1}Hk_rJmCB6o5QGuUPA#2joZdQpTf@
zZ$;#v=hch(5XL%*=}G88RgxZ;nf6R*GBm}2T;CoSm>0UTQk<L2a5T(|gY7|H*ch=H
zd9D2^d`Q=I%wIB&YGG3dUQod%UrR(M;t)QJu7Nc`l`~~-gB}XnqzlmG5^(_ZMx3T|
z2p)L#22YcpbyVf9LUseIqV7?D0lu^K=-|V0#{3zXQPdV+lc!*M)&jxnQN{tit6Bhr
zz!pHgDCP3#{F%-|pphUAo|ZqIf5F8178OV2_61YT(iroQJhk9fwVH?IuNKt7GtyW%
z<ZcqZ81w0v*ciM6;|p&p+{rzMI$m1;7;ClT#d!{LAVv-z{Pn^OxmjeuiX6okBjmVF
zMC_Cc7ZoZ&ifQYjohig+-{JBir-;!wE5ZzR@NrcKzFAb`U4m9je+&n46wQa^Oxe&`
zl(Pm?UVRl|&tZu|*RrB=OJ@<3$%CDB`40kMldb&l956;~k&TdP^5>le(GH3~H_&RA
z;UiFHmtS;lbdLdNobxbuosc*v`xlSFBKhg!0`DMZ6vn#csKVl)^Nz{*lF`M-(Q$|Y
z7WzgN*<svq>4{0-qtQ7PG3<aS#xVLGdnSTjI|=k;;PF@C_U}3QPfKRor$7{<$4g|{
z(p2GkEWR{SgYrUt(5$JBi<w@^Uo3l;mceOxWN9f#`O4A-;WG$G07EJ#LC;FPs}vrV
znXW=OCp)_4Zo!`nXoa_7fRRwv3fSI&zeH2rA^1xj9x-!W>UU{5Mt+$3%W%QS#lA{t
zMMf&D`5rEm<YLpcL@PBiVLdpK&c*&~iB@aO3>(2hsr<aF9*v(~)?`15BGEMO%Kl}=
zBTF>Th$HS&$BtyOxP?YCz_lY<7B_8F2Do;lJd0an)CahBq#_r4x8?fWu<5U0cI0O4
zvjUT1_MkaRd~7alEHe1Jijeo<TX1!(9+qq1XscY3*76Y*(Jsc^L(7X)_;Dn!EdToY
z>XjAM)sF>6`T(#8#Dg^O3^u*k6lgOYRe9E{rV0I_`>Uk6vMgMIN~lN1@XGwvvSH;K
Od1hq|xbnl5cl;N&lu-Tv

delta 4284
zcmZu!d2m(L89(2->$~T?``)>^$xBFLUP#EEKmvxFuw-Emdqi1En+6C;SQ5+&OB4+T
z5s^jV0E)N(D&mG>+?B;ntIl-B>OcP3nNB;?&UE~v<J4N$e&@cB+L^xOo&Eg2-}$z4
z-b)wZ>?LSApZ9lW5#n$lv+3-wK)XUuMf8u@dtmH~ej2wv&Ht*<{;(-&{;BC>8o$)~
zgXKKecHZP4t@!Nr=;@2cn`*qTfqVm`kL;slKP4wAIelfgT^lC<giV5g_!o2m0q@^X
zB(N6W!e*?<chG|sd=Kkw0`v!H>ZKZi$d9lEz=n*U!1f6E_fgzLfsD}pG6I=jf>5LP
z(;`oe9-zBX0C|As4`FqX-o2NA`ydth*gG^#6RLiQ-kTtxAEuCv)e$;RRY%j+F`BP>
z57YUo`U1kaM`$+g(~qYIPlPLpr2EwVtW#8=#KfcYK0qSmF$(+DVb0J1_hdbuo;^!%
z!^vUt1ie*>ev;0s#x+mTgMeM*Y3dbV_guP5&eMWJ2w_~H1q$&QT5uS<*0Z$bs6u{@
zJ`9hj#zkzLP>tv5NjRk%m*``l>KACfs*;bZ9=u5HBr1@+G)!lq*x1YTEIbjedxbuw
z2$0M4T-bk=>N*O3jXncd>sRP}JcIZVUI=Gjr;`-wD18}TQH?j~YvJxUX()%hMX#uJ
z#@qDuaQ!=U=uNfmUAiCMQjOp}dKlhTt@r66ct<rppr_$o)woKJz<a83jlLi5zD}=(
z^$kRq-x$k>%O{kVugYI#ubHr>d@W6G{l)gW9d&os4XpAaJARc~kRQ1-GLV%Y5s@QM
z)soc`ISO_7e+23Z8ZsL*XJj_yH0I36xgY2QusV4GlFvc%GEB`$jzU%Tl<X>die2SR
z@sihIO-}MdFig%gu=TLeSPqQ-)*RGka7}Y;ZWx-)@jv}XP3}<B<c8|vtm9B-+QhUS
z>>HYC$2Es?Lk3Q%3wD-av$367qkElVs+n9HeVTvnt|;m5>gyWl9PH}Y+}qc=t@p03
zj=}z|U40$9dk1?q4Rr4A+}5!@)opI<Pi^l{*{Q+Kfo@!_ZP=Ky2m3oV3=MXr`1bx?
zT^(I}x;E|{ly~ROm}25gYSUJIORB%m+TJ;k>gn9(j|=MEu(4x%S1Q%n-DPg*Om)>y
z8~r|a4;>x3<z1NLiE$pC0~<Pe`un#!w7P0?|Bl|i-ocvw9X<W2!J1vQHA_?7^T!yk
zQiNBJ^OX(B>)I!l82v7J3`U=t*qkv;NpbfAi&PR+loL-lAZ7(6L9DWvGl#{aVZRhx
zd14XgjK%YPmkZs}g~OxV<*~T$bHQSaGL7lXWd@5f!Axc`n?*!B_gv0AaT{~u*@%$g
zW?00sgk#ylY4!+NN5YQ_K^Y@L=f3L)<;{Yohl#kI-T^vt+}3H43c@u97daYYtuEBp
zIa(}DxPt4h9>gk&x#1YGH0erNUSH!^IhJE4ICCRnu@d00ph|A7FNA7&w7zV(9p&`p
zSrOqdO{`-&ev!HdoU79@!T=BE;*ku-)qzka%yW3$jR!S!J#OAL0>NZPm!wbS#k|a5
zN>J-@ZaSov6za_maNBb5!1z@MPJfVXo%khcD=5QCQytkqy&yR?YQ}Df6=X8t(P-3S
z!VzsO%9W3D*9@k~%hPj)Z$)N|xn81q5XSJB=1uT&c#Ox9XPm{nY%eRQN3f)VvW<X|
zT3V<NBOC7Ou7)Nvm=LWNM^-y;vf>$NQHBVJ&N4jDiwDyYqi_tv<l(~kgoufy9oXIz
zaY<0lX_PYMHieARUjspF-BqqU3cZPpdlNW--lY1a9GAx@$|FIA)fq0oM_vskK_LGe
z)Ut*krxt<<+-e?@H(Go-wIMUIftyj=(Q(hph6)&xdm9ScH==wn9HML_is&Zn4&N)f
z&_jTuhv4954(O2}x;Z!m9fLSjY(%>lqQ@NfuU-ex@>@i=*ebtlXf5l(D6kTu*D;ut
z;OI##DtejY7zwcrkq{l1Z^y-}8+Ro8P-u2BdI%BQIYA**@loycnEY#F{_tiT*F+bN
zZ^3a5>+bOtTWAwPj6j|h#duPBB%&=!Emzq+)M}ecMYAvZxn{+1i5^Ac*<uHJg--8O
z3HpAh3Zoar$4wuLB2^fw;Hs!8R^=pBl&B&@6){zms=`%;@EM7FzL!e}<ZCk~wr*7P
z6BrG}3=(uUQak{-M9fkYT|h$0)*^+Z;x|c&!I0C=(4Y@Uf98_fG9PVL?u$NDz8<$#
z&r*f$XNf4*6+Sn)#V|Z=W}#Vj$>TGt>`j1~NRL;_-_87e_U;f43>3ZxO}Ueng#!=D
zOS8HazM00y7+3hF!dESYZz_D7N0^=94vRPz;y2F@@i|Ub%E8%xs~`Gb{4#o%(&<A`
z6#6gvO%M&eM;YdB^x!HVi)wkIso2^ZZapTiHdRch^c@;UIuz+(fh?%vL*o?^n)BB-
zD!+;61~c@vVLj6q_XM-hP0=mKfiWhkeWodugtQIJHmDcON4l3!gV|g+T7xF88Po3#
zB_D?okjGj|STn=e4q7l`4$JE;+44rS&RX%@5&dq*3g*Z^G$$27t_X5PkSl^*5#&fP
z(vn}<h@S3*1b2q@7(o}Kj`0{{cy<TRj!C>)u56i5co>WfWaJv_kx|rG$t++tjqr#u
zVOS|p$|EghKO=SsiRDd9lWJ0=xWicpj=5$G93@oCA6kmxket|Bi#`a9WGNp^M^geH
zcnbPaSRMMihF6Br+fju&wgTV0nF_N|r#dr4hdkMujMXqRNT-y!sG!rxBQu8<>C2JC
zyii9=ur;4MfZ`|1f3*6;lYMMXQAMpfaGmc>oO>FKB0QtfsE%0S5EDZp{D=;28s2>n
z-ce^Tj`5O;C2DToq7}txUY%=rd#*-W8Z!#@97gpll`uzuj*}KOrTfz$m1iL5|D(_l
zMaS6G3-v4oATQ4;&=$}G^0PUG#*>h33HkM$I<XL<m=rNY2|i(w=F_raZbj{wdtx-=
z`zP(4V}7cl1zpwWx>8+&o6K-DjFyvg!)UQ_vQmCB_X1SNt@9QveGrKPIS*c#*cNkq
zl&L%$5OD~&;To73RF+dVFz{iijP@+da)~$qdLveoIfQpPMcg9!)4a;e)wtYX;OHI=
z7BL#1ie`UEHqD<>F@pT@KDh{!(>e(+kO~fHa&_Ay1m5=QQmK?@=1+CnfX2e$cwT-n
z|D1_gFD?$ty$fzJOMHBV9G2%6%#7h^mN*P7i~Bso=#`>vBHs8)%q8hBOu0Rx3j<@4
zFE)p_!l{MRatFBQP{%9QKLb@Kcm}RO$P9%1%fgQOv>?FCG>-R2#BrUN7~k_T;HXHb
zZP_gQ=Uays?I=WP?7z}@o|DIDg5`xz4VNK|blR%Li_n6Z1HDMB!(<Bae7UGCFJnC>
z3)NF^GfU*UmgUJkZFx{CPqo!%9|538LtDKBj1gO;{i;a*u`MUQfa1?HJo2mXAt=*G
zYw;%c7;u(;`8W}AP!25~fyMIs#W~(Vj71C)%TX!GK?5I?rAziLc^Hk07+|6ORC?Zn
zGcJ8(%s<dDoC=?7=&^Vo2N<b%S?l6B(Bq+3ya8)|<_KRcnPHy*k&9kYEbEsR=5}D(
zrWnAOyn`j7Ln%Kqy~1Fr99UWcQl42l2~NuEOBY5@As_*CyNrB#TKdZ-!K1QqSuQ*#
z?^rgc=S;}x1K7q$B&`K}1wpV(Q;$UmmODIVX1X+3p=B7^Q5vko0V5NiWkSm{3Zq(p
zPrg`jCO+DhX_JhKs2*;al<68cP-9Gu8sR{RgynUp{AtT)+4vhwJzKBKq2&d`%QVkO
zAntO<j#Z{{bB$z(Ysa)SZn04w;@YvYG;Xy~7vkEn@=SdCwd-@DW^fy`W4GdSEL15*
z7^<TvSVftQx#^gS^&9XLxH{$>%QdjHRlLc>ieVKCE_&+|EAmt<aBN;%@$JpgYb&d&
z&W1|*4d7D|H`35D8a}m)d~GWJgaS-;_^v{G=)vtWtGy&zg(TGVWOyaP9dc3oT3YPN
M3+>hN^Y(TB0l0vQ1poj5

diff --git a/precompile/modules/initia_stdlib/sources/minitswap.move b/precompile/modules/initia_stdlib/sources/minitswap.move
index 2ed18e7f..a780fdf8 100644
--- a/precompile/modules/initia_stdlib/sources/minitswap.move
+++ b/precompile/modules/initia_stdlib/sources/minitswap.move
@@ -42,6 +42,7 @@ module initia_std::minitswap {
     const EINVAILD_METADATA: u64 = 16;
     const ESMALL_ARB_PROFIT: u64 = 17;
     const EVIRTUAL_POOL_EXISTS: u64 = 18;
+    const EDEPRECATED: u64 = 19;
 
     const A_PRECISION: u256 = 100;
     const U64_MAX: u128 = 18_446_744_073_709_551_615;
@@ -1451,17 +1452,11 @@ module initia_std::minitswap {
         primary_fungible_store::deposit(signer::address_of(account), return_asset);
     }
 
-    public entry fun finalize_arb(
-        account: &signer,
-        arb_index: u64,
-        output_index: u64,
-        withdrawal_proofs: vector<String>,
-        sender: address,
-        sequence: u64,
-        version: String,
-        state_root: String,
-        storage_root: String,
-        latest_block_hash: String
+    // Since MsgFinalizeWithdrawal can now be executed by anyone,
+    // Make executor triggers MsgFinalizeWithdrawal by themselves
+    // and then calls `finalize_arb_v2`.
+    public entry fun finalize_arb_v2(
+        account: &signer, arb_index: u64
     ) acquires ModuleStore, VirtualPool {
         // check arb info
         let module_store = borrow_global<ModuleStore>(@initia_std);
@@ -1470,46 +1465,41 @@ module initia_std::minitswap {
                 &module_store.global_arb_batch_map,
                 table_key::encode_u64(arb_index)
             );
-        let pool = borrow_global<VirtualPool>(object::object_address(&*pool_obj));
+        let pool_addr = object::object_address(&*pool_obj);
+        let pool = borrow_global<VirtualPool>(pool_addr);
         let arb_info =
             table::borrow(
                 &pool.arb_batch_map,
                 table_key::encode_u64(arb_index)
             );
 
-        // execute finalize token withdrawal
-        let pool_signer = object::generate_signer_for_extending(&pool.extend_ref);
-        let withdrawal_msg =
-            generate_finalize_token_withdrawal_msg(
-                pool.op_bridge_id,
-                output_index,
-                withdrawal_proofs,
-                sender,
-                signer::address_of(&pool_signer),
-                sequence,
-                string::utf8(b"uinit"),
-                arb_info.ibc_op_init_sent,
-                version,
-                state_root,
-                storage_root,
-                latest_block_hash
-            );
-        cosmos::stargate(&pool_signer, withdrawal_msg);
+        let uinit_pool_balance = coin::balance(pool_addr, init_metadata());
+        assert!(
+            uinit_pool_balance >= arb_info.ibc_op_init_sent,
+            error::invalid_state(ENOT_ENOUGH_BALANCE)
+        );
 
         // execute hook
         let module_signer =
             object::generate_signer_for_extending(&module_store.extend_ref);
-        cosmos::move_execute(
-            &module_signer,
-            @initia_std,
-            string::utf8(b"minitswap"),
-            string::utf8(b"finalize_arb_hook"),
-            vector[],
-            vector[
-                bcs::to_bytes(&arb_index),
-                bcs::to_bytes(&signer::address_of(account))
-            ]
-        );
+
+        finalize_arb_hook(&module_signer, arb_index, signer::address_of(account));
+    }
+
+    // deprecated
+    public entry fun finalize_arb(
+        _account: &signer,
+        _arb_index: u64,
+        _output_index: u64,
+        _withdrawal_proofs: vector<String>,
+        _sender: address,
+        _sequence: u64,
+        _version: String,
+        _state_root: String,
+        _storage_root: String,
+        _latest_block_hash: String
+    ) {
+        abort(error::unavailable(EDEPRECATED));
     }
 
     public entry fun finalize_arb_hook(
@@ -2533,38 +2523,6 @@ module initia_std::minitswap {
         amount: u64
     }
 
-    fun generate_finalize_token_withdrawal_msg(
-        bridge_id: u64,
-        output_index: u64,
-        withdrawal_proofs: vector<String>,
-        sender: address,
-        receiver: address,
-        sequence: u64,
-        denom: String,
-        amount: u64,
-        version: String,
-        state_root: String,
-        storage_root: String,
-        latest_block_hash: String
-    ): vector<u8> {
-        json::marshal(
-            &FinalizeTokenWithdrawalRequest {
-                _type_: string::utf8(b"/opinit.ophost.v1.MsgFinalizeTokenWithdrawal"),
-                bridge_id,
-                output_index,
-                withdrawal_proofs,
-                sender: to_sdk(sender),
-                receiver: to_sdk(receiver),
-                sequence,
-                amount: CosmosCoin { denom, amount },
-                version,
-                state_root,
-                storage_root,
-                latest_block_hash
-            }
-        )
-    }
-
     fun init_metadata(): Object<Metadata> {
         let addr = object::create_object_address(&@initia_std, b"uinit");
         object::address_to_object<Metadata>(addr)
@@ -3143,28 +3101,6 @@ module initia_std::minitswap {
         );
     }
 
-    #[test]
-    fun test_finalize_token_withdrawal_msg() {
-        let msg =
-            generate_finalize_token_withdrawal_msg(
-                1,
-                2,
-                vector[string::utf8(b"abc"), string::utf8(b"123")],
-                @0x1,
-                @0x2,
-                3,
-                string::utf8(b"uinit"),
-                100,
-                string::utf8(b"version"),
-                string::utf8(b"state_root"),
-                string::utf8(b"storage_root"),
-                string::utf8(b"latest_block_hash")
-            );
-        let json_str =
-            b"{\"@type\":\"/opinit.ophost.v1.MsgFinalizeTokenWithdrawal\",\"amount\":{\"amount\":\"100\",\"denom\":\"uinit\"},\"bridge_id\":\"1\",\"latest_block_hash\":\"latest_block_hash\",\"output_index\":\"2\",\"receiver\":\"init1qqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqzwsp0lj\",\"sender\":\"init1qqqqqqqqqqqqqqqqqqqqqqqqqqqqqqqpqr5e3d\",\"sequence\":\"3\",\"state_root\":\"state_root\",\"storage_root\":\"storage_root\",\"version\":\"version\",\"withdrawal_proofs\":[\"abc\",\"123\"]}";
-        assert!(msg == json_str, 0);
-    }
-
     #[test]
     fun test_ibc_denom() {
         let ibc_denom =