From f3dc5c4e275cf85c043228c9fac34783355089e4 Mon Sep 17 00:00:00 2001
From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Date: Tue, 8 Sep 2026 17:13:08 +0100
Subject: [PATCH] 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.
---
 src/tests/libxrpl/telemetry/NodeIdResource.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/tests/libxrpl/telemetry/NodeIdResource.cpp b/src/tests/libxrpl/telemetry/NodeIdResource.cpp
index f0b36c7966..b1e04d83bc 100644
--- a/src/tests/libxrpl/telemetry/NodeIdResource.cpp
+++ b/src/tests/libxrpl/telemetry/NodeIdResource.cpp
@@ -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