From 545b35d44090440355ed52085cab95c51bc49fa4 Mon Sep 17 00:00:00 2001 From: Pratik Mankawde <3397372+pratikmankawde@users.noreply.github.com> Date: Mon, 20 Jul 2026 15:26:33 +0100 Subject: [PATCH] fix(telemetry): add getMeter override to SpanGuardScope test mock getMeter() became a pure virtual on this branch (metric gap-fill), which made the TestTelemetry mock abstract and broke the scope-leak test build. Mirror NullTelemetry: serve an inert meter from a process-wide noop provider. The scope tests exercise only tracing, so the meter is unused. Co-Authored-By: Claude Opus 4.8 (1M context) --- src/tests/libxrpl/telemetry/SpanGuardScope.cpp | 16 ++++++++++++++++ 1 file changed, 16 insertions(+) diff --git a/src/tests/libxrpl/telemetry/SpanGuardScope.cpp b/src/tests/libxrpl/telemetry/SpanGuardScope.cpp index 8aac770262..8b589eb461 100644 --- a/src/tests/libxrpl/telemetry/SpanGuardScope.cpp +++ b/src/tests/libxrpl/telemetry/SpanGuardScope.cpp @@ -19,6 +19,9 @@ #include #include #include +#include +#include +#include #include #include #include @@ -141,6 +144,19 @@ public: return provider_->GetTracer(std::string(name)); } + /** + * @return A meter from a noop provider; the scope tests exercise only + * tracing, so metrics instruments are inert. + */ + opentelemetry::nostd::shared_ptr + getMeter(std::string_view name) override + { + static auto noopProvider = + opentelemetry::nostd::shared_ptr( + new opentelemetry::metrics::NoopMeterProvider()); + return noopProvider->GetMeter(std::string(name), std::string(kMeterVersion)); + } + opentelemetry::nostd::shared_ptr startSpan(std::string_view name, opentelemetry::trace::SpanKind kind) override {