From 7cd21c5699b048e9b8d35320534b31d6ea46543d Mon Sep 17 00:00:00 2001 From: Samuel Williams Date: Sat, 1 Feb 2025 13:29:27 +1300 Subject: [PATCH] Fix warn log test. --- gems.rb | 2 -- lib/metrics/config.rb | 2 +- test/metrics/config.rb | 10 ++-------- 3 files changed, 3 insertions(+), 11 deletions(-) diff --git a/gems.rb b/gems.rb index 61f1148..507d0af 100644 --- a/gems.rb +++ b/gems.rb @@ -21,8 +21,6 @@ gem "decode" gem "rubocop" - gem "sus-fixtures-console" - gem "bake-test" gem "bake-test-external" end diff --git a/lib/metrics/config.rb b/lib/metrics/config.rb index 8ac6d79..d53e8d9 100644 --- a/lib/metrics/config.rb +++ b/lib/metrics/config.rb @@ -36,7 +36,7 @@ def require_backend(env = ENV) if backend = env["METRICS_BACKEND"] begin if require(backend) - Metrics.extend(Backend::Interface) + Metrics.singleton_class.prepend(Backend::Interface) return true end diff --git a/test/metrics/config.rb b/test/metrics/config.rb index 6b3854d..f96d788 100644 --- a/test/metrics/config.rb +++ b/test/metrics/config.rb @@ -6,22 +6,16 @@ require "metrics/config" require "json" -require "sus/fixtures/console" - describe Metrics::Config do let(:config) {subject.default} with ".require_backend" do - include_context Sus::Fixtures::Console::CapturedLogger - it "logs a warning if backend cannot be loaded" do + expect(config).to receive(:warn).and_return(nil) + expect( config.require_backend({"METRICS_BACKEND" => "metrics/backend/missing"}) ).to be == false - - expect_console.to have_logged( - severity: be == :warn, - ) end end end