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

feat: parallel testing #2821

Merged
merged 10 commits into from
Feb 11, 2025
Merged

feat: parallel testing #2821

merged 10 commits into from
Feb 11, 2025

Conversation

ivaylonikolov7
Copy link
Contributor

@ivaylonikolov7 ivaylonikolov7 commented Jan 23, 2025 •
edited
Loading

Description:

  • Parallel testing in miltiple threads to increase test execution speed.
  • Use MirrorNodeContractCallQuery where applicable to circumvent throttle mechanic
  • reduce stress iteration count to improve unit test speed
  • remove backoff mechanism when running on local-node
  • increase retry attempts when running on local node

Fixes #2710

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

@ivaylonikolov7
build: update local node initial account balance
c74fb71 
Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7
test: run tests in paralel
1195571 
Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7
chore: update stress iteration count
1fa1898 
Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7
build: add jobs
c305474 
Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7
build: paralel codecov tests
2029342 
Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7
build: remove redundant unit test run
592c7b5 
Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7
fix: regenerate transaction id if query payment fails
c2639ac 
Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7
feat: local-node config for maxAttempts and disable backoff mechanism…
0efde37 
… on localnode

Signed-off-by: Ivaylo Nikolov <[email protected]>
@ivaylonikolov7 ivaylonikolov7 marked this pull request as ready for review January 27, 2025 12:48
@ivaylonikolov7 ivaylonikolov7 requested review from a team as code owners January 27, 2025 12:48
0xivanov
0xivanov approved these changes Feb 11, 2025
View reviewed changes
Copy link
Contributor

@0xivanov 0xivanov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@ivaylogarnev-limechain ivaylogarnev-limechain merged commit ab58be7 into main Feb 11, 2025
9 checks passed
@ivaylogarnev-limechain ivaylogarnev-limechain deleted the feat/paralel-testing branch February 11, 2025 09:24
@ivaylonikolov7 ivaylonikolov7 mentioned this pull request Feb 11, 2025
Closed
@ivaylogarnev-limechain ivaylogarnev-limechain mentioned this pull request Feb 11, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mishomihov00 mishomihov00 mishomihov00 left review comments

@PavelSBorisov PavelSBorisov PavelSBorisov approved these changes

@0xivanov 0xivanov 0xivanov approved these changes

@ivaylogarnev-limechain ivaylogarnev-limechain ivaylogarnev-limechain approved these changes

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

Successfully merging this pull request may close these issues.
5 participants
@ivaylonikolov7 @PavelSBorisov @0xivanov @ivaylogarnev-limechain @mishomihov00