Merge pull request #752 from MITLibraries/etd-357-more-papertrail

ETD-357 Add papertrail to Thesis and User models

Author: Adam Jazairi

app/models/thesis.rb

@@ -22,6 +22,8 @@
 #
 
 class Thesis < ApplicationRecord
+  has_paper_trail
+  
   belongs_to :copyright, optional: true
   belongs_to :license, optional: true
 

app/models/user.rb

@@ -19,6 +19,8 @@
 #
 
 class User < ApplicationRecord
+  has_paper_trail
+
   # We need to initialize  some fields before validation, or
   # the record won't save.
   before_validation(on: :create) do

test/models/thesis_test.rb

@@ -735,4 +735,21 @@ class ThesisTest < ActiveSupport::TestCase
     thesis.reload
     assert_equal 'Published', thesis.publication_status
   end
+
+  test 'editing thesis generates an audit trail' do
+    t = theses(:one)
+    assert_equal t.versions.count, 0
+    t.title = 'updated'
+    t.save
+    assert_equal t.versions.count, 1
+    assert_equal t.versions.last.event, 'update'
+  end
+
+  test 'audit records include the changeset' do
+    t = theses(:one)
+    t.title = 'updated'
+    t.save
+    change = t.versions.last
+    assert_equal change.changeset['title'], %w[MyString updated]
+  end
 end

test/models/user_test.rb

@@ -365,4 +365,21 @@ class UserTest < ActiveSupport::TestCase
     u.reload
     assert_equal count - 1, u.editable_theses.count
   end
+
+  test 'editing user generates an audit trail' do
+    u = users(:yo)
+    assert_equal u.versions.count, 0
+    u.given_name = 'Hello'
+    u.save
+    assert_equal u.versions.count, 1
+    assert_equal u.versions.last.event, 'update'
+  end
+
+  test 'audit records include the changeset' do
+    u = users(:yo)
+    u.given_name = 'Hello'
+    u.save
+    change = u.versions.last
+    assert_equal change.changeset['given_name'], %w[Yo Hello]
+  end
 end