From 0212c8dc4750e8cee4dc0592c8d9c882e570f057 Mon Sep 17 00:00:00 2001
From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com>
Date: Wed, 23 Sep 2026 19:04:51 +0100
Subject: [PATCH] style: Rename gtest test cases to snake_case

develop's new fix-gtest-names pre-commit hook requires a snake_case test-case
name. CI runs it with --all-files, so the existing CamelCase names fail the
check on every branch that carries these files.
---
 src/tests/libxrpl/telemetry/SpanGuardScope.cpp | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/src/tests/libxrpl/telemetry/SpanGuardScope.cpp b/src/tests/libxrpl/telemetry/SpanGuardScope.cpp
index 69329c3f79..c3ea539e00 100644
--- a/src/tests/libxrpl/telemetry/SpanGuardScope.cpp
+++ b/src/tests/libxrpl/telemetry/SpanGuardScope.cpp
@@ -606,7 +606,7 @@ TEST_F(SpanGuardScopeTest, activate_sets_ambient_without_owning)
 // OTel key-value-iterable, so a dropped or mistyped pair would be invisible
 // without reading the exported event back. Values are asserted individually as
 // well as by count: two attributes with one value blanked still counts as two.
-TEST_F(SpanGuardScopeTest, spanGuard_addEvent_records_name_and_attribute_values)
+TEST_F(SpanGuardScopeTest, span_guard_add_event_records_name_and_attribute_values)
 {
     namespace cs = consensus::span;
 
@@ -640,7 +640,7 @@ TEST_F(SpanGuardScopeTest, spanGuard_addEvent_records_name_and_attribute_values)
 // The name-only overload records the event with NO attributes, so a regression
 // that leaked attributes between the two overloads shows up here rather than as
 // an extra key on a production event.
-TEST_F(SpanGuardScopeTest, spanGuard_addEvent_without_attributes_records_bare_event)
+TEST_F(SpanGuardScopeTest, span_guard_add_event_without_attributes_records_bare_event)
 {
     namespace cs = consensus::span;
 
@@ -664,7 +664,7 @@ TEST_F(SpanGuardScopeTest, spanGuard_addEvent_without_attributes_records_bare_ev
 
 // The scoped guard records event attributes too. consensus.accept.apply relies
 // on it for one tx.included event per transaction of the accepted set.
-TEST_F(SpanGuardScopeTest, scopedGuard_addEvent_records_name_and_attribute_values)
+TEST_F(SpanGuardScopeTest, scoped_guard_add_event_records_name_and_attribute_values)
 {
     namespace cs = consensus::span;
 
@@ -692,7 +692,7 @@ TEST_F(SpanGuardScopeTest, scopedGuard_addEvent_records_name_and_attribute_value
 // A scoped child of a captured context is the ambient parent of the spans
 // created after it on the same thread. A hash-derived root created inside that
 // scope stays a root. consensus.accept.apply relies on both.
-TEST_F(SpanGuardScopeTest, scopedChildOfCapturedContextIsAmbientForLaterSpans)
+TEST_F(SpanGuardScopeTest, scoped_child_of_captured_context_is_ambient_for_later_spans)
 {
     namespace cs = consensus::span;