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

perf(test-suite): speedup test suite up to 6 times #30

Merged
merged 15 commits into from
Nov 12, 2021
Merged

perf(test-suite): speedup test suite up to 6 times #30

merged 15 commits into from
Nov 12, 2021

Conversation

shumkov
Copy link
Member

@shumkov shumkov commented Nov 11, 2021 •
edited
Loading

Issue being fixed or feature implemented

Test suite is running for 30 minutes which tooo slooow. After simple optimisations it takes 5 mins.

What was done?

  • Remove specified drive protocol version
  • Set mnemonic to null to disable sync for funded client
  • Remove unnecessary waiting for blocks and balance changes
  • Set tenderdash empty block interval to 10s
  • Move update from dashmate configure script to start network script

How Has This Been Tested?

With CI and comparing elapsed time.

Breaking Changes

None

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation

For repository code-owners and collaborators only

  • I have assigned this pull request to a milestone

@shumkov shumkov added this to the v0.21.x milestone Nov 11, 2021
@shumkov shumkov changed the title perf(test-suite): speedup some tests perf(test-suite): speedup tests Nov 11, 2021
@shumkov shumkov marked this pull request as ready for review November 11, 2021 19:17
@shumkov shumkov changed the title perf(test-suite): speedup tests perf(test-suite): speedup tests up to 5 min Nov 11, 2021
@shumkov shumkov changed the title perf(test-suite): speedup tests up to 5 min perf(test-suite): speedup tests x6 Nov 12, 2021
@shumkov shumkov changed the title perf(test-suite): speedup tests x6 perf(test-suite): speedup test suite up to 6 times Nov 12, 2021
@shumkov shumkov merged commit a732827 into master Nov 12, 2021
@shumkov shumkov deleted the opimize-test-suite branch November 12, 2021 10:43
pshenmic pushed a commit that referenced this pull request Jun 24, 2022
@markin-io
Implements regtest (#30)
3eeb01e
shumkov added a commit that referenced this pull request Nov 23, 2022
@shumkov
Merge pull request #30 from dashevo/node-binding
39b2324 
feat: binding for Node.JS
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jawid-h jawid-h jawid-h approved these changes

@antouhou antouhou Awaiting requested review from antouhou

@shuplenkov shuplenkov Awaiting requested review from shuplenkov

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
v0.21.x
Development

Successfully merging this pull request may close these issues.
2 participants
@shumkov @jawid-h