Add version tests for quotes, reviews and signups

test/models/beer_test.rb

@@ -10,9 +10,7 @@ class BeerTest < ActiveSupport::TestCase
   end
 
   test 'beer does have versions (papertrail)' do
-    assert_nothing_raised do
-      @beer.versions
-    end
+    assert_nothing_raised { @beer.versions }
 
     with_versioning do
       @beer.save

test/models/quote_test.rb

@@ -57,4 +57,43 @@ class QuoteTest < ActiveSupport::TestCase
     @quote.delete
     refute @quote.deleted_at.nil?
   end
+
+  test 'has a new version when text changes' do
+    with_versioning do
+      @quote.save
+      assert_difference '@quote.versions.count' do
+        @quote.text = "New quote body"
+        @quote.save
+      end
+    end
+  end
+
+  test 'has a new version when user changes' do
+    with_versioning do
+      @quote.save
+      assert_difference '@quote.versions.count' do
+        @quote.user = users(:two)
+        @quote.save
+      end
+    end
+  end
+
+  test 'has a new version when reporter changes' do
+    with_versioning do
+      @quote.save
+      assert_difference '@quote.versions.count' do
+        @quote.reporter = users(:three)
+        @quote.save
+      end
+    end
+  end
+
+  test 'has a new version when it is deleted' do
+    with_versioning do
+      @quote.save
+      assert_difference 'PaperTrail::Version.count' do
+        @quote.destroy
+      end
+    end
+  end
 end

test/models/review_test.rb

@@ -63,4 +63,48 @@ class ReviewTest < ActiveSupport::TestCase
     @review.beer_id = -1
     refute @review.valid?
   end
+
+  test 'has versions (papertrail)' do
+    assert_nothing_raised { @review.versions }
+  end
+
+  test 'has a new version when rating changes' do
+    with_versioning do
+      @review.save
+      assert_difference '@review.versions.count' do
+        @review.rating = 6
+        @review.save
+      end
+    end
+  end
+
+  test 'has a new version when description changes' do
+    with_versioning do
+      @review.save
+      assert_difference '@review.versions.count' do
+        @review.actiontext_description = "New description"
+        @review.save
+      end
+    end
+  end
+
+  test 'has a new version when proefdatum changes' do
+    with_versioning do
+      @review.save
+      assert_difference '@review.versions.count' do
+        @review.proefdatum = Date.today - 1.day
+        @review.save
+      end
+    end
+  end
+
+  test 'has a new version when it is deleted' do
+    with_versioning do
+      @review.save
+      # TODO: Why is this 2?
+      assert_difference 'PaperTrail::Version.count', 2 do
+        @review.destroy
+      end
+    end
+  end
 end

test/models/signup_test.rb

@@ -8,7 +8,7 @@ class SignupTest < ActiveSupport::TestCase
                           end_time: DateTime.now + 1.day + 2.hours,
                           deadline: DateTime.now + 5,
                           user: @user)
-    @signup = Signup.new(event: @event, user: @user)
+    @signup = Signup.new(event: @event, user: @user) # Status is true by default
   end
 
   test 'valid signup' do
@@ -61,4 +61,37 @@ class SignupTest < ActiveSupport::TestCase
     @signup.status = nil
     refute @signup.valid?
   end
+
+  test 'has versions (papertrail)' do
+    assert_nothing_raised { @signup.versions }
+  end
+
+  test 'has a new version when status changes' do
+    with_versioning do
+      @signup.save
+      assert_difference '@signup.versions.count' do
+        @signup.status = false
+        @signup.save
+      end
+    end
+  end
+
+  test 'has a new version when reason changes' do
+    with_versioning do
+      @signup.save
+      assert_difference '@signup.versions.count' do
+        @signup.reason = "New reason"
+        @signup.save
+      end
+    end
+  end
+
+  test 'has a new version when it is deleted' do
+    with_versioning do
+      @signup.save
+      assert_difference 'PaperTrail::Version.count' do
+        @signup.destroy
+      end
+    end
+  end
 end