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.

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

refactor: enhance consistency testing tool #17136

Merged
merged 19 commits into from
Jan 13, 2025
Merged
Changes from 1 commit
Commits
Show all changes
19 commits
Select commit Hold shift + click to select a range
97eac92
refactor: enhance consistency testing tool
mustafauzunn Dec 19, 2024
63edb17
refactor: consumer for system transactions
mustafauzunn Dec 19, 2024
f4caa08
Merge branch '16703-refactor-consumer-system-transactions' of github.…
mustafauzunn Dec 20, 2024
8478fd9
Merge branch 'develop' into 16703-refactor-consumer-system-transactions
mustafauzunn Dec 20, 2024
072a689
refactor: after merge with consumer refactor branch
mustafauzunn Dec 20, 2024
84a256f
Merge branch '16703-refactor-consumer-system-transactions' of github.…
mustafauzunn Dec 20, 2024
5697b4a
docs: add javadoc
mustafauzunn Dec 20, 2024
7643692
Merge branch 'develop' of github.com:hashgraph/hedera-services into 1…
mustafauzunn Dec 20, 2024
312f184
refactor: consistency testing tool
mustafauzunn Dec 20, 2024
7a1f1aa
test: add unit tests for the new logic in handle and prehandle
mustafauzunn Jan 6, 2025
9da0556
Merge branch 'main' of github.com:hashgraph/hedera-services into 1679…
mustafauzunn Jan 6, 2025
c1f7e96
refactor: address PR comments
mustafauzunn Jan 7, 2025
c8f4bbf
refactor: unit tests
mustafauzunn Jan 7, 2025
fa9cd85
Merge branch 'main' of github.com:hashgraph/hedera-services into 1679…
mustafauzunn Jan 7, 2025
f39d6ff
refactor: unit test import
mustafauzunn Jan 7, 2025
c2af1a6
refactor: implementation
mustafauzunn Jan 7, 2025
4f73ef8
refactor: implementation
mustafauzunn Jan 10, 2025
203da89
Merge branch 'main' of github.com:hashgraph/hedera-services into 1679…
mustafauzunn Jan 10, 2025
ee1eaab
refactor: PR comments
mustafauzunn Jan 10, 2025
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
refactor: consistency testing tool
Signed-off-by: Mustafa Uzun <mustafa.uzun@limechain.tech>
  • Loading branch information
mustafauzunn committed Dec 20, 2024
commit 312f184c85e36a3927e767c9464364248cdf36b3
Original file line number Diff line number Diff line change
Expand Up @@ -237,8 +237,9 @@
}

if (isSystemTransaction(transaction)) {
scopedSystemTransaction.accept(new ScopedSystemTransaction(
consensusEvent.getCreatorId(), consensusEvent.getSoftwareVersion(), transaction));
return;

Check warning on line 242 in platform-sdk/platform-apps/tests/ConsistencyTestingTool/src/main/java/com/swirlds/demo/consistency/ConsistencyTestingToolState.java

View check run for this annotation

Codecov / codecov/patch

platform-sdk/platform-apps/tests/ConsistencyTestingTool/src/main/java/com/swirlds/demo/consistency/ConsistencyTestingToolState.java#L240-L242

Added lines #L240 - L242 were not covered by tests
}

final long transactionContents =
Expand All @@ -264,8 +265,9 @@
}

if (isSystemTransaction(transaction)) {
stateSignatureTransaction.accept(
new ScopedSystemTransaction(event.getCreatorId(), event.getSoftwareVersion(), transaction));
return;

Check warning on line 270 in platform-sdk/platform-apps/tests/ConsistencyTestingTool/src/main/java/com/swirlds/demo/consistency/ConsistencyTestingToolState.java

View check run for this annotation

Codecov / codecov/patch

platform-sdk/platform-apps/tests/ConsistencyTestingTool/src/main/java/com/swirlds/demo/consistency/ConsistencyTestingToolState.java#L268-L270

Added lines #L268 - L270 were not covered by tests
}
final long transactionContents =
byteArrayToLong(transaction.getApplicationTransaction().toByteArray(), 0);
Expand Down Expand Up @@ -301,9 +303,9 @@

roundsHandled++;

round.forEachEventTransaction((ev, tx) -> {
applyTransactionToState(ev, tx, stateSignatureTransaction);
IvanKavaldzhiev marked this conversation as resolved.
Show resolved Hide resolved
});

Check warning on line 308 in platform-sdk/platform-apps/tests/ConsistencyTestingTool/src/main/java/com/swirlds/demo/consistency/ConsistencyTestingToolState.java

View check run for this annotation

Codecov / codecov/patch

platform-sdk/platform-apps/tests/ConsistencyTestingTool/src/main/java/com/swirlds/demo/consistency/ConsistencyTestingToolState.java#L306-L308

Added lines #L306 - L308 were not covered by tests
stateLong = NonCryptographicHashing.hash64(stateLong, round.getRoundNum());

transactionHandlingHistory.processRound(ConsistencyTestingToolRound.fromRound(round, stateLong));
Expand Down
Loading