tezos-checker / checker

An in-development "robocoin" system for the Tezos blockchain
24 stars 16 forks source link

Cleaner sealing #307

Closed gkaracha closed 2 years ago

gkaracha commented 2 years ago

Tiny PR I've been meaning to open for a little while. When sealing the contract we can pass the external addresses as a record instead of an uninformative tuple of addresses address * address * address * address. This makes is harder to pass the addresses in the wrong order (which has happened to me in the past during development).

github-actions[bot] commented 2 years ago
Gas costs 30bba1a4420197da491a1a67fc88547c7fb9aeb1 30ddc43c237033ec58faa6fb4cf36d1b706ffa72 Diff
checker%touch 42380 None -42380
checker%deactivate_burrow 23378 None -23378
checker%withdraw_collateral 23019 None -23019
checker%create_burrow 22283 None -22283
checker%deposit_collateral 19985 None -19985
checker%activate_burrow 19801 None -19801
checker%set_burrow_delegate 18304 None -18304
checker%remove_liquidity 13269 None -13269
wtez%transfer 12726 None -12726
checker%add_liquidity 12574 None -12574
checker%buy_kit 12420 None -12420
checker%sell_kit 12419 None -12419
wtez%withdraw 9875 None -9875
wctez%mint 9127 None -9127
wctez%redeem 8655 None -8655
wtez%set_delegate 8409 None -8409
wtez%deposit 7490 None -7490
checker%burn_kit 7361 None -7361
checker%mint_kit 7133 None -7133
checker%transfer 6313 None -6313
checker%touch_burrow 6104 None -6104
checker%update_operators 5591 None -5591
wctez%transfer 4902 None -4902
mockFA2%transfer 4776 None -4776
wtez%update_operators 4170 None -4170
mockFA2%redeem 3671 None -3671
wctez%update_operators 3444 None -3444
mockFA2%mint 3436 None -3436
mockFA2%update_operators 3318 None -3318
Entrypoint sizes 30bba1a4420197da491a1a67fc88547c7fb9aeb1 30ddc43c237033ec58faa6fb4cf36d1b706ffa72 Diff
touch 56556 None -56556
mark_for_liquidation 16949 None -16949
touch_liquidation_slices 14350 None -14350
cancel_liquidation_slice 12130 None -12130
liquidation_auction_place_bid 2141 None -2141
remove_liquidity 1998 None -1998
add_liquidity 1878 None -1878
mint_kit 1537 None -1537
burn_kit 1514 None -1514
deactivate_burrow 1465 None -1465
buy_kit 1395 None -1395
sell_kit 1391 None -1391
withdraw_collateral 1298 None -1298
activate_burrow 1272 None -1272
create_burrow 1192 None -1192
deposit_collateral 1145 None -1145
liquidation_auction_claim_win 1096 None -1096
touch_burrow 611 None -611
set_burrow_delegate 457 None -457
update_operators 435 None -435
receive_ctez_marginal_price 140 None -140
receive_price 120 None -120
Test coverage 30bba1a4420197da491a1a67fc88547c7fb9aeb1 30ddc43c237033ec58faa6fb4cf36d1b706ffa72 Diff
cfmm.ml 100 None -100
kit.ml 100 None -100
fa2Implementation.ml 100 None -100
fa2Ledger.ml 100 None -100
parameters.ml 100 None -100
fixedPoint.ml 100 None -100
liquidationAuctionTypes.ml 100 None -100
price.ml 100 None -100
error.ml 100 None -100
ptr.ml 100 None -100
cfmmTypes.ml 100 None -100
constants.ml 100 None -100
driftDerivative.ml 100 None -100
mem.ml 100 None -100
tokenMetadata.ml 100 None -100
checkerEntrypoints.ml 97.73 None -97.73
common.ml 97.59 None -97.59
burrow.ml 96.21 None -96.21
sliceList.ml 95.38 None -95.38
checker.ml 94.93 None -94.93
checkerMain.ml 93.33 None -93.33
avl.ml 91.05 None -91.05
tok.ml 78.38 None -78.38
liquidationAuction.ml 77.83 None -77.83
checkerTypes.ml 77.78 None -77.78
lqt.ml 72.97 None -72.97
ctok.ml 72.97 None -72.97
burrowOrigination.ml 8 None -8
TOTAL 92.57 None -92.57