Isolate sentry logger and std lib logger specs

Author: solnic

…/sentry/breadcrumb/sentry_logger_spec.rb …ruby/spec/isolated/sentry_logger_spec.rbsentry-ruby/spec/sentry/breadcrumb/sentry_logger_spec.rb renamed to sentry-ruby/spec/isolated/sentry_logger_spec.rb sentry-ruby/spec/sentry/breadcrumb/sentry_logger_spec.rb renamed to sentry-ruby/spec/isolated/sentry_logger_spec.rb

@@ -4,6 +4,9 @@
   before do
     perform_basic_setup do |config|
       config.breadcrumbs_logger = [:sentry_logger]
+      config.enable_logs = true
+      config.max_log_events = 1
+      config.enabled_patches = [:logger]
     end
   end
 
@@ -100,4 +103,22 @@
       end
     end
   end
+
+  it "does not conflict with :logger patch" do
+    logger = ::Logger.new(nil)
+
+    logger.info("Hello World")
+
+    expect(sentry_logs).to_not be_empty
+
+    log_event = sentry_logs.last
+
+    expect(log_event[:level]).to eql("info")
+    expect(log_event[:body]).to eql("Hello World")
+
+    breadcrumb = breadcrumbs.peek
+
+    expect(breadcrumb.level).to eq("info")
+    expect(breadcrumb.message).to eq("Hello World")
+  end
 end

…-ruby/spec/sentry/std_lib_logger_spec.rb …uby/spec/isolated/std_lib_logger_spec.rbsentry-ruby/spec/sentry/std_lib_logger_spec.rb renamed to sentry-ruby/spec/isolated/std_lib_logger_spec.rb sentry-ruby/spec/sentry/std_lib_logger_spec.rb renamed to sentry-ruby/spec/isolated/std_lib_logger_spec.rb

@@ -1,7 +1,9 @@
 # frozen_string_literal: true
 
+SimpleCov.command_name "StdLibLogger"
+
 RSpec.describe Sentry::StdLibLogger do
-  let(:logger) { ::Logger.new(IO::NULL) }
+  let(:logger) { ::Logger.new($stdout) }
 
   context "when enable_logs is set to true but logger patch is not enabled" do
     before do
@@ -11,7 +13,9 @@
     end
 
     it "does not send log using stdlib logger" do
-      logger.send(:info, "Hello World")
+      expect {
+        logger.send(:info, "Hello World")
+      }.to output(/Hello World/).to_stdout
 
       expect(sentry_logs).to be_empty
     end
@@ -29,7 +33,9 @@
     ["info", "warn", "error", "fatal"].each do |level|
       describe "##{level}" do
         it "send logs using stdlib logger" do
-          logger.send(level, "Hello World")
+          expect {
+            logger.send(level, "Hello World")
+          }.to output(/Hello World/).to_stdout
 
           expect(sentry_logs).to_not be_empty