Fix NoMethodError when calling anonymise_for! with no anonymise config

CHANGELOG.md

@@ -4,7 +4,7 @@
 - Drop support for EOL Rails 6.1 (EOL since October 1st 2024)
 - Add support for Rails 7.2
 - Fix Dependabot updates
-- Fix `NoMethodError` when calling `selector_for?` on a model class without an `anonymise` config block
+- Fix `NoMethodError` when calling `selector_for?` or `anonymise_for!` on a model class without an `anonymise` config block
 
 # v1.4.0
 

lib/anony/anonymisable.rb

@@ -58,6 +58,10 @@ def valid_anonymisation?
       # Finds the records that relate to a particular subject and runs anonymise on
       # each of them. If a selector is not defined it will raise an exception.
       def anonymise_for!(subject, subject_id)
+        unless anonymise_config
+          raise ArgumentError, "#{name} does not have an Anony configuration"
+        end
+
         records = anonymise_config.
           select(subject, subject_id)
         records.map do |record|

spec/anony/anonymisable_spec.rb

@@ -253,19 +253,45 @@ def some_instance_method?
   end
 
   context "no anonymise block" do
-    describe "#valid_anonymisation?" do
-      let(:klass) do
-        Class.new(ActiveRecord::Base) do
-          include Anony::Anonymisable
+    let(:klass) do
+      Class.new(ActiveRecord::Base) do
+        include Anony::Anonymisable
 
-          self.table_name = :employees
+        def self.name
+          "Employee"
         end
+
+        self.table_name = :employees
       end
+    end
 
+    describe "#valid_anonymisation?" do
       it "fails" do
         expect(klass).to_not be_valid_anonymisation
       end
     end
+
+    describe "#selector_for?" do
+      it "does not throw an exception" do
+        expect { klass.selector_for?(:foo) }.to_not raise_error
+      end
+
+      it "returns false" do
+        expect(klass.selector_for?(:foo)).to be false
+      end
+    end
+
+    describe "#anonymise_for!" do
+      let(:model) do
+        klass.create!(first_name: "abc", last_name: "foo", company_name: "alpha")
+      end
+
+      it "throws an exception" do
+        expect { klass.anonymise_for!(:first_name, "abc") }.to raise_error(
+          ArgumentError, "Employee does not have an Anony configuration"
+        )
+      end
+    end
   end
 
   context "with an empty anonymise block" do