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

test: btt tests for joinswapPoolAmountOut #170

Merged
merged 4 commits into from
Jul 22, 2024

Conversation

0xteddybear
Copy link

No description provided.

@0xteddybear 0xteddybear marked this pull request as ready for review July 22, 2024 16:56
Comment on lines 15 to 16
├── it queries the contracts token in balance
├── it emits LOG_JOIN event for token
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
├── it queries the contracts token in balance
├── it emits LOG_JOIN event for token
├── it queries token in balance
├── it emits LOG_JOIN event for token in

├── it emits LOG_JOIN event for token
├── it mints the pool shares
├── it sends pool shares to caller
├── it calls _pullUnderlying for token
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
├── it calls _pullUnderlying for token
├── it calls _pullUnderlying for token in

Copy link
Member

@wei3erHase wei3erHase left a comment

Choose a reason for hiding this comment

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

smol typos, other than that 👍

Copy link
Member

@wei3erHase wei3erHase left a comment

Choose a reason for hiding this comment

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

good! 👍

@wei3erHase wei3erHase merged commit 49aba91 into dev Jul 22, 2024
4 checks passed
@wei3erHase wei3erHase deleted the test/btt-joinswapPoolAmountOut branch July 22, 2024 18:02
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.

2 participants