-
Notifications
You must be signed in to change notification settings - Fork 23
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 consistency #239
Test consistency #239
Conversation
/workflows/benchmarks agents/token/send,agents/token/test_swap.py,agents/token/test_swap_and_send.py 1 |
Finished benchmarks Test Run Summary
Detailed Results
Total run time: 8.61 minutes |
/workflows/benchmarks agents/token/test_swap.py,agents/token/test_swap_and_send.py |
Finished benchmarks Test Run Summary
Detailed Results
Total run time: 29.16 minutes |
/workflows/benchmarks agents/token/research |
Finished benchmarks Test Run Summary
Detailed Results
Total run time: 48.74 minutes |
/workflows/benchmarks agents/token/research/test_research.py::test_get_top_5_memecoins,agents/token/research/test_research.py::test_get_top_5_memecoins,agents/token/research/test_research.py::test_price_change_informationm,agents/token/research/test_research_and_swap.py::test_research_and_buy_multiple,agents/token/research/test_research_swap_and_send.py::test_research_buy_multiple_send_multiple 5 |
Finished benchmarks Test Run Summary
Detailed Results
Total run time: 12.80 minutes |
/workflows/benchmarks agents/token/research/test_research.py::test_get_top_5_memecoins,agents/token/research/test_research.py::test_price_change_information |
Finished benchmarks Test Run Summary
Detailed Results
Total run time: 4.47 minutes |
/workflows/benchmarks agents/token/research/test_advanced.py::test_research_and_swap_many_tokens_subjective_simple 5 |
Finished benchmarks Test Run Summary
Detailed Results
Total run time: 8.76 minutes |
/workflows/benchmarks agents/token/research/test_research.py::test_get_top_5_memecoins,agents/token/research/test_advanced.py::test_research_and_swap_many_tokens_subjective_simple 10 |
Finished benchmarks Test Run Summary
Detailed Results
Total run time: 22.46 minutes |
|
||
receiver = test_accounts[0] | ||
|
||
shib_address = ETHAddress(auto_tx.network.tokens["shib"]) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we have a lot of tests that fetch the meme coin with largest market cap - since this is dynamic, should we get the data from coingecko in each test?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You're right, I just made an issue for it here: #247
Seems a bit lower prio though since the highest market cap coins won't likely change that soon
Closes #166