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

[REVIEW] - fix swap in oraichain (inj, kwt, milky ) #222

Merged
merged 2 commits into from
Mar 26, 2024
Merged

Conversation

haunv3
Copy link
Contributor

@haunv3 haunv3 commented Mar 25, 2024

No description provided.

@haunv3 haunv3 changed the title fix swap in oraichain (inj, kwt, milky ) [REVIEW] - fix swap in oraichain (inj, kwt, milky ) Mar 25, 2024
trung2891
trung2891 previously approved these changes Mar 25, 2024
Copy link

github-actions bot commented Mar 26, 2024

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       1  96.96%   24
packages/oraidex-common/src/bigdecimal.ts                                             80      29  63.75%   23, 40, 45-53, 120-153, 173-177
packages/oraidex-common/src/constant.ts                                              130       0  100.00%
packages/oraidex-common/src/helper.ts                                                210      65  69.04%   141, 145-146, 162-167, 265-267, 275-287, 298-301, 324-328, 332-334, 352-371, 375-382, 386-387, 391-392, 397-405, 415-425, 435-436
packages/oraidex-common/src/network.ts                                                68       0  100.00%
packages/oraidex-common/src/pairs.ts                                                  44       2  95.45%   151, 157
packages/oraidex-common/src/token.ts                                                  27       0  100.00%
packages/oraidex-server/src/db-query.ts                                              128      29  77.34%   41-79, 160, 222, 235-237, 281
packages/oraidex-server/src/helper.ts                                                167     107  35.92%   102, 118-120, 124, 128, 135, 143-156, 161-174, 180-230, 235-280, 297-317, 330-336, 345-351, 370-410, 431, 437-458
packages/oraidex-server/src/map-cache.ts                                              37       2  94.59%   21-22
packages/oraidex-server/src/polyfill.ts                                                2       0  100.00%
packages/oraidex-sync/src/constants.ts                                                26       0  100.00%
packages/oraidex-sync/src/db.ts                                                      122      55  54.91%   43, 63-68, 131-151, 214-221, 229-262, 286-361, 440-494, 557-609
packages/oraidex-sync/src/helper.ts                                                  217      33  84.79%   65, 69, 78, 89, 99-109, 258-264, 307-309, 388-396, 405-418, 484-486
packages/oraidex-sync/src/index.ts                                                    99      84  15.15%   20-223
packages/oraidex-sync/src/pairs.ts                                                    11       0  100.00%
packages/oraidex-sync/src/parse.ts                                                    25       2  92.00%   38-39
packages/oraidex-sync/src/pool-helper.ts                                             289     123  57.43%   84-85, 101-104, 125, 214-219, 301-318, 325-328, 332-337, 341-346, 350-357, 361-368, 372-378, 383-424, 433-464, 468-471, 475-477, 481-482, 531-539, 622-683
packages/oraidex-sync/src/query.ts                                                    63      50  20.63%   21-162
packages/oraidex-sync/src/tx-parsing.ts                                              266     182  31.57%   51-56, 81-151, 219-220, 228, 257-413, 437-582, 594-619
packages/universal-swap/src/handler.ts                                               263     151  42.58%   70, 92, 107-108, 117-118, 148-168, 231-260, 271-383, 389-632, 654, 670, 708, 727, 773
packages/universal-swap/src/helper.ts                                                310      69  77.74%   73-74, 202, 235, 239, 329, 336, 349-352, 446, 455-486, 519, 570, 587, 611, 618-641, 674, 694-725, 810-819, 831-836, 855
packages/universal-swap/src/index.ts                                                   5       0  100.00%
packages/universal-swap/src/swap-filter.ts                                            10       0  100.00%
packages/universal-swap/src/types.ts                                                  10       0  100.00%
packages/universal-swap/src/wrapper.ts                                                26      20  23.07%   14-28, 40-54, 67-83, 100-116
packages/universal-swap/src/proto/proto-gen.ts                                        13       0  100.00%
TOTAL                                                                               2687    1004  62.63%

Diff against main

Filename                                  Stmts    Miss  Cover
--------------------------------------  -------  ------  -------
packages/universal-swap/src/handler.ts       +2       0  +0.44%
packages/universal-swap/src/helper.ts       +10      -5  +2.41%
TOTAL                                       +12      -5  +0.35%

Results for commit: 1ab22ee

Minimum allowed coverage is 0%

♻️ This comment has been updated with latest results

Copy link
Contributor

@trung2891 trung2891 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

@haunv3 haunv3 merged commit 16f7c7d into main Mar 26, 2024
1 check passed
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.

None yet

4 participants