Some MerkleDb unit tests take too long to run #10405
Labels
Good First Issue
Either a small change or a medium change concentrated in a single component.
hacktoberfest
Issues shown by lists for the Hacktoberfest and made for newcomers to do the first contribution.
Platform Data Structures
Platform
Tickets pertaining to the platform
Tech Debt Reduced
Issues which reduce technical debt.
Test Development
Test Development
🆕🐥 First Timers Only
This issue is reserved for people who have never contributed to Hedera or any open source project in general.
We know that creating a pull request (PR) is a major barrier for new contributors.
The goal of this issue and all other issues labeled by 'good first issue' is to help you make your first contribution to Hedera.
👾 Description of the issue
I ran a clean build (./gradlew clean; ./gradlew build) locally, it took 12 mins, out of which more than 4 mins were spent in
:swirlds-merkledb:test
:In particular,
HalfDiskVirtualKeySetTest
took a minute or so, and there were a couple more slow tests. Some of the tests are named "hammer test", which indicates they should be tagged appropriately, but they aren't:📋 Step by step guide to do a contribution
If you have never contributed to an open source project at GitHub, the following step-by-step guide will introduce you to the workflow. More information and concrete samples for shell commands for each step can be found in our CONTRIBUTING.md file.
A more detailed general documentation of the GitHub PR workflow can be found here.
sign-off
information to the commit to accept the "Developer Certificate of Origin" (https://developercertificate.org). More details can be found in our CONTRIBUTING.md🎉 Contribute to Hacktoberfest
Solve this issue as part of the Hacktoberfest event and get a chance to receive cool goodies like a T-Shirt. 🎽
🤔 Additional Information
If you have any questions, just ask us directly in this issue by adding a comment. You can join our community chat at Discord. A general manual about open-source contributions can be found here.
The text was updated successfully, but these errors were encountered: