Skip to content

Commit

Permalink
fix: 17503 Fixed ISS tests of JRS panel
Browse files Browse the repository at this point in the history
Signed-off-by: Ivan Malygin <ivan@swirldslabs.com>
  • Loading branch information
imalygin committed Jan 24, 2025
1 parent ee24004 commit 18099ea
Show file tree
Hide file tree
Showing 3 changed files with 12 additions and 3 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@

import com.hedera.hapi.node.base.SemanticVersion;
import com.swirlds.common.constructable.ConstructableIgnored;
import com.swirlds.common.context.PlatformContext;
import com.swirlds.common.io.SelfSerializable;
import com.swirlds.common.io.streams.SerializableDataInputStream;
import com.swirlds.common.io.streams.SerializableDataOutputStream;
Expand All @@ -44,6 +45,7 @@
import java.io.IOException;
import java.nio.charset.StandardCharsets;
import java.time.Instant;
import java.util.ArrayList;
import java.util.LinkedList;
import java.util.List;
import java.util.function.Function;
Expand Down Expand Up @@ -96,6 +98,11 @@ public ISSTestingToolState(@NonNull final Function<SemanticVersion, SoftwareVers
}

public void initState(InitTrigger trigger, Platform platform) {
throwIfImmutable();

final PlatformContext platformContext = platform.getContext();
super.init(platformContext.getTime(), platformContext.getMetrics(), platformContext.getMerkleCryptography());

// since the test occurrences are relative to the genesis timestamp, the data only needs to be parsed at genesis
if (trigger == InitTrigger.GENESIS) {
final ISSTestingToolConfig testingToolConfig =
Expand Down Expand Up @@ -177,6 +184,10 @@ List<PlannedLogError> getPlannedLogErrorList() {
*/
private ISSTestingToolState(final ISSTestingToolState that) {
super(that);
this.runningSum = that.runningSum;
this.genesisTimestamp = that.genesisTimestamp;
this.plannedIssList = new ArrayList<>(that.plannedIssList);
this.plannedLogErrorList = new ArrayList<>(that.plannedLogErrorList);
}

/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -94,9 +94,6 @@ public void onStateInitialized(
*/
private void handleTransaction(
@NonNull final ISSTestingToolState state, @NonNull final ConsensusTransaction transaction) {
if (transaction.isSystem()) {
return;
}
final int delta =
ByteUtils.byteArrayToInt(transaction.getApplicationTransaction().toByteArray(), 0);
state.incrementRunningSum(delta);
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,7 @@
requires com.swirlds.common;
requires com.swirlds.config.api;
requires com.swirlds.logging;
requires com.swirlds.metrics.api;
requires com.swirlds.platform.core.test.fixtures;
requires com.swirlds.platform.core;
requires com.swirlds.state.api;
Expand Down

0 comments on commit 18099ea

Please sign in to comment.