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

Merged
merged 54 commits into from
Mar 19, 2025
Merged

Integration test and tx mapping with send2 api call #77

merged 54 commits into from
Mar 19, 2025

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
guilledk and others added 5 commits January 27, 2025 20:00
@guilledk
Copy link

Last commit I push addresses the first point in #23

poplexity and others added 23 commits January 30, 2025 15:49
@poplexity @guilledk
Adding bad_memo test
e89645e
@poplexity @guilledk
A more-bad string for the memo, fix import for now
a89bed6
@guilledk
Fix wrong authorization account on transfer_tx helper
45ecc2a
@guilledk
Refactor integration test suite & tools
3074a04
@guilledk
Update CI
722dc28
@guilledk
Increase time between 500 tx batch in 2k tx test & remove all warning…
3cc1360 
…s from integration_main module compile step
@guilledk
Change timeout some more in 2k tx test to make it pass in ci
f6ae416
@poplexity
Merge pull request #80 from telosnetwork/test_refactor
d6748bc 
Refactor integration test suite & tools
@aamirpashaa
Add two tests for address zero
7290ced
@aamirpashaa
Change the number of txs in a batch in doresources sandwich
68f8408
@aamirpashaa
Add get account bug test
444d92c
@aamirpashaa
Add delegate call bug test
4443a73
@aamirpashaa
Add tests for RLPs with extra bytes and leading zero in value field
d0de326
@aamirpashaa
Add setrevision sandwich
0d97c62
@guilledk
Merge pull request #81 from telosnetwork/address_zero_tests
6232a6f 
Add remaining integration tests
@guilledk
Add non evm bad memo utf8 test
3b7cc3c
@guilledk
Factor doresources sandwich into single test
239e2dd
@guilledk
Add pending == latest rpc tests
50e7b2c
@guilledk
Remove pending == latest checks, only check if rpc didnt return error
d7f474e
@guilledk
Remove compile time warnings
9afd959
@guilledk
Enable storage override only for testnet block 137,430,500
1f6e1cd
@poplexity
Merge pull request #85 from telosnetwork/testnet_injection_storage_ov…
b4f5ecb 
…erride

Enable storage override only for testnet block 137,430,500
@poplexity
Merge pull request #84 from telosnetwork/lowhanging_test_fruit
3371495 
Lowhanging test fruit
@poplexity poplexity merged commit 36d70ce into telos-main Mar 19, 2025
1 check passed
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
5 participants
@lesa-telos @guilledk @poplexity @coa-telos @aamirpashaa