Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update code with first case is swap in osmosis #272

Merged
merged 3 commits into from
May 27, 2024

Conversation

haunv3
Copy link
Contributor

@haunv3 haunv3 commented May 27, 2024

No description provided.

@haunv3 haunv3 changed the title add case with first is swap in osmosis Update code with first case is swap in osmosis May 27, 2024
trung2891
trung2891 previously approved these changes May 27, 2024
packages/universal-swap/src/handler.ts Outdated Show resolved Hide resolved
Copy link

badge

Code Coverage Summary

Filename                                                                           Stmts    Miss  Cover    Missing
-------------------------------------------------------------------------------  -------  ------  -------  --------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------------
packages/oraidex-common-ui/src/components/TVChartContainer/helpers/constants.ts        6       0  100.00%
packages/oraidex-common-ui/src/components/TVChartContainer/helpers/utils.ts           33       2  93.93%   23-24
packages/oraidex-common/src/alpha-network.ts                                           6       0  100.00%
packages/oraidex-common/src/bigdecimal.ts                                             80      29  63.75%   23, 40, 45-53, 120-153, 173-177
packages/oraidex-common/src/constant.ts                                              122       0  100.00%
packages/oraidex-common/src/helper.ts                                                251      68  72.90%   142, 146-147, 163-168, 277-279, 287-299, 309-313, 336-340, 344-346, 364-383, 387-394, 398-399, 403-404, 409-417, 427-448, 534, 560
packages/oraidex-common/src/network.ts                                                18       0  100.00%
packages/oraidex-common/src/pairs.ts                                                  28       2  92.85%   151, 157
packages/oraidex-common/src/token.ts                                                  27       0  100.00%
packages/universal-swap/src/handler.ts                                               413     277  32.92%   74, 96, 111-112, 119-122, 133-134, 164-185, 265-294, 305-310, 314-419, 423-424, 446-455, 459-460, 486-591, 595-598, 603-732, 738-995, 1005-1010, 1032, 1039-1075, 1094, 1104-1110, 1117, 1157-1202, 1220, 1266
packages/universal-swap/src/helper.ts                                                382      78  79.58%   80-81, 218, 353, 360, 373-376, 452-465, 494, 560, 600, 609-647, 688, 761, 778, 806, 813-836, 870, 874, 894-927, 1012-1021, 1033-1038, 1057, 1084
packages/universal-swap/src/swap-filter.ts                                            11       0  100.00%
packages/universal-swap/src/types.ts                                                   2       0  100.00%
packages/universal-swap/src/proto/proto-gen.ts                                        13       0  100.00%
TOTAL                                                                               1392     456  67.24%

Diff against main

Filename                                  Stmts    Miss  Cover
--------------------------------------  -------  ------  -------
packages/universal-swap/src/handler.ts       +9      +9  -0.74%
packages/universal-swap/src/helper.ts        +2       0  +0.11%
TOTAL                                       +11      +9  -0.39%

Results for commit: f0e411c

Minimum allowed coverage is 0%

♻️ This comment has been updated with latest results

@haunv3 haunv3 merged commit 7b294ed into main May 27, 2024
1 check passed
@quangdz1704 quangdz1704 deleted the feat/smart-router-osmosis-pool branch May 27, 2024 03:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants