X-Git-Url: https://git.openstreetmap.org./rails.git/blobdiff_plain/b78bb9f6313f3139ac83784df52bf39a5619d3e7..416334cbbc3ac3037dd559cfa5dc561950da642f:/test/models/changeset_comment_test.rb diff --git a/test/models/changeset_comment_test.rb b/test/models/changeset_comment_test.rb index 5ef0c1d93..2139c8d39 100644 --- a/test/models/changeset_comment_test.rb +++ b/test/models/changeset_comment_test.rb @@ -1,35 +1,32 @@ -# -*- coding: utf-8 -*- require "test_helper" class ChangesetCommentTest < ActiveSupport::TestCase - fixtures :changesets, :users - # validations def test_does_not_accept_invalid_author comment = create(:changeset_comment) comment.author = nil - assert !comment.valid? + assert_not comment.valid? comment.author_id = 999111 - assert !comment.valid? + assert_not comment.valid? end def test_does_not_accept_invalid_changeset comment = create(:changeset_comment) comment.changeset = nil - assert !comment.valid? + assert_not comment.valid? comment.changeset_id = 999111 - assert !comment.valid? + assert_not comment.valid? end def test_does_not_accept_empty_visible comment = create(:changeset_comment) comment.visible = nil - assert !comment.valid? + assert_not comment.valid? end def test_comments_of_changeset_count @@ -39,21 +36,21 @@ class ChangesetCommentTest < ActiveSupport::TestCase end def test_body_valid - ok = %W(Name vergrößern foo\nbar - ルシステムにも対応します 輕觸搖晃的遊戲) + ok = %W[Name vergrößern foo\nbar + ルシステムにも対応します 輕觸搖晃的遊戲] bad = ["foo\x00bar", "foo\x08bar", "foo\x1fbar", "foo\x7fbar", "foo\ufffebar", "foo\uffffbar"] ok.each do |body| changeset_comment = create(:changeset_comment) changeset_comment.body = body - assert changeset_comment.valid?, "#{body} is invalid, when it should be" + assert_predicate changeset_comment, :valid?, "#{body} is invalid, when it should be" end bad.each do |body| changeset_comment = create(:changeset_comment) changeset_comment.body = body - assert !changeset_comment.valid?, "#{body} is valid when it shouldn't be" + assert_not changeset_comment.valid?, "#{body} is valid when it shouldn't be" end end end