paper_trail_viewer 1.0.0 → 1.1.0

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.0
 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: 2022-03-27 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: kaminari
@@ -174,12 +174,11 @@ dependencies:
 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
@@ -200,18 +199,23 @@ files:
 - 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