Subscribe to rails deprecations

powerhome · Feb 9, 2024 · f3ee03f · f3ee03f
1 parent 1ff3b89
commit f3ee03f
Show file tree

Hide file tree

Showing 9 changed files with 48 additions and 14 deletions.
diff --git a/packages/dep_shield/Gemfile b/packages/dep_shield/Gemfile
@@ -6,9 +6,7 @@ source "https://rubygems.org"
 gemspec
 
 gem "nokogiri", "< 1.16"
-
 gem "rake", "~> 13.0"
-
 gem "rspec", "~> 3.0"
-
 gem "rubocop-powerhome", path: "../rubocop-powerhome"
+gem "rails", "<= 7.0.6"
diff --git a/packages/dep_shield/Gemfile.lock b/packages/dep_shield/Gemfile.lock
@@ -13,7 +13,7 @@ PATH
   specs:
     dep_shield (0.2.1)
       nitro_config
-      rails (>= 6.0.6.1, <= 7.0.6)
+      railties (>= 6.0.6.1, <= 7.0.6)
       sentry-rails (= 5.5.0)
       sentry-ruby (= 5.5.0)
 
@@ -119,7 +119,7 @@ GEM
     mini_mime (1.1.5)
     mini_portile2 (2.8.5)
     minitest (5.21.1)
-    net-imap (0.4.9.1)
+    net-imap (0.4.10)
       date
       net-protocol
     net-pop (0.1.2)
@@ -266,6 +266,7 @@ DEPENDENCIES
   dep_shield!
   license_finder (>= 7.0)
   nokogiri (< 1.16)
+  rails (<= 7.0.6)
   rake (~> 13.0)
   rspec (~> 3.0)
   rspec-rails (~> 5.1.2)

diff --git a/packages/dep_shield/dep_shield.gemspec b/packages/dep_shield/dep_shield.gemspec
@@ -31,15 +31,15 @@ Gem::Specification.new do |spec|
   spec.executables = spec.files.grep(%r{\Aexe/}) { |f| File.basename(f) }
   spec.require_paths = ["lib"]
 
+  spec.add_dependency "nitro_config"
+  spec.add_dependency "railties", ">= 6.0.6.1", "<= 7.0.6"
+  spec.add_dependency "sentry-rails", "5.5.0"
+  spec.add_dependency "sentry-ruby", "5.5.0"
+
   spec.add_development_dependency "appraisal", "~> 2.5.0"
   spec.add_development_dependency "combustion", "~> 1.4"
   spec.add_development_dependency "license_finder", ">= 7.0"
   spec.add_development_dependency "rspec", "~> 3.0"
   spec.add_development_dependency "rspec-rails", "~> 5.1.2"
   spec.add_development_dependency "sqlite3", "~> 1.4.2"
-
-  spec.add_dependency "nitro_config"
-  spec.add_dependency "rails", ">= 6.0.6.1", "<= 7.0.6"
-  spec.add_dependency "sentry-rails", "5.5.0"
-  spec.add_dependency "sentry-ruby", "5.5.0"
 end
diff --git a/packages/dep_shield/gemfiles/rails_6_0.gemfile.lock b/packages/dep_shield/gemfiles/rails_6_0.gemfile.lock
@@ -13,7 +13,7 @@ PATH
   specs:
     dep_shield (0.2.1)
       nitro_config
-      rails (>= 6.0.6.1, <= 7.0.6)
+      railties (>= 6.0.6.1, <= 7.0.6)
       sentry-rails (= 5.5.0)
       sentry-ruby (= 5.5.0)
 

diff --git a/packages/dep_shield/gemfiles/rails_6_1.gemfile.lock b/packages/dep_shield/gemfiles/rails_6_1.gemfile.lock
@@ -13,7 +13,7 @@ PATH
   specs:
     dep_shield (0.2.1)
       nitro_config
-      rails (>= 6.0.6.1, <= 7.0.6)
+      railties (>= 6.0.6.1, <= 7.0.6)
       sentry-rails (= 5.5.0)
       sentry-ruby (= 5.5.0)
 

diff --git a/packages/dep_shield/gemfiles/rails_7_0.gemfile.lock b/packages/dep_shield/gemfiles/rails_7_0.gemfile.lock
@@ -13,7 +13,7 @@ PATH
   specs:
     dep_shield (0.2.1)
       nitro_config
-      rails (>= 6.0.6.1, <= 7.0.6)
+      railties (>= 6.0.6.1, <= 7.0.6)
       sentry-rails (= 5.5.0)
       sentry-ruby (= 5.5.0)
 

diff --git a/packages/dep_shield/lib/dep_shield/railtie.rb b/packages/dep_shield/lib/dep_shield/railtie.rb
@@ -0,0 +1,17 @@
+# frozen_string_literal: true
+
+require "dep_shield"
+
+module DepShield
+  # Ties DepShield to rails deprecations system
+  class Railtie < Rails::Railtie
+    initializer "dep_shield.subscribe" do
+      ActiveSupport::Notifications.subscribe("deprecation.rails") do |name, _start, _finish, _id, payload|
+        message = payload[:message] || "this is deprecated in rails"
+        callstack = payload[:callstack] || caller
+
+        DepShield.raise_or_capture!(name: name, message: message, callstack: callstack)
+      end  
+    end
+  end
+end
diff --git a/packages/dep_shield/spec/dep_shield/railtie_spec.rb b/packages/dep_shield/spec/dep_shield/railtie_spec.rb
@@ -0,0 +1,18 @@
+# frozen_string_literal: true
+
+require "spec_helper"
+
+RSpec.describe DepShield::Deprecation do
+  it "subscribes to deprecation.rails" do
+    expect(DepShield).to(
+      receive(:raise_or_capture!)
+        .with(name: "deprecation.rails", message: "This has been so deprecated", callstack: ["file1.rb", "file2.rb"])
+    )
+
+    ActiveSupport::Notifications.instrument(
+      "deprecation.rails",
+      message: "This has been so deprecated",
+      callstack: ["file1.rb", "file2.rb"]
+      )
+  end
+end
diff --git a/packages/dep_shield/spec/spec_helper.rb b/packages/dep_shield/spec/spec_helper.rb
@@ -1,7 +1,7 @@
 # frozen_string_literal: true
 
 require "bundler"
-require "dep_shield"
+require "dep_shield/railtie"
 require "rspec/rails"
 
 Bundler.require :default, :development