Add Unit Tests #17
Labels
enhancement
New feature or request
Milestone
Comments
spazcoin
added a commit
that referenced
this issue
Jan 24, 2023
* add unit tests for the following token swap methods * swapExactTokensForTokens * swapExactTokensForETH * swapExactETHForTokens * swapExactTokensForTokensSupportingFeeOnTransferTokens * swapTokensForExactETH * swapTokensForExactTokens Those unit tests cover both Solarbeam and Huckleberry, 2-hop transactions, and multiple uses of filtering '==', '<=', '>=' on both 'timeStamp' and 'blockNumber'. * add a unit test for failure case where tx query returns no results 1.5hr contributes to #17
spazcoin
added a commit
to spazcoin/subscrape
that referenced
this issue
Jan 27, 2023
* unit tests: * lots of new unit tests for adding and removing liquidity, Zenlink tx, and created (but not working) Solarbeam StableSwap transaction tests. * add debug ability to skip sets of unit tests to help me focus on new/changing tests. * production code: * standardize using lowercase versions of EVM addresses. Some interfaces were returning capital letters, causing string comparisons to fail. (this might be checksum encoding, but we don't need it.) * refactor `removeLiquidity` to handle situation where tokens from burned LP are not in same order as initial request (similar pattern from `addLiquidity`). * add/remove liquidity - support Deposit/Withdraw events, which might hold the other part of tokens being transferred. contributes to ChaosDAO-org#17 7 hours
spazcoin
added a commit
to spazcoin/subscrape
that referenced
this issue
Jan 30, 2023
* refactor `add_liquidity` to support single-sided LP creation (ignore intermediate tokens) * more unit tests for different token swap methods and add liquidity methods 4.5hrs contributes to ChaosDAO-org#17
spazcoin
added a commit
to spazcoin/subscrape
that referenced
this issue
Feb 19, 2023
* print filename:lineno in log messages for better debugging * add automated testing for Python 3.11 (and drop 3.9 to save time) * don't check "output vs input" tolerance if tx request values were zero. * unit test fix: only use lowercase account numbers 1hrs contributes to ChaosDAO-org#17
spazcoin
added a commit
to spazcoin/subscrape
that referenced
this issue
Feb 19, 2023
* print filename:lineno in log messages for better debugging * don't check "output vs input" tolerance if tx request values were zero. * unit test fix: only use lowercase account numbers 1hrs contributes to ChaosDAO-org#17
spazcoin
added a commit
to spazcoin/subscrape
that referenced
this issue
Feb 20, 2023
* unit tests and parsing script support for: * removeLiquidity * removeLiquidityETH * removeLiquidityETHWithPermit * removeLiquidityETHSupportingFeeOnTransferTokens * removeLiquidityETHWithPermitSupportingFeeOnTransferTokens * If two transactions in same block (same timestamp), store as slightly different timestamp so they don't get overwritten. * Fix bug where subscrape would query the block explorer twice, even when only requesting data for a single block. 3hrs contributes to ChaosDAO-org#17
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
No description provided.
The text was updated successfully, but these errors were encountered: