paper_trail_viewer 1.0.0 → 1.1.1

data/paper_trail_viewer.gemspec

@@ -13,22 +13,13 @@ Gem::Specification.new do |spec|
   spec.license       = 'MIT'
 
   spec.files         = `git ls-files -z`.split("\x0") + %w[javascript/compiled.js]
-  spec.executables   = spec.files.grep(%r{^bin/}) { |f| File.basename(f) }
   spec.test_files    = spec.files.grep(%r{^(test|spec|features)/})
   spec.require_paths = ['lib']
 
-  spec.required_ruby_version = '>= 2.7'
+  spec.required_ruby_version = '>= 3.0'
 
   spec.add_dependency 'kaminari'
+  spec.add_dependency 'ostruct'
   spec.add_dependency 'paper_trail', ['>= 11.1']
-  spec.add_dependency 'rails', ['>= 6.0', '< 8.0']
-
-  spec.add_development_dependency 'appraisal', '~> 1.0'
-  spec.add_development_dependency 'capybara', '~> 3.0'
-  spec.add_development_dependency 'factory_bot_rails', '~> 6.0'
-  spec.add_development_dependency 'puma', '~> 5.0'
-  spec.add_development_dependency 'rake', '~> 13.0'
-  spec.add_development_dependency 'rspec-rails', '~> 5.0'
-  spec.add_development_dependency 'sqlite3', '>= 1.3.6'
-  spec.add_development_dependency 'webdrivers', '~> 5.0'
+  spec.add_dependency 'rails', ['>= 6.0', '< 9.0']
 end

data/spec/app_template.rb

@@ -1,10 +1,12 @@
-gem 'paper_trail_viewer', path: __FILE__ + '/../../../'
+gem 'paper_trail_viewer', path: __dir__ + '/../'
 
 generate 'paper_trail:install'
 generate 'resource', 'entity name:string status:string --no-test-framework'
 generate 'resource', 'platform name:string status:string --no-test-framework'
 
 model_body = <<-RUBY
+  ::ActiveRecord.try(:use_yaml_unsafe_load=, true)
+
   has_paper_trail
 
   validates_presence_of :name

data/spec/rails_helper.rb

@@ -2,8 +2,8 @@ ENV['RAILS_ENV'] ||= 'test'
 require File.expand_path('dummy/config/environment', __dir__)
 require 'rspec/rails'
 
+require 'capybara/rspec'
 require 'selenium/webdriver'
-require 'webdrivers/chromedriver'
 
 Capybara.server = :puma, { Silent: true }
 
@@ -13,6 +13,14 @@ RSpec.configure do |config|
   config.use_transactional_fixtures = true
   config.infer_spec_type_from_file_location!
   config.before(:each, type: :system) do
-    driven_by :selenium_chrome_headless
+    driven_by :selenium, using: :headless_chrome, screen_size: [1600, 900] do |opts|
+      # https://github.com/SeleniumHQ/selenium/issues/14453
+      opts.add_argument('--disable-search-engine-choice-screen')
+      # various fixes for GH actions
+      opts.add_argument('--disable-dev-shm-usage')
+      opts.add_argument('--disable-extensions')
+      opts.add_argument('--disable-site-isolation-trials')
+      opts.add_argument('--remote-debugging-pipe')
+    end
   end
 end

data/spec/system/paper_trail_viewer_spec.rb

@@ -14,101 +14,116 @@ describe PaperTrailViewer, versioning: true do
   end
 
   context 'with changes' do
-    let(:reimu) { FactoryBot.create(:entity, name: 'Miko Hakurei Reimu', status: 'Highly Responsive to Prayers') }
-    let(:flanchan) { FactoryBot.create(:entity, name: 'Flandre Scarlet', status: 'The Embodiment of Scarlet Devil') }
-    let(:sakuya) { FactoryBot.create(:entity, name: 'Sakuya Izayoi', status: 'Flowering Night') }
-    let(:kyuu_hachi) { FactoryBot.create(:platform, name: 'PC-9801', status: 'SUGOI!!1!') }
+    let!(:reimu) { FactoryBot.create(:entity, name: 'Miko Hakurei Reimu', status: 'Highly Responsive to Prayers') }
+    let!(:flanchan) { FactoryBot.create(:entity, name: 'Flandre Scarlet', status: 'The Embodiment of Scarlet Devil') }
+    let!(:sakuya) { FactoryBot.create(:entity, name: 'Sakuya Izayoi', status: 'Flowering Night') }
+    let!(:kyuu_hachi) { FactoryBot.create(:platform, name: 'PC-9801', status: 'SUGOI!!1!') }
     let!(:uinodouzu) { FactoryBot.create(:platform, name: 'Mikorusofto Uinodouzu', status: 'o-O') }
 
-    let!(:flanchan_id) { flanchan.id }
-
     before do
-      sakuya
-      reimu.update(name: 'Hakurei Reimu', status: 'Phantasmagoria of Dimensional Dream')
-      reimu.update(status: 'Perfect Cherry Blossom')
-      flanchan.destroy
-      kyuu_hachi.update(status: 'Kimochi warui.')
-      kyuu_hachi.destroy
-      uinodouzu
+      reimu.update!(name: 'Hakurei Reimu', status: 'Phantasmagoria of Dimensional Dream')
+      reimu.update!(status: 'Perfect Cherry Blossom')
+      flanchan.destroy!
+      kyuu_hachi.update!(status: 'Kimochi warui.')
+      kyuu_hachi.destroy!
     end
 
     describe 'index' do
-      context 'when getting all changes' do
-        context 'with authorization' do
-          context 'when getting default index' do
-            before { visit engine_urls.root_path }
-
-            it 'has all changes' do
-              expect(page).to have_selector('[data-ci-type="version-row"]', count: 10)
-            end
-
-            it 'has changes for all changed item types' do
-              within('table') do
-                expect(page).to have_content('Entity')
-                expect(page).to have_content('Platform')
-              end
-            end
-
-            it 'orders changes with newest and highest id at the top' do
-              ids = PaperTrail::Version.pluck(:id)
-              expect(page).to have_content(/#{ids.sort.reverse.join('.+')}/m)
-            end
-          end
+      before { visit engine_urls.root_path }
+
+      it 'has all changes' do
+        expect(page).to have_selector('[data-ci-type="version-row"]', count: 10)
+      end
+
+      it 'has changes for all changed item types' do
+        within('table') do
+          expect(page).to have_content('Entity')
+          expect(page).to have_content('Platform')
         end
       end
 
+      it 'orders changes with newest and highest id at the top' do
+        ids = PaperTrail::Version.pluck(:id)
+        expect(page).to have_content(/#{ids.sort.reverse.join('.+')}/m)
+      end
+
       context 'when getting changes for a specific type' do
-        context 'when changes exist' do
-          before { visit engine_urls.root_path(item_type: 'Entity') }
+        before { visit engine_urls.root_path(item_type: 'Entity') }
 
-          it 'shows a subset of the changes' do
-            expect(page).to have_selector('[data-ci-type="version-row"]', count: 6)
-          end
+        it 'shows a subset of the changes' do
+          expect(page).to have_selector('[data-ci-type="version-row"]', count: 6)
         end
       end
 
       context 'when getting changes for a specific record' do
-        context 'when changes exist' do
-          before { visit engine_urls.root_path(item_type: 'Entity', item_id: reimu.id) }
+        before { visit engine_urls.root_path(item_type: 'Entity', item_id: reimu.id) }
 
-          it 'shows a subset of the changes' do
-            expect(page).to have_selector('[data-ci-type="version-row"]', count: 3)
-          end
+        it 'shows a subset of the changes' do
+          expect(page).to have_selector('[data-ci-type="version-row"]', count: 3)
         end
       end
     end
 
+    describe 'tracking an item' do
+      it 'works' do
+        visit engine_urls.root_path
+        expect(page).to have_selector('[data-ci-type="version-row"]', count: 10)
+
+        open_reimu_actions_menu
+        click_on 'Track item'
+
+        expect(page).to have_selector('[data-ci-type="version-row"]', count: 3)
+      end
+    end
+
     describe 'rolling back changes' do
-      context 'when changes exist' do
-        context 'when authorized' do
-          it 'rollbacks a newly-created record by deleting it' do
-            expect(Entity).to exist(reimu.id)
+      it 'can be done via the UI' do
+        visit engine_urls.root_path(item_type: 'Entity', item_id: reimu.id)
 
-            put engine_urls.version_path(reimu.versions.first)
+        open_reimu_actions_menu
 
-            expect(Entity).not_to exist(reimu.id)
+        accept_confirm do # confirm that we really want to roll back
+          accept_alert do # accept alert saying that rollback happened
+            find('span', text: 'Roll back').click
           end
+        end
 
-          it 'rollbacks an edit by reverting to the previous state' do
-            reimu.reload
-            expect(reimu.status).to eq 'Perfect Cherry Blossom'
+        expect(Entity.exists?(reimu.id)).to eq false
+      end
 
-            put engine_urls.version_path(reimu.versions.last)
+      it 'rolls back a newly-created record by deleting it' do
+        expect(Entity).to exist(reimu.id)
 
-            reimu.reload
-            expect(reimu.status).to eq 'Phantasmagoria of Dimensional Dream'
-          end
+        put engine_urls.version_path(reimu.versions.first)
 
-          it 'rollbacks a delete by restoring the record' do
-            expect(Entity.exists?(flanchan_id)).to eq false
+        expect(Entity).not_to exist(reimu.id)
+      end
 
-            put engine_urls.version_path(PaperTrail::Version.where(item_id: flanchan_id, item_type: 'Entity').last)
+      it 'rolls back an edit by reverting to the previous state' do
+        reimu.reload
+        expect(reimu.status).to eq 'Perfect Cherry Blossom'
 
-            found = Entity.find(flanchan_id)
-            expect(found.status).to eq 'The Embodiment of Scarlet Devil'
-          end
-        end
+        put engine_urls.version_path(reimu.versions.last)
+
+        reimu.reload
+        expect(reimu.status).to eq 'Phantasmagoria of Dimensional Dream'
       end
+
+      it 'rolls back a delete by restoring the record' do
+        expect(Entity.exists?(flanchan.id)).to eq false
+
+        version = PaperTrail::Version.where(item_id: flanchan.id, item_type: 'Entity').last
+        put engine_urls.version_path(version)
+
+        found = Entity.find(flanchan.id)
+        expect(found.status).to eq 'The Embodiment of Scarlet Devil'
+      end
+    end
+  end
+
+  def open_reimu_actions_menu
+    within("tr[data-ci-id='#{reimu.versions.find_by!(event: :create).id}']") do
+      find('[data-ci-type="version-action-button"]').click
     end
   end
 end

metadata

@@ -1,7 +1,7 @@
 --- !ruby/object:Gem::Specification
 name: paper_trail_viewer
 version: !ruby/object:Gem::Version
-  version: 1.0.0
+  version: 1.1.1
 platform: ruby
 authors:
 - Janosch Müller
@@ -9,7 +9,7 @@ authors:
 autorequire:
 bindir: bin
 cert_chain: []
-date: 2022-03-26 00:00:00.000000000 Z
+date: 2024-09-03 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: kaminari
@@ -26,162 +26,62 @@ dependencies:
       - !ruby/object:Gem::Version
         version: '0'
 - !ruby/object:Gem::Dependency
-  name: paper_trail
+  name: ostruct
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '11.1'
+        version: '0'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '11.1'
+        version: '0'
 - !ruby/object:Gem::Dependency
-  name: rails
+  name: paper_trail
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.0'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '8.0'
+        version: '11.1'
   type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: '6.0'
-    - - "<"
-      - !ruby/object:Gem::Version
-        version: '8.0'
-- !ruby/object:Gem::Dependency
-  name: appraisal
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '1.0'
-- !ruby/object:Gem::Dependency
-  name: capybara
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '3.0'
+        version: '11.1'
 - !ruby/object:Gem::Dependency
-  name: factory_bot_rails
+  name: rails
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '6.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
         version: '6.0'
-- !ruby/object:Gem::Dependency
-  name: puma
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-- !ruby/object:Gem::Dependency
-  name: rake
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '13.0'
-- !ruby/object:Gem::Dependency
-  name: rspec-rails
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-- !ruby/object:Gem::Dependency
-  name: sqlite3
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - ">="
+    - - "<"
       - !ruby/object:Gem::Version
-        version: 1.3.6
-  type: :development
+        version: '9.0'
+  type: :runtime
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - ">="
       - !ruby/object:Gem::Version
-        version: 1.3.6
-- !ruby/object:Gem::Dependency
-  name: webdrivers
-  requirement: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
-      - !ruby/object:Gem::Version
-        version: '5.0'
-  type: :development
-  prerelease: false
-  version_requirements: !ruby/object:Gem::Requirement
-    requirements:
-    - - "~>"
+        version: '6.0'
+    - - "<"
       - !ruby/object:Gem::Version
-        version: '5.0'
+        version: '9.0'
 description: Browse, view and revert changes to records when using Ruby on Rails and
   the `paper_trail` gem.
 email:
-executables:
-- setup
+executables: []
 extensions: []
 extra_rdoc_files: []
 files:
-- ".github/workflows/test.yml"
+- ".github/workflows/tests.yml"
 - ".gitignore"
-- Appraisals
 - CHANGELOG.md
 - Gemfile
 - LICENSE.txt
@@ -193,25 +93,27 @@ files:
 - app/views/paper_trail_viewer/viewer/index.html.erb
 - bin/setup
 - config/routes.rb
-- gemfiles/rails_6.0_paper_trail_11.1.gemfile
-- gemfiles/rails_6.0_paper_trail_12.2.gemfile
-- gemfiles/rails_7.0_paper_trail_12.2.gemfile
 - javascript/compiled.js
 - javascript/src/app.tsx
 - javascript/src/components/change_diff.tsx
 - javascript/src/components/config_modal.tsx
+- javascript/src/components/context_menu.tsx
 - javascript/src/components/controls.tsx
 - javascript/src/components/full_object_modal.tsx
 - javascript/src/components/index.ts
 - javascript/src/components/modal.tsx
 - javascript/src/components/pagination.tsx
+- javascript/src/components/version_context_menu.tsx
 - javascript/src/components/versions_list.tsx
 - javascript/src/index.ts
 - javascript/src/types.ts
 - lib/paper_trail_viewer.rb
 - lib/paper_trail_viewer/data_source/active_record.rb
+- lib/paper_trail_viewer/data_source/base.rb
 - lib/paper_trail_viewer/data_source/bigquery.rb
 - lib/paper_trail_viewer/engine.rb
+- lib/paper_trail_viewer/query.rb
+- lib/paper_trail_viewer/rollback.rb
 - lib/paper_trail_viewer/version.rb
 - package.json
 - paper_trail_viewer.gemspec
@@ -233,7 +135,7 @@ required_ruby_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="
     - !ruby/object:Gem::Version
-      version: '2.7'
+      version: '3.0'
 required_rubygems_version: !ruby/object:Gem::Requirement
   requirements:
   - - ">="

data/Appraisals

@@ -1,13 +0,0 @@
-{
-  '6.0' => %w[11.1 12.2],
-  '7.0' => %w[12.2],
-}.each do |rails_version, paper_trail_versions|
-  paper_trail_versions.each do |paper_trail_version|
-    appraise "rails-#{rails_version}-paper_trail-#{paper_trail_version}" do
-      gem 'rails', "~> #{rails_version}"
-      gem 'sqlite3', '~> 1.4'
-      gem 'paper_trail', "~> #{paper_trail_version}"
-      gem 'kaminari'
-    end
-  end
-end

data/gemfiles/rails_6.0_paper_trail_11.1.gemfile

@@ -1,10 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "rails", "~> 6.0"
-gem "sqlite3", "~> 1.4"
-gem "paper_trail", "~> 11.1"
-gem "kaminari"
-
-gemspec :path => "../"

data/gemfiles/rails_6.0_paper_trail_12.2.gemfile

@@ -1,10 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "rails", "~> 6.0"
-gem "sqlite3", "~> 1.4"
-gem "paper_trail", "~> 12.2"
-gem "kaminari"
-
-gemspec :path => "../"

data/gemfiles/rails_7.0_paper_trail_12.2.gemfile

@@ -1,10 +0,0 @@
-# This file was generated by Appraisal
-
-source "https://rubygems.org"
-
-gem "rails", "~> 7.0"
-gem "sqlite3", "~> 1.4"
-gem "paper_trail", "~> 12.2"
-gem "kaminari"
-
-gemspec :path => "../"