fix(test): compare the trace strategy against its enumerator

getConsensusTraceStrategy() returns ConsensusTraceStrategy, so comparing
it to a string literal does not compile.

No coverage is lost: strategyName()'s spelling has its own assertions in
the TelemetryConfig test, in this same binary.
This commit is contained in:
Pratik Mankawde
2026-09-08 17:13:08 +01:00
parent 5542170422
commit f3dc5c4e27

View File

@@ -107,7 +107,7 @@ TEST(NodeIdResource, set_node_id_on_disabled_path_is_inert)
EXPECT_FALSE(telemetry->shouldTraceConsensus());
EXPECT_FALSE(telemetry->shouldTracePeer());
EXPECT_FALSE(telemetry->shouldTraceLedger());
EXPECT_EQ(telemetry->getConsensusTraceStrategy(), "deterministic");
EXPECT_EQ(telemetry->getConsensusTraceStrategy(), ConsensusTraceStrategy::Deterministic);
}
#ifdef XRPL_ENABLE_TELEMETRY