diff --git a/test/models/beer_test.rb b/test/models/beer_test.rb index f2c4b6ad1..7ff29738c 100644 --- a/test/models/beer_test.rb +++ b/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 diff --git a/test/models/quote_test.rb b/test/models/quote_test.rb index c335dfc94..fa4a99911 100755 --- a/test/models/quote_test.rb +++ b/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 diff --git a/test/models/review_test.rb b/test/models/review_test.rb index 6f077cc1f..0f505149f 100644 --- a/test/models/review_test.rb +++ b/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 diff --git a/test/models/signup_test.rb b/test/models/signup_test.rb index 9ed9ee4c7..ce96e4b95 100644 --- a/test/models/signup_test.rb +++ b/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