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.

Sign up for GitHub

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

Jump to bottom

Integration test and tx mapping with send2 api call #77

Open
wants to merge 26 commits into
base: telos-main
Choose a base branch
from
Open

Integration test and tx mapping with send2 api call #77

wants to merge 26 commits into from

Conversation

lesa-telos
Copy link

@lesa-telos lesa-telos commented Dec 9, 2024 •
edited
Loading

Fixes #57

Description

  • Update tests.
  • Use sent_transaction2 api that retries transactions if the transaction fails.
  • Add Eth transaction error mapping.

Test scenarios

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have cleaned up the code in the areas my change touches
  • My changes generate no new warnings
  • Any dependent changes have been merged and published in downstream modules
  • I have checked my code and corrected any misspellings
  • I have removed any unnecessary console messages
  • I have included all english text to the translation file and/or created a new issue with the required translations for the currently supported languages
  • I have tested for mobile functionality and responsiveness
  • I have added appropriate test coverage
  • I have updated relevant documentation and/or opened a separate issue to cover the updates (Issue URL: )

lesa-telos and others added 26 commits October 22, 2024 21:30
@coa-telos
Add test for eip1559 and eip2930 transaction types.
6a6b5b9
Add double approve test.
51ef536
@coa-telos
Add RLP tests
6e4d94f
Add contract deployment history blocks check.
5420601
Check tx receipt when calling approve.
08a79b1
@coa-telos
Add signed and unsigned txs tests
5da4f14
Do not send to reth tx pool.
db14769
Wait for receipt.
1b9a322
@coa-telos
Read nonce from the node
0d5fb63
Assert nonce for the account after the reth is synced with blocks in …
3147a2e 
…container.
Add a revision test check.
f0c8a43
Remove unused config struct.
20ba9a4
Merge branch 'telos-main' into integration-more-tests
e7aa464 
# Conflicts:
#	crates/telos/node/tests/live_test_runner.rs
Use send2 in tests.
79ddc09
Add custom errors.
21ded28
Update tests.
9ec1938
@coa-telos
Handle more errors
fcecc32
Handle http error.
6288bd0
@coa-telos
Warn on every serer error
7054f9a
@coa-telos
Handle send_transaction2 error
7a7cd30
@coa-telos
Update antelope and consensus client
939b670
@coa-telos
Update antelope client and handle details on error
3c3362e
Clean imports.
6d351d9
Add options when sending tx with retry.
1b21d2d
Merge branch 'telos-main' into integration-more-tests
3b9b9df 
# Conflicts:
#	Cargo.lock
#	Cargo.toml
Update Cargo.lock
41bf1f5
@lesa-telos lesa-telos changed the title Integration more tests Integration test and tx mapping with send2 api call Dec 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@poplexity poplexity Awaiting requested review from poplexity

@aamirpashaa aamirpashaa Awaiting requested review from aamirpashaa

@guilledk guilledk Awaiting requested review from guilledk

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Attempt to map Telos error responses to better reponses from the RPC for sendRawTransaction
2 participants
@lesa-telos @coa-telos