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

fix(tests): add more sepolia endpoints in tests #2262

Merged
merged 1 commit into from
Nov 4, 2024

Conversation

laruh
Copy link
Member

@laruh laruh commented Nov 4, 2024

In current dev we use only one url in sepolia endpoints, which is experiencing issues right now:
Failed to get client version for all urls

failures:

---- mm2_tests::eth_tests::test_sign_eth_transaction stdout ----
04 10:22:29, for_tests:1547] sending rpc request {"userpass":"pass","method":"enable","coin":"ETH","urls":["https://rpc2.sepolia.org/"],"swap_contract_address":"0xeA6D65434A15377081495a9E7C5893543E7c32cB","mm2":1} to http://127.0.0.150:7783/
thread 'mm2_tests::eth_tests::test_sign_eth_transaction' panicked at 'assertion failed: `(left == right)`
  left: `500`,
 right: `200`: 'enable "ETH"' failed: {"error":"rpc:184] dispatcher_legacy:141] lp_commands_legacy:169] lp_coins:4463] eth:6530] Failed to get client version for all urls"}', mm2src/mm2_main/tests/mm2_tests/eth_tests.rs:75:5
note: run with `RUST_BACKTRACE=1` environment variable to display a backtrace

---- mm2_tests::eth_tests::test_sign_eth_transaction_eip1559 stdout ----
04 10:22:30, for_tests:1547] sending rpc request {"userpass":"pass","method":"enable","coin":"ETH","urls":["https://rpc2.sepolia.org/"],"swap_contract_address":"0xeA6D65434A15377081495a9E7C5893543E7c32cB","mm2":1} to http://127.0.0.195:7783/
thread 'mm2_tests::eth_tests::test_sign_eth_transaction_eip1559' panicked at 'assertion failed: `(left == right)`
  left: `500`,
 right: `200`: 'enable "ETH"' failed: {"error":"rpc:184] dispatcher_legacy:141] lp_commands_legacy:169] lp_coins:4463] eth:6530] Failed to get client version for all urls"}', mm2src/mm2_main/tests/mm2_tests/eth_tests.rs:75:5

---- mm2_tests::mm2_tests_inner::test_convert_eth_address stdout ----
04 10:23:05, mm2_tests_inner:2682] log path: /var/folders/g6/rgtlsw6n123b0gt5483s5_cm0000gn/T/mm2_2024-11-04_10-23-05-131_127.0.0.84/mm2.log
04 10:23:05, for_tests:1547] sending rpc request {"userpass":"pass","method":"enable","coin":"ETH","urls":["https://rpc2.sepolia.org/"],"swap_contract_address":"0xeA6D65434A15377081495a9E7C5893543E7c32cB","path_to_address":{"account_id":0,"chain":"External","address_id":0},"mm2":1} to http://127.0.0.84:7783/
thread 'mm2_tests::mm2_tests_inner::test_convert_eth_address' panicked at 'assertion failed: `(left == right)`
  left: `500`,
 right: `200`: 'enable' failed: {"error":"rpc:184] dispatcher_legacy:141] lp_commands_legacy:169] lp_coins:4463] eth:6530] Failed to get client version for all urls"}', /Users/runner/work/komodo-defi-framework/komodo-defi-framework/mm2src/mm2_test_helpers/src/for_tests.rs:2003:5

This pr provides more urls.

@laruh laruh requested a review from shamardy November 4, 2024 13:35
@laruh laruh changed the title fix(test-nodes) add more sepolia endpoints in tests fix(tests): add more sepolia endpoints in tests Nov 4, 2024
@shamardy shamardy merged commit 97d1813 into dev Nov 4, 2024
22 of 27 checks passed
@shamardy shamardy deleted the fix-sepolia-nodes-in-tests branch November 4, 2024 13:50
dimxy added a commit that referenced this pull request Nov 5, 2024
* dev:
  fix(hd-wallet): use `CoinBalanceMap` for UTXO and QTUM (#2259)
  fix(tests): add more sepolia endpoints in tests (#2262)
dimxy added a commit that referenced this pull request Nov 7, 2024
* dev:
  fix(nft): add token_id field to the tx history primary key, fix balance (#2209)
  feat(cosmos): support IBC types in tx history implementation (#2245)
  fix(hd-wallet): use `CoinBalanceMap` for UTXO and QTUM (#2259)
  fix(tests): add more sepolia endpoints in tests (#2262)
  fix(legacy-swap): check for confirmations on recover taker (#2242)
  fix(legacy-swap): remove the need for takers to confirm their payment (#2249)
  refactor(P2P): types and modules (#2256)
  fix(evm): correctly display eth addr in iguana v2 activation result (#2254)
  feat(utxo): prioritize electrum connections (#1966)
  refactor(SwapOps): make all methods async (#2251)
  refactor(SwapOps): make `send_maker_payment` async (#2250)
  remove old p2p implementation (#2248)
dimxy added a commit that referenced this pull request Nov 11, 2024
* dev:
  fix(foot-shooting): remove leftover code that panics via RPC (#2270)
  refactor(MarketCoinOps): make `wait_for_htlc_tx_spend` async (#2265)
  feat(eth-swap): maker tpu v2 implementation (#2211)
  fix(nft): add token_id field to the tx history primary key, fix balance (#2209)
  feat(cosmos): support IBC types in tx history implementation (#2245)
  fix(hd-wallet): use `CoinBalanceMap` for UTXO and QTUM (#2259)
  fix(tests): add more sepolia endpoints in tests (#2262)
  fix(legacy-swap): check for confirmations on recover taker (#2242)
  fix(legacy-swap): remove the need for takers to confirm their payment (#2249)
  refactor(P2P): types and modules (#2256)
  fix(evm): correctly display eth addr in iguana v2 activation result (#2254)
  feat(utxo): prioritize electrum connections (#1966)
  refactor(SwapOps): make all methods async (#2251)
  refactor(SwapOps): make `send_maker_payment` async (#2250)
  remove old p2p implementation (#2248)
  feat(cosmos-offline-tests): prepare IBC channels inside the container  (#2246)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants