Remove the legacy history adapter

Closes #3374

Author: mshibuya

.rubocop_todo.yml

@@ -245,7 +245,6 @@ Style/IfUnlessModifier:
     - 'lib/rails_admin/config/fields/types/multiple_active_storage.rb'
     - 'lib/rails_admin/config/inspectable.rb'
     - 'lib/rails_admin/extension.rb'
-    - 'lib/rails_admin/extensions/history/history.rb'
     - 'lib/rails_admin/extensions/pundit/authorization_adapter.rb'
     - 'lib/rails_admin/support/csv_converter.rb'
 

lib/rails_admin.rb

@@ -5,7 +5,6 @@
 require 'rails_admin/extensions/cancancan'
 require 'rails_admin/extensions/pundit'
 require 'rails_admin/extensions/paper_trail'
-require 'rails_admin/extensions/history'
 require 'rails_admin/support/csv_converter'
 require 'rails_admin/support/hash_helper'
 require 'yaml'

lib/rails_admin/config.rb

@@ -110,7 +110,7 @@ def authenticate_with(&blk)
         @authenticate || DEFAULT_AUTHENTICATION
       end
 
-      # Setup auditing/history/versioning provider that observe objects lifecycle
+      # Setup auditing/versioning provider that observe objects lifecycle
       def audit_with(*args, &block)
         extension = args.shift
         if extension

lib/rails_admin/extensions/history.rb

@@ -49,8 +49,6 @@
 
   context 'on destroy' do
     before do
-      RailsAdmin::History.destroy_all
-      RailsAdmin.config { |c| c.audit_with :history }
       @players = Array.new(3) { FactoryBot.create(:player) }
       @delete_ids = @players[0..1].collect(&:id)
 
@@ -63,15 +61,8 @@
       click_button "Yes, I'm sure"
     end
 
-    it 'does not contain deleted records', active_record: true do
-      expect(RailsAdmin::AbstractModel.new('Player').count).to eq(1)
-      expect(RailsAdmin::History.count).to eq(@delete_ids.count)
-      RailsAdmin::History.all.each do |history|
-        expect(history.table).to eq('Player')
-      end
-      RailsAdmin::History.all.each do |history|
-        expect(@delete_ids).to include(history.item)
-      end
+    it 'does not contain deleted records' do
+      expect(RailsAdmin::AbstractModel.new('Player').all.pluck(:id)).to eq([@players[2].id])
       expect(page).to have_selector('.alert-success', text: '2 Players successfully deleted')
     end
   end

lib/rails_admin/extensions/history/auditing_adapter.rb

@@ -1,7 +1,7 @@
 require 'spec_helper'
 require 'paper_trail/frameworks/rspec' if defined?(PaperTrail)
 
-RSpec.describe 'RailsAdmin PaperTrail history', active_record: true do
+RSpec.describe 'RailsAdmin PaperTrail auditing', active_record: true do
   before(:each) do
     RailsAdmin.config do |config|
       config.audit_with :paper_trail, 'User', 'PaperTrail::Version'

lib/rails_admin/extensions/history/history.rb

@@ -43,25 +43,18 @@
 
   context 'on update' do
     it 'is fillable and emptyable', active_record: true do
-      RailsAdmin.config do |c|
-        c.audit_with :history
-      end
-
       @league = FactoryBot.create :league
       @divisions = Array.new(3) { Division.create!(name: "div #{Time.now.to_f}", league: League.create!(name: "league #{Time.now.to_f}")) }
 
       put edit_path(model_name: 'league', id: @league.id, league: {name: 'National League', division_ids: [@divisions[0].id]})
 
-      old_name = @league.name
       @league.reload
       expect(@league.name).to eq('National League')
       @divisions[0].reload
       expect(@league.divisions).to include(@divisions[0])
       expect(@league.divisions).not_to include(@divisions[1])
       expect(@league.divisions).not_to include(@divisions[2])
 
-      expect(RailsAdmin::History.where(item: @league.id).collect(&:message)).to include("name: \"#{old_name}\" -> \"National League\"")
-
       put edit_path(model_name: 'league', id: @league.id, league: {division_ids: ['']})
 
       @league.reload