Allow configuration of minitest file paths and file names

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ## main (unreleased)
 
+* [#95](https://github.com/Shopify/deprecation_toolkit/pull/95): Allow configuration of deprecation file paths and file names.
+
 ## 2.0.4 (2023-11-20)
 
 * [#90](https://github.com/Shopify/deprecation_toolkit/pull/90) & [#93](https://github.com/Shopify/deprecation_toolkit/pull/93): Stop using deprecated behavior from Active Support. (@etiennebarrie)

README.md

@@ -117,6 +117,31 @@ This setting accepts an array of regular expressions. To match on all warnings,
 DeprecationToolkit::Configuration.warnings_treated_as_deprecation = [//]
 ```
 
+### 🔨 `#DeprecationToolkit::Configuration#deprecation_file_path_format`
+
+DeprecationToolkit allows you to choose the file path format for deprecation files.
+
+For Minitest, it defaults to using class name so the following code would correspond to `#{deprecation_path}/deprecation_toolkit/behaviors/raise_test.yml`
+
+```ruby
+module DeprecationToolkit
+  module Behaviors
+    class RaiseTest < ActiveSupport::TestCase
+    end
+  end
+end
+```
+
+For rspec if defaults to the file location with spec removed. `/spec/models/user_spec.rb` would correspond to `/models/user.yml`.
+
+If you have a specific use case you can configure this with a custom format using a proc. The proc is called with an instance of the test.
+
+```ruby
+Configuration.deprecation_file_path_format = -> (test) do
+  Kernel.const_source_location(test.class.name)[0].sub(%r{^./test/}, "").sub(/_test.rb$/, "")
+end
+```
+
 ## RSpec
 
 By default Deprecation Toolkit uses Minitest as its test runner. To use Deprecation Toolkit with RSpec you'll have to configure it.

lib/deprecation_toolkit/configuration.rb

@@ -12,5 +12,14 @@ class Configuration
     config_accessor(:deprecation_path) { "test/deprecations" }
     config_accessor(:test_runner) { :minitest }
     config_accessor(:warnings_treated_as_deprecation) { [] }
+    config_accessor(:deprecation_file_path_format) do
+      proc do |test|
+        if DeprecationToolkit::Configuration.test_runner == :rspec
+          test.example_group.file_path.sub(%r{^./spec/}, "").sub(/_spec.rb$/, "")
+        else
+          test.class.name.underscore
+        end
+      end
+    end
   end
 end

lib/deprecation_toolkit/read_write_helper.rb

@@ -48,12 +48,7 @@ def recorded_deprecations_path(test)
         Configuration.deprecation_path
       end
 
-      path =
-        if DeprecationToolkit::Configuration.test_runner == :rspec
-          test.example_group.file_path.sub(%r{^./spec/}, "").sub(/_spec.rb$/, "")
-        else
-          test.class.name.underscore
-        end
+      path = Configuration.deprecation_file_path_format.call(test)
 
       Pathname(deprecation_folder).join("#{path}.yml")
     end

test/deprecation_toolkit/behaviors/record_test.rb

@@ -31,6 +31,27 @@ class RecordTest < ActiveSupport::TestCase
         end
       end
 
+      test ".trigger record deprecations records using file path when configuration is set to use file path" do
+        Configuration.deprecation_file_path_format = ->(test) do
+          Kernel.const_source_location(test.class.name)[0].sub(%r{^.*/(test/)}, '\1').sub(/\.[^.]+$/, "")
+        end
+
+        assert_deprecations_recorded("Foo", "Bar", path: "test/deprecation_toolkit/behaviors/record_test.yml") do
+          deprecator.warn("Foo")
+          deprecator.warn("Bar")
+
+          trigger_deprecation_toolkit_behavior
+        end
+      ensure
+        Configuration.deprecation_file_path_format = proc do |test|
+          if DeprecationToolkit::Configuration.test_runner == :rspec
+            test.example_group.file_path.sub(%r{^./spec/}, "").sub(/_spec.rb$/, "")
+          else
+            test.class.name.underscore
+          end
+        end
+      end
+
       test ".trigger record deprecations for proc path" do
         Configuration.deprecation_path = proc do
           File.join(@deprecation_path, "prefix")
@@ -106,16 +127,20 @@ def example; end
 
       private
 
-      def assert_deprecations_recorded(*deprecation_triggered, to: @deprecation_path)
+      def assert_deprecations_recorded(
+        *deprecation_triggered,
+        to: @deprecation_path,
+        path: "deprecation_toolkit/behaviors/record_test.yml"
+      )
         yield
 
         triggered = deprecation_triggered.map { |msg| "DEPRECATION WARNING: #{msg}" }
 
-        assert_equal(recorded_deprecations(to: to), triggered)
+        assert_equal(recorded_deprecations(to: to, path: path), triggered)
       end
 
-      def recorded_deprecations(to: @deprecation_path)
-        YAML.load_file("#{to}/deprecation_toolkit/behaviors/record_test.yml").fetch(name)
+      def recorded_deprecations(to: @deprecation_path, path: "deprecation_toolkit/behaviors/record_test.yml")
+        YAML.load_file("#{to}/#{path}").fetch(name)
       end
     end
   end