Remove deprecated: paper_trail_on, paper_trail_off

CHANGELOG.md

@@ -2,6 +2,8 @@
 
 ### Changed
 
+- Removed deprecated methods `paper_trail_on` and `paper_trail_off`.
+  Use `paper_trail_on!` and `paper_trail_off!` instead.
 - Using a Rails initializer to reopen PaperTrail::Version or otherwise extend
   PaperTrail is no longer recommended. An alternative is described in the
   readme. See https://github.com/airblade/paper_trail/pull/557 and

lib/paper_trail/has_paper_trail.rb

@@ -96,21 +96,11 @@ def paper_trail_off!
         PaperTrail.enabled_for_model(self, false)
       end
 
-      def paper_trail_off
-        warn "DEPRECATED: use `paper_trail_off!` instead of `paper_trail_off`. Support for `paper_trail_off` will be removed in PaperTrail 4.0"
-        self.paper_trail_off!
-      end
-
       # Switches PaperTrail on for this class.
       def paper_trail_on!
         PaperTrail.enabled_for_model(self, true)
       end
 
-      def paper_trail_on
-        warn "DEPRECATED: use `paper_trail_on!` instead of `paper_trail_on`. Support for `paper_trail_on` will be removed in PaperTrail 4.0"
-        self.paper_trail_on!
-      end
-
       def paper_trail_enabled_for_model?
         return false unless self.include?(PaperTrail::Model::InstanceMethods)
         PaperTrail.enabled_for_model?(self)

spec/models/widget_spec.rb

@@ -282,21 +282,6 @@
         end
       end
 
-      describe '#paper_trail_off' do
-        it { is_expected.to respond_to(:paper_trail_off) }
-
-        it 'should set the invoke `paper_trail_off!`' do
-          is_expected.to receive(:warn)
-          is_expected.to receive(:paper_trail_off!)
-          subject.paper_trail_off
-        end
-
-        it 'should display a deprecation warning' do
-          is_expected.to receive(:warn).with("DEPRECATED: use `paper_trail_on!` instead of `paper_trail_on`. Support for `paper_trail_on` will be removed in PaperTrail 4.0")
-          subject.paper_trail_on
-        end
-      end
-
       describe '#paper_trail_on!' do
         before { subject.paper_trail_off! }
 
@@ -308,23 +293,6 @@
           expect(subject.paper_trail_enabled_for_model?).to be true
         end
       end
-
-      describe '#paper_trail_on' do
-        before { subject.paper_trail_off! }
-
-        it { is_expected.to respond_to(:paper_trail_on) }
-
-        it 'should set the invoke `paper_trail_on!`' do
-          is_expected.to receive(:warn)
-          is_expected.to receive(:paper_trail_on!)
-          subject.paper_trail_on
-        end
-
-        it 'should display a deprecation warning' do
-          is_expected.to receive(:warn).with("DEPRECATED: use `paper_trail_on!` instead of `paper_trail_on`. Support for `paper_trail_on` will be removed in PaperTrail 4.0")
-          subject.paper_trail_on
-        end
-      end
     end
   end
 end