Fix coverage by making sure isolated specs have SimpleCov.command_name

Author: sl0thentr0py

sentry-rails/Rakefile

@@ -26,7 +26,7 @@ namespace :spec do
 end
 
 task :isolated_specs do
-  Dir["spec/isolated/*"].each do |file|
+  Dir["spec/isolated/**/*_spec.rb"].each do |file|
     sh "bundle exec ruby #{file}"
   end
 end

sentry-rails/spec/isolated/active_job_activation.rb

@@ -13,6 +13,9 @@
 require "sentry/rails"
 require "minitest/autorun"
 
+# isolated tests need a SimpleCov name otherwise they will overwrite coverage
+SimpleCov.command_name "RailsIsolatedActiveJobActivation"
+
 class TestApp < Rails::Application
 end
 

sentry-ruby/Rakefile

@@ -14,7 +14,7 @@ RSpec::Core::RakeTask.new(:spec).tap do |task|
 end
 
 task :isolated_specs do
-  Dir["spec/isolated/*"].each do |file|
+  Dir["spec/isolated/**/*_spec.rb"].each do |file|
     sh "bundle exec rspec #{file}"
   end
 end

sentry-ruby/spec/isolated/init.rb

@@ -1,6 +1,7 @@
 # frozen_string_literal: true
 
-require_relative "../spec_helper"
+require "sentry-ruby"
+require "sentry/test_helper"
 
 Sentry.init do |config|
   config.dsn = Sentry::TestHelper::DUMMY_DSN

sentry-ruby/spec/isolated/init_spec.rb

@@ -2,7 +2,8 @@
 
 require_relative "../spec_helper"
 
-SimpleCov.command_name "RSpecIsolated"
+# isolated tests need a SimpleCov name otherwise they will overwrite coverage
+SimpleCov.command_name "RSpecIsolatedInit"
 
 RSpec.describe Sentry do
   context "works within a trap context", when: { ruby_engine?: "ruby" } do

sentry-ruby/spec/isolated/puma_spec.rb

@@ -6,7 +6,8 @@
 # Because puma doesn't have any dependency, if Rack is not installed the entire test won't work
 return if ENV["RACK_VERSION"] == "0"
 
-SimpleCov.command_name "RSpecIsolated"
+# isolated tests need a SimpleCov name otherwise they will overwrite coverage
+SimpleCov.command_name "RSpecIsolatedPuma"
 
 RSpec.describe Puma::Server do
   class TestServer