Skip to content

Commit

Permalink
fix(iota, iota-replay): fix test_profiler test by unignoring it and p…
Browse files Browse the repository at this point in the history
…ass valid tx_digest (#4396)

* fix(iota, iota-replay): fix test_profiler test by unignoring it and pass valid tx_digest
  • Loading branch information
Dkwcs authored Dec 10, 2024
1 parent 6dab9e5 commit e51c090
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 14 deletions.
13 changes: 1 addition & 12 deletions crates/iota-replay/src/replay.rs
Original file line number Diff line number Diff line change
Expand Up @@ -727,18 +727,7 @@ impl LocalExec {
reason: "System transaction".to_string(),
});
}
// Before protocol version 16, the generation of effects depends on the wrapped
// tombstones. It is not possible to retrieve such data for replay.
if tx_info.protocol_version.as_u64() < 16 {
warn!(
"Protocol version ({:?}) too old: {}, skipping transaction",
tx_info.protocol_version, tx_digest
);
return Err(ReplayEngineError::TransactionNotSupported {
digest: *tx_digest,
reason: "Protocol version too old".to_string(),
});
}

// Initialize the state necessary for execution
// Get the input objects
let input_objects = self.initialize_execution_env_state(tx_info).await?;
Expand Down
3 changes: 1 addition & 2 deletions crates/iota/src/unit_tests/profiler_tests.rs
Original file line number Diff line number Diff line change
Expand Up @@ -36,7 +36,6 @@ fn test_macro_shows_feature_enabled() {
}
}

#[ignore]
#[cfg(feature = "gas-profiler")]
#[tokio::test(flavor = "multi_thread")]
async fn test_profiler() {
Expand All @@ -49,7 +48,7 @@ async fn test_profiler() {
let profile_output = output_dir.path().join("profile.json");

let testnet_url = "https://api.testnet.iota.cafe".to_string();
let tx_digest = "98KxVD14f2JgceKx4X27HaVAA2YGJ3Aazf6Y4tabpHa8".to_string();
let tx_digest = "7qq4W43TqHg9tQPMvdAFW4Tz6J88KnPppBPR1hNKmQAd".to_string();

let cmd = ReplayToolCommand::ProfileTransaction {
tx_digest,
Expand Down

0 comments on commit e51c090

Please sign in to comment.