]> git.openstreetmap.org Git - rails.git/commitdiff
Fix new rubocop warnings
authorTom Hughes <tom@compton.nu>
Tue, 15 Mar 2022 18:24:32 +0000 (18:24 +0000)
committerTom Hughes <tom@compton.nu>
Tue, 15 Mar 2022 19:24:09 +0000 (19:24 +0000)
31 files changed:
test/controllers/issues_controller_test.rb
test/helpers/application_helper_test.rb
test/integration/oauth_test.rb
test/lib/bounding_box_test.rb
test/lib/rich_text_test.rb
test/models/acl_test.rb
test/models/changeset_comment_test.rb
test/models/changeset_tag_test.rb
test/models/client_application_test.rb
test/models/issue_test.rb
test/models/message_test.rb
test/models/node_tag_test.rb
test/models/note_comment_test.rb
test/models/note_test.rb
test/models/oauth_token_test.rb
test/models/old_node_tag_test.rb
test/models/old_relation_tag_test.rb
test/models/old_way_tag_test.rb
test/models/redaction_test.rb
test/models/relation_tag_test.rb
test/models/report_test.rb
test/models/request_token_test.rb
test/models/trace_test.rb
test/models/tracepoint_test.rb
test/models/user_preference_test.rb
test/models/user_test.rb
test/models/way_tag_test.rb
test/models/way_test.rb
test/system/report_diary_entry_test.rb
test/validators/characters_validator_test.rb
test/validators/whitespace_validator_test.rb

index 5e980e0b8a02e5962e0bca421129a83fac210b7a..f9f29afbbb889aee343080f5c3295b5f29a606d1 100644 (file)
@@ -100,7 +100,7 @@ class IssuesControllerTest < ActionDispatch::IntegrationTest
     session_for(create(:moderator_user))
     post resolve_issue_path(:id => issue)
     assert_response :redirect
-    assert issue.reload.resolved?
+    assert_predicate issue.reload, :resolved?
   end
 
   def test_resolve_administrator
@@ -127,7 +127,7 @@ class IssuesControllerTest < ActionDispatch::IntegrationTest
     session_for(create(:administrator_user))
     post resolve_issue_path(:id => issue)
     assert_response :redirect
-    assert issue.reload.resolved?
+    assert_predicate issue.reload, :resolved?
   end
 
   def test_ignore_moderator
@@ -154,7 +154,7 @@ class IssuesControllerTest < ActionDispatch::IntegrationTest
     session_for(create(:moderator_user))
     post ignore_issue_path(:id => issue)
     assert_response :redirect
-    assert issue.reload.ignored?
+    assert_predicate issue.reload, :ignored?
   end
 
   def test_ignore_administrator
@@ -181,7 +181,7 @@ class IssuesControllerTest < ActionDispatch::IntegrationTest
     session_for(create(:administrator_user))
     post ignore_issue_path(:id => issue)
     assert_response :redirect
-    assert issue.reload.ignored?
+    assert_predicate issue.reload, :ignored?
   end
 
   def test_reopen_moderator
@@ -210,7 +210,7 @@ class IssuesControllerTest < ActionDispatch::IntegrationTest
     session_for(create(:moderator_user))
     post reopen_issue_path(:id => issue)
     assert_response :redirect
-    assert issue.reload.open?
+    assert_predicate issue.reload, :open?
   end
 
   def test_reopen_administrator
@@ -239,6 +239,6 @@ class IssuesControllerTest < ActionDispatch::IntegrationTest
     session_for(create(:administrator_user))
     post reopen_issue_path(:id => issue)
     assert_response :redirect
-    assert issue.reload.open?
+    assert_predicate issue.reload, :open?
   end
 end
index d3e4d4f072b3006a53bed6a36f95063f665d95d5..06c1c09f85133fb359e007de5ff5c38902188d8d 100644 (file)
@@ -8,11 +8,11 @@ class ApplicationHelperTest < ActionView::TestCase
       text = "Test #{link} is <b>made</b> into a link"
 
       html = linkify(text)
-      assert html.html_safe?
+      assert_predicate html, :html_safe?
       assert_dom_equal "Test <a href=\"#{link}\" rel=\"nofollow\">#{link}</a> is &lt;b&gt;made&lt;/b&gt; into a link", html
 
       html = linkify(text.html_safe)
-      assert html.html_safe?
+      assert_predicate html, :html_safe?
       assert_dom_equal "Test <a href=\"#{link}\" rel=\"nofollow\">#{link}</a> is <b>made</b> into a link", html
     end
 
@@ -20,11 +20,11 @@ class ApplicationHelperTest < ActionView::TestCase
       text = "Test #{link} is not <b>made</b> into a link"
 
       html = linkify(text)
-      assert html.html_safe?
+      assert_predicate html, :html_safe?
       assert_dom_equal "Test #{link} is not &lt;b&gt;made&lt;/b&gt; into a link", html
 
       html = linkify(text.html_safe)
-      assert html.html_safe?
+      assert_predicate html, :html_safe?
       assert_dom_equal "Test #{link} is not <b>made</b> into a link", html
     end
   end
index 579941774567b621b156cccf1ff197da99b2f3e9..2c473aac5e799040cff309c6ebf43411bbd9b991 100644 (file)
@@ -358,7 +358,7 @@ class OAuthTest < ActionDispatch::IntegrationTest
     params = CGI.parse(URI.parse(response.location).query)
 
     assert_not_nil params["oauth_verifier"]
-    assert params["oauth_verifier"].first.present?
+    assert_predicate params["oauth_verifier"].first, :present?
 
     params["oauth_verifier"].first
   end
index 866820a6e2fecf799a9eb18e285e250085409662..d82e2c9ae453b2b6ab4426398ee4f1ef1fcdb4fb 100644 (file)
@@ -223,7 +223,7 @@ class BoundingBoxTest < ActiveSupport::TestCase
 
   def test_complete
     assert_not @bbox_from_nils.complete?, "should contain a nil"
-    assert @bbox_from_string.complete?, "should not contain a nil"
+    assert_predicate @bbox_from_string, :complete?, "should not contain a nil"
   end
 
   def test_centre_lon
index fc9355717565018f02612962663fbd7eb6c5cd4d..9d00d658d5bac697b3e4a8bce6e4bfb01fc52469 100644 (file)
@@ -224,7 +224,7 @@ class RichTextTest < ActiveSupport::TestCase
 
   def assert_html(richtext, &block)
     html = richtext.to_html
-    assert html.html_safe?
+    assert_predicate html, :html_safe?
     root = Nokogiri::HTML::DocumentFragment.parse(html)
     assert_select root, "*" do
       yield block
index 49f065612ffa8459d6198466e35deede9b55302a..88d1c0e7d028f4b3675139f24766a0a36828ec6a 100644 (file)
@@ -3,7 +3,7 @@ require "test_helper"
 class AclTest < ActiveSupport::TestCase
   def test_k_required
     acl = create(:acl)
-    assert acl.valid?
+    assert_predicate acl, :valid?
     acl.k = nil
     assert_not acl.valid?
   end
index b620dc8e7e805f5f822d8921c17d870a93f42384..2139c8d39a9bc9f7145303b8508ec1081321b2ed 100644 (file)
@@ -44,7 +44,7 @@ class ChangesetCommentTest < ActiveSupport::TestCase
     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|
index 90674fbbe420042b90237f86b7388d9a59819f9f..0c5ffb151cd69b4f4926867f1a4824f2d0fcd1a7 100644 (file)
@@ -10,7 +10,7 @@ class ChangesetTagTest < ActiveSupport::TestCase
       tag.changeset_id = changeset.id
       tag.k = key * i
       tag.v = "v"
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -23,7 +23,7 @@ class ChangesetTagTest < ActiveSupport::TestCase
       tag.changeset_id = changeset.id
       tag.k = "k"
       tag.v = val * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -34,7 +34,7 @@ class ChangesetTagTest < ActiveSupport::TestCase
       tag.k = k
       tag.v = "v"
       assert_not tag.valid?, "Key #{k} should be too long"
-      assert tag.errors[:k].any?
+      assert_predicate tag.errors[:k], :any?
     end
   end
 
@@ -45,14 +45,14 @@ class ChangesetTagTest < ActiveSupport::TestCase
       tag.k = "k"
       tag.v = v
       assert_not tag.valid?, "Value #{v} should be too long"
-      assert tag.errors[:v].any?
+      assert_predicate tag.errors[:v], :any?
     end
   end
 
   def test_empty_tag_invalid
     tag = ChangesetTag.new
     assert_not tag.valid?, "Empty tag should be invalid"
-    assert tag.errors[:changeset].any?
+    assert_predicate tag.errors[:changeset], :any?
   end
 
   def test_uniqueness
@@ -61,9 +61,9 @@ class ChangesetTagTest < ActiveSupport::TestCase
     tag.changeset_id = existing.changeset_id
     tag.k = existing.k
     tag.v = existing.v
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
     assert_not tag.valid?
     assert_raise(ActiveRecord::RecordInvalid) { tag.save! }
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
   end
 end
index c0b7b7977fcd6d84a7e4b6c5a9422768c5caccce..c3f1df23c23c996da989ddaaaf7dba90c9fb31ac 100644 (file)
@@ -8,7 +8,7 @@ class ClientApplicationTest < ActiveSupport::TestCase
     ok.each do |url|
       app = build(:client_application)
       app.url = url
-      assert app.valid?, "#{url} is invalid, when it should be"
+      assert_predicate app, :valid?, "#{url} is invalid, when it should be"
     end
 
     bad.each do |url|
@@ -25,7 +25,7 @@ class ClientApplicationTest < ActiveSupport::TestCase
     ok.each do |url|
       app = build(:client_application)
       app.support_url = url
-      assert app.valid?, "#{url} is invalid, when it should be"
+      assert_predicate app, :valid?, "#{url} is invalid, when it should be"
     end
 
     bad.each do |url|
@@ -42,7 +42,7 @@ class ClientApplicationTest < ActiveSupport::TestCase
     ok.each do |url|
       app = build(:client_application)
       app.callback_url = url
-      assert app.valid?, "#{url} is invalid, when it should be"
+      assert_predicate app, :valid?, "#{url} is invalid, when it should be"
     end
 
     bad.each do |url|
index 02e2580b81daa08ad5dc081248d134f7d43e621d..f166282c1dcbf1bad0aec59d1eafd40f34b2f31b 100644 (file)
@@ -4,7 +4,7 @@ class IssueTest < ActiveSupport::TestCase
   def test_assigned_role
     issue = create(:issue)
 
-    assert issue.valid?
+    assert_predicate issue, :valid?
     issue.assigned_role = "bogus"
     assert_not issue.valid?
   end
index 1a12410916775e2ed67cb1ff67069c0f68161b18..97afad56e17f80c72ec2f71add8c9f03146fb4c3 100644 (file)
@@ -6,17 +6,17 @@ class MessageTest < ActiveSupport::TestCase
   def test_check_empty_message_fails
     message = build(:message, :title => nil, :body => nil, :sent_on => nil)
     assert_not message.valid?
-    assert message.errors[:title].any?
-    assert message.errors[:body].any?
-    assert message.errors[:sent_on].any?
+    assert_predicate message.errors[:title], :any?
+    assert_predicate message.errors[:body], :any?
+    assert_predicate message.errors[:sent_on], :any?
     assert_not message.message_read
   end
 
   def test_validating_msgs
     message = create(:message, :unread)
-    assert message.valid?
+    assert_predicate message, :valid?
     message = create(:message, :read)
-    assert message.valid?
+    assert_predicate message, :valid?
   end
 
   def test_invalid_send_recipient
index 47295770f540992bbda0e9327b0da39a01cf9918..ffc8fb422d4655ac806a9ae861478c06bf35762a 100644 (file)
@@ -5,7 +5,7 @@ class NodeTagTest < ActiveSupport::TestCase
     tag = create(:node_tag)
     [0, 255].each do |i|
       tag.k = "k" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -13,7 +13,7 @@ class NodeTagTest < ActiveSupport::TestCase
     tag = create(:node_tag)
     [0, 255].each do |i|
       tag.v = "v" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -21,20 +21,20 @@ class NodeTagTest < ActiveSupport::TestCase
     tag = create(:node_tag)
     tag.k = "k" * 256
     assert_not tag.valid?, "Key should be too long"
-    assert tag.errors[:k].any?
+    assert_predicate tag.errors[:k], :any?
   end
 
   def test_length_value_invalid
     tag = create(:node_tag)
     tag.v = "v" * 256
     assert_not tag.valid?, "Value should be too long"
-    assert tag.errors[:v].any?
+    assert_predicate tag.errors[:v], :any?
   end
 
   def test_empty_node_tag_invalid
     tag = NodeTag.new
     assert_not tag.valid?, "Empty tag should be invalid"
-    assert tag.errors[:node].any?
+    assert_predicate tag.errors[:node], :any?
   end
 
   def test_uniqueness
@@ -43,9 +43,9 @@ class NodeTagTest < ActiveSupport::TestCase
     tag.node_id = existing.node_id
     tag.k = existing.k
     tag.v = existing.v
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
     assert_not tag.valid?
     assert_raise(ActiveRecord::RecordInvalid) { tag.save! }
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
   end
 end
index bfedcf12592730f18947552ddf5022ff3b0048de..f4dd645ee21a9392d8e13f6f557e6142aac07e9e 100644 (file)
@@ -8,7 +8,7 @@ class NoteCommentTest < ActiveSupport::TestCase
     ok.each do |event|
       note_comment = create(:note_comment)
       note_comment.event = event
-      assert note_comment.valid?, "#{event} is invalid, when it should be"
+      assert_predicate note_comment, :valid?, "#{event} is invalid, when it should be"
     end
 
     bad.each do |event|
@@ -27,7 +27,7 @@ class NoteCommentTest < ActiveSupport::TestCase
     ok.each do |body|
       note_comment = create(:note_comment)
       note_comment.body = body
-      assert note_comment.valid?, "#{body} is invalid, when it should be"
+      assert_predicate note_comment, :valid?, "#{body} is invalid, when it should be"
     end
 
     bad.each do |body|
index b74cebcc4cc4d2da028f451eb97e0a5bf57ff0ce..8d5a557cd4dfbbe9b8ffb043fe6047aec7f3049c 100644 (file)
@@ -8,7 +8,7 @@ class NoteTest < ActiveSupport::TestCase
     ok.each do |status|
       note = create(:note)
       note.status = status
-      assert note.valid?, "#{status} is invalid, when it should be"
+      assert_predicate note, :valid?, "#{status} is invalid, when it should be"
     end
 
     bad.each do |status|
@@ -37,13 +37,13 @@ class NoteTest < ActiveSupport::TestCase
   end
 
   def test_visible?
-    assert create(:note, :status => "open").visible?
-    assert create(:note, :status => "closed").visible?
+    assert_predicate create(:note, :status => "open"), :visible?
+    assert_predicate create(:note, :status => "closed"), :visible?
     assert_not create(:note, :status => "hidden").visible?
   end
 
   def test_closed?
-    assert create(:note, :status => "closed", :closed_at => Time.now.utc).closed?
+    assert_predicate create(:note, :status => "closed", :closed_at => Time.now.utc), :closed?
     assert_not create(:note, :status => "open", :closed_at => nil).closed?
   end
 
index 3e3aaae7c8738fc80774b8af34163c9a12bbf10e..28f2c34a17343cee7aace42f4a1c6c89cea45f1e 100644 (file)
@@ -7,7 +7,7 @@ class OauthTokenTest < ActiveSupport::TestCase
     tok = OauthToken.new
     assert_not tok.invalidated?, "Token should be created valid."
     tok.invalidate!
-    assert tok.invalidated?, "Token should now be invalid."
+    assert_predicate tok, :invalidated?, "Token should now be invalid."
   end
 
   ##
@@ -16,7 +16,7 @@ class OauthTokenTest < ActiveSupport::TestCase
     tok = RequestToken.create(:client_application => create(:client_application))
     assert_not tok.authorized?, "Token should be created unauthorised."
     tok.authorize!(create(:user))
-    assert tok.authorized?, "Token should now be authorised."
+    assert_predicate tok, :authorized?, "Token should now be authorised."
     tok.invalidate!
     assert_not tok.authorized?, "Token should now be invalid."
   end
index 62869c6adfff3e61023f08382c943c92ac32f1d8..ddcda22039d657316831f300d2b3a57e91d3b686 100644 (file)
@@ -5,7 +5,7 @@ class OldNodeTagTest < ActiveSupport::TestCase
     tag = create(:old_node_tag)
     [0, 255].each do |i|
       tag.k = "k" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -13,7 +13,7 @@ class OldNodeTagTest < ActiveSupport::TestCase
     tag = create(:old_node_tag)
     [0, 255].each do |i|
       tag.v = "v" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -21,20 +21,20 @@ class OldNodeTagTest < ActiveSupport::TestCase
     tag = create(:old_node_tag)
     tag.k = "k" * 256
     assert_not tag.valid?
-    assert tag.errors[:k].any?
+    assert_predicate tag.errors[:k], :any?
   end
 
   def test_length_value_invalid
     tag = create(:old_node_tag)
     tag.v = "v" * 256
     assert_not tag.valid?, "Value should be too long"
-    assert tag.errors[:v].any?
+    assert_predicate tag.errors[:v], :any?
   end
 
   def test_empty_tag_invalid
     tag = OldNodeTag.new
     assert_not tag.valid?, "Empty tag should be invalid"
-    assert tag.errors[:old_node].any?
+    assert_predicate tag.errors[:old_node], :any?
   end
 
   def test_uniqueness
@@ -44,9 +44,9 @@ class OldNodeTagTest < ActiveSupport::TestCase
     tag.version = existing.version
     tag.k = existing.k
     tag.v = existing.v
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
     assert_not tag.valid?
     assert_raise(ActiveRecord::RecordInvalid) { tag.save! }
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
   end
 end
index dbdbf54c526f6c5ede048e3626766e827e4458d5..a373e386df53bf839bed90ba965b6cd26221e133 100644 (file)
@@ -5,7 +5,7 @@ class OldRelationTagTest < ActiveSupport::TestCase
     tag = create(:old_relation_tag)
     [0, 255].each do |i|
       tag.k = "k" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -13,7 +13,7 @@ class OldRelationTagTest < ActiveSupport::TestCase
     tag = create(:old_relation_tag)
     [0, 255].each do |i|
       tag.v = "v" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -21,20 +21,20 @@ class OldRelationTagTest < ActiveSupport::TestCase
     tag = create(:old_relation_tag)
     tag.k = "k" * 256
     assert_not tag.valid?, "Key should be too long"
-    assert tag.errors[:k].any?
+    assert_predicate tag.errors[:k], :any?
   end
 
   def test_length_value_invalid
     tag = create(:old_relation_tag)
     tag.v = "v" * 256
     assert_not tag.valid?, "Value should be too long"
-    assert tag.errors[:v].any?
+    assert_predicate tag.errors[:v], :any?
   end
 
   def test_empty_tag_invalid
     tag = OldRelationTag.new
     assert_not tag.valid?, "Empty tag should be invalid"
-    assert tag.errors[:old_relation].any?
+    assert_predicate tag.errors[:old_relation], :any?
   end
 
   def test_uniqueness
@@ -44,9 +44,9 @@ class OldRelationTagTest < ActiveSupport::TestCase
     tag.version = existing.version
     tag.k = existing.k
     tag.v = existing.v
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
     assert_not tag.valid?
     assert_raise(ActiveRecord::RecordInvalid) { tag.save! }
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
   end
 end
index d794cc275ea0eaff4e52ae6c962ecff830877f0e..51c644c18f63b8592edadbe4d12d749b59ed78b9 100644 (file)
@@ -5,7 +5,7 @@ class OldWayTagTest < ActiveSupport::TestCase
     tag = create(:old_way_tag)
     [0, 255].each do |i|
       tag.k = "k" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -13,7 +13,7 @@ class OldWayTagTest < ActiveSupport::TestCase
     tag = create(:old_way_tag)
     [0, 255].each do |i|
       tag.v = "v" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -21,20 +21,20 @@ class OldWayTagTest < ActiveSupport::TestCase
     tag = create(:old_way_tag)
     tag.k = "k" * 256
     assert_not tag.valid?, "Key should be too long"
-    assert tag.errors[:k].any?
+    assert_predicate tag.errors[:k], :any?
   end
 
   def test_length_value_invalid
     tag = create(:old_way_tag)
     tag.v = "v" * 256
     assert_not tag.valid?, "Value should be too long"
-    assert tag.errors[:v].any?
+    assert_predicate tag.errors[:v], :any?
   end
 
   def test_empty_tag_invalid
     tag = OldWayTag.new
     assert_not tag.valid?, "Empty tag should be invalid"
-    assert tag.errors[:old_way].any?
+    assert_predicate tag.errors[:old_way], :any?
   end
 
   def test_uniqueness
@@ -44,9 +44,9 @@ class OldWayTagTest < ActiveSupport::TestCase
     tag.version = existing.version
     tag.k = existing.k
     tag.v = existing.v
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
     assert_not tag.valid?
     assert_raise(ActiveRecord::RecordInvalid) { tag.save! }
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
   end
 end
index d74a9117969ba2ac89868889dbfd15351c0b30cc..06159cb5d79540280f3a62d53fed1b1b0d74da9c 100644 (file)
@@ -30,7 +30,7 @@ class RedactionTest < ActiveSupport::TestCase
     assert_nothing_raised do
       node_v1.redact!(r)
     end
-    assert(node_v1.redacted?, "Expected node version 1 to be redacted after redact! call.")
+    assert_predicate(node_v1, :redacted?, "Expected node version 1 to be redacted after redact! call.")
     assert_not(node_v2.redacted?, "Expected node version 2 to not be redacted after redact! call.")
   end
 end
index 02d6d95c705a6d2ca798372bdc1dff3ab05e42a5..4e9108a634da14576367a047c8d6e57536c1e989 100644 (file)
@@ -5,7 +5,7 @@ class RelationTagTest < ActiveSupport::TestCase
     tag = create(:relation_tag)
     [0, 255].each do |i|
       tag.k = "k" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -13,7 +13,7 @@ class RelationTagTest < ActiveSupport::TestCase
     tag = create(:relation_tag)
     [0, 255].each do |i|
       tag.v = "v" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -21,20 +21,20 @@ class RelationTagTest < ActiveSupport::TestCase
     tag = create(:relation_tag)
     tag.k = "k" * 256
     assert_not tag.valid?, "Key should be too long"
-    assert tag.errors[:k].any?
+    assert_predicate tag.errors[:k], :any?
   end
 
   def test_length_value_invalid
     tag = create(:relation_tag)
     tag.v = "v" * 256
     assert_not tag.valid?, "Value should be too long"
-    assert tag.errors[:v].any?
+    assert_predicate tag.errors[:v], :any?
   end
 
   def test_empty_tag_invalid
     tag = RelationTag.new
     assert_not tag.valid?, "Empty relation tag should be invalid"
-    assert tag.errors[:relation].any?
+    assert_predicate tag.errors[:relation], :any?
   end
 
   def test_uniqueness
@@ -43,10 +43,10 @@ class RelationTagTest < ActiveSupport::TestCase
     tag.relation_id = existing.relation_id
     tag.k = existing.k
     tag.v = existing.v
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
     assert_not tag.valid?
     assert_raise(ActiveRecord::RecordInvalid) { tag.save! }
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
   end
 
   ##
index b65753b822d4e1a075d5337d37d951457990a648..d172ff7db97ace0a81c10b953661ea105913903c 100644 (file)
@@ -4,7 +4,7 @@ class ReportTest < ActiveSupport::TestCase
   def test_issue_required
     report = create(:report)
 
-    assert report.valid?
+    assert_predicate report, :valid?
     report.issue = nil
     assert_not report.valid?
   end
@@ -12,7 +12,7 @@ class ReportTest < ActiveSupport::TestCase
   def test_user_required
     report = create(:report)
 
-    assert report.valid?
+    assert_predicate report, :valid?
     report.user = nil
     assert_not report.valid?
   end
@@ -20,7 +20,7 @@ class ReportTest < ActiveSupport::TestCase
   def test_details_required
     report = create(:report)
 
-    assert report.valid?
+    assert_predicate report, :valid?
     report.details = ""
     assert_not report.valid?
   end
@@ -28,7 +28,7 @@ class ReportTest < ActiveSupport::TestCase
   def test_category_required
     report = create(:report)
 
-    assert report.valid?
+    assert_predicate report, :valid?
     report.category = ""
     assert_not report.valid?
   end
index c231371cefeb684a698b10a11640499b3c2257e2..4f17b725605c2274729c8ea7a7fc5230cc867a27 100644 (file)
@@ -2,9 +2,9 @@ require "test_helper"
 
 class RequestTokenTest < ActiveSupport::TestCase
   def test_oob
-    assert RequestToken.new.oob?
-    assert RequestToken.new(:callback_url => "oob").oob?
-    assert RequestToken.new(:callback_url => "OOB").oob?
+    assert_predicate RequestToken.new, :oob?
+    assert_predicate RequestToken.new(:callback_url => "oob"), :oob?
+    assert_predicate RequestToken.new(:callback_url => "OOB"), :oob?
     assert_not RequestToken.new(:callback_url => "http://test.host/").oob?
   end
 end
index 99aac41d8115c79b074b885b36887f8f23ccacf7..2dcae58d0dc372f91ae3c10758809ec147d5d50a 100644 (file)
@@ -76,14 +76,14 @@ class TraceTest < ActiveSupport::TestCase
   def test_tagstring
     trace = build(:trace)
     trace.tagstring = "foo bar baz"
-    assert trace.valid?
+    assert_predicate trace, :valid?
     assert_equal 3, trace.tags.length
     assert_equal "foo", trace.tags[0].tag
     assert_equal "bar", trace.tags[1].tag
     assert_equal "baz", trace.tags[2].tag
     assert_equal "foo, bar, baz", trace.tagstring
     trace.tagstring = "foo, bar baz ,qux"
-    assert trace.valid?
+    assert_predicate trace, :valid?
     assert_equal 3, trace.tags.length
     assert_equal "foo", trace.tags[0].tag
     assert_equal "bar baz", trace.tags[1].tag
@@ -92,18 +92,18 @@ class TraceTest < ActiveSupport::TestCase
   end
 
   def test_public?
-    assert build(:trace, :visibility => "public").public?
+    assert_predicate build(:trace, :visibility => "public"), :public?
     assert_not build(:trace, :visibility => "private").public?
     assert_not build(:trace, :visibility => "trackable").public?
-    assert build(:trace, :visibility => "identifiable").public?
-    assert build(:trace, :deleted, :visibility => "public").public?
+    assert_predicate build(:trace, :visibility => "identifiable"), :public?
+    assert_predicate build(:trace, :deleted, :visibility => "public"), :public?
   end
 
   def test_trackable?
     assert_not build(:trace, :visibility => "public").trackable?
     assert_not build(:trace, :visibility => "private").trackable?
-    assert build(:trace, :visibility => "trackable").trackable?
-    assert build(:trace, :visibility => "identifiable").trackable?
+    assert_predicate build(:trace, :visibility => "trackable"), :trackable?
+    assert_predicate build(:trace, :visibility => "identifiable"), :trackable?
     assert_not build(:trace, :deleted, :visibility => "public").trackable?
   end
 
@@ -111,7 +111,7 @@ class TraceTest < ActiveSupport::TestCase
     assert_not build(:trace, :visibility => "public").identifiable?
     assert_not build(:trace, :visibility => "private").identifiable?
     assert_not build(:trace, :visibility => "trackable").identifiable?
-    assert build(:trace, :visibility => "identifiable").identifiable?
+    assert_predicate build(:trace, :visibility => "identifiable"), :identifiable?
     assert_not build(:trace, :deleted, :visibility => "public").identifiable?
   end
 
@@ -197,7 +197,7 @@ class TraceTest < ActiveSupport::TestCase
 
     trace.import
 
-    assert trace.icon.attached?
+    assert_predicate trace.icon, :attached?
   end
 
   def test_import_creates_large_picture
@@ -207,7 +207,7 @@ class TraceTest < ActiveSupport::TestCase
 
     trace.import
 
-    assert trace.image.attached?
+    assert_predicate trace.image, :attached?
   end
 
   def test_import_handles_bz2
index 2ffbef09fae8f7c5d7de527c2fcb889dc3615e63..115b8ec9b21935421a44ff6321970e35c7fa59d0 100644 (file)
@@ -3,7 +3,7 @@ require "test_helper"
 class TracepointTest < ActiveSupport::TestCase
   def test_timestamp_required
     tracepoint = create(:tracepoint)
-    assert tracepoint.valid?
+    assert_predicate tracepoint, :valid?
     tracepoint.timestamp = nil
     assert_not tracepoint.valid?
   end
index 2f5e16b18f47703b1b827b0bf0a33e296dd62623..3df38aa6fe75796ca1bc8e38c88f83a3a4542b46 100644 (file)
@@ -20,7 +20,7 @@ class UserPreferenceTest < ActiveSupport::TestCase
       up.user = create(:user)
       up.k = key * i
       up.v = val * i
-      assert up.valid?
+      assert_predicate up, :valid?
       assert up.save!
       resp = UserPreference.find(up.id)
       assert_equal key * i, resp.k, "User preference with #{i} #{key} chars (i.e. #{key.length * i} bytes) fails"
index 644283e84f67a687b1ca9ed400416a8a5ffae936..72e1ca5d9817e74b3d163eb4fb38d8cd6c3e5c3d 100644 (file)
@@ -11,12 +11,12 @@ class UserTest < ActiveSupport::TestCase
                         :home_lon => nil,
                         :home_zoom => nil)
     assert_not user.valid?
-    assert user.errors[:email].any?
-    assert user.errors[:pass_crypt].any?
-    assert user.errors[:display_name].any?
-    assert user.errors[:home_lat].none?
-    assert user.errors[:home_lon].none?
-    assert user.errors[:home_zoom].none?
+    assert_predicate user.errors[:email], :any?
+    assert_predicate user.errors[:pass_crypt], :any?
+    assert_predicate user.errors[:display_name], :any?
+    assert_predicate user.errors[:home_lat], :none?
+    assert_predicate user.errors[:home_lon], :none?
+    assert_predicate user.errors[:home_zoom], :none?
   end
 
   def test_unique_email
@@ -55,7 +55,7 @@ class UserTest < ActiveSupport::TestCase
   def test_display_name_length
     user = build(:user)
     user.display_name = "123"
-    assert user.valid?, "should allow 3 char name name"
+    assert_predicate user, :valid?, "should allow 3 char name name"
     user.display_name = "12"
     assert_not user.valid?, "should not allow 2 char name"
     user.display_name = ""
@@ -81,7 +81,7 @@ class UserTest < ActiveSupport::TestCase
     ok.each do |display_name|
       user = build(:user)
       user.display_name = display_name
-      assert user.valid?, "#{display_name} is invalid, when it should be"
+      assert_predicate user, :valid?, "#{display_name} is invalid, when it should be"
     end
 
     bad.each do |display_name|
@@ -214,29 +214,29 @@ class UserTest < ActiveSupport::TestCase
   end
 
   def test_visible?
-    assert build(:user, :pending).visible?
-    assert build(:user, :active).visible?
-    assert build(:user, :confirmed).visible?
+    assert_predicate build(:user, :pending), :visible?
+    assert_predicate build(:user, :active), :visible?
+    assert_predicate build(:user, :confirmed), :visible?
     assert_not build(:user, :suspended).visible?
     assert_not build(:user, :deleted).visible?
   end
 
   def test_active?
     assert_not build(:user, :pending).active?
-    assert build(:user, :active).active?
-    assert build(:user, :confirmed).active?
+    assert_predicate build(:user, :active), :active?
+    assert_predicate build(:user, :confirmed), :active?
     assert_not build(:user, :suspended).active?
     assert_not build(:user, :deleted).active?
   end
 
   def test_moderator?
     assert_not create(:user).moderator?
-    assert create(:moderator_user).moderator?
+    assert_predicate create(:moderator_user), :moderator?
   end
 
   def test_administrator?
     assert_not create(:user).administrator?
-    assert create(:administrator_user).administrator?
+    assert_predicate create(:administrator_user), :administrator?
   end
 
   def test_has_role?
@@ -250,7 +250,7 @@ class UserTest < ActiveSupport::TestCase
     user = create(:user, :with_home_location, :description => "foo")
     user.soft_destroy
     assert_equal "user_#{user.id}", user.display_name
-    assert user.description.blank?
+    assert_predicate user.description, :blank?
     assert_nil user.home_lat
     assert_nil user.home_lon
     assert_not user.avatar.attached?
index 5860786c6147d5c4a7e66b32310c40ef5775269e..cb225947c48ca554d8c78e0aa01c0359e1322d7a 100644 (file)
@@ -5,7 +5,7 @@ class WayTagTest < ActiveSupport::TestCase
     tag = create(:way_tag)
     [0, 255].each do |i|
       tag.k = "k" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -13,7 +13,7 @@ class WayTagTest < ActiveSupport::TestCase
     tag = create(:way_tag)
     [0, 255].each do |i|
       tag.v = "v" * i
-      assert tag.valid?
+      assert_predicate tag, :valid?
     end
   end
 
@@ -21,20 +21,20 @@ class WayTagTest < ActiveSupport::TestCase
     tag = create(:way_tag)
     tag.k = "k" * 256
     assert_not tag.valid?, "Key should be too long"
-    assert tag.errors[:k].any?
+    assert_predicate tag.errors[:k], :any?
   end
 
   def test_length_value_invalid
     tag = create(:way_tag)
     tag.v = "v" * 256
     assert_not tag.valid?, "Value should be too long"
-    assert tag.errors[:v].any?
+    assert_predicate tag.errors[:v], :any?
   end
 
   def test_empty_tag_invalid
     tag = WayTag.new
     assert_not tag.valid?, "Empty way tag should be invalid"
-    assert tag.errors[:way].any?
+    assert_predicate tag.errors[:way], :any?
   end
 
   def test_uniqueness
@@ -43,9 +43,9 @@ class WayTagTest < ActiveSupport::TestCase
     tag.way_id = existing.way_id
     tag.k = existing.k
     tag.v = existing.v
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
     assert_not tag.valid?
     assert_raise(ActiveRecord::RecordInvalid) { tag.save! }
-    assert tag.new_record?
+    assert_predicate tag, :new_record?
   end
 end
index 6b166ec7eee0b602c230ca07ccbc65d0eadefa24..420eb42a891ab5d3397942e4b021c0330ad77983 100644 (file)
@@ -27,7 +27,7 @@ class WayTest < ActiveSupport::TestCase
     node_c = create(:node)
     way = create(:way_with_nodes, :nodes_count => 1)
     # Take one of the current ways and add nodes to it until we are near the limit
-    assert way.valid?
+    assert_predicate way, :valid?
     # it already has 1 node
     1.upto(Settings.max_number_of_way_nodes / 2) do
       way.add_nd_num(node_a.id)
@@ -35,9 +35,9 @@ class WayTest < ActiveSupport::TestCase
     end
     way.save
     # print way.nds.size
-    assert way.valid?
+    assert_predicate way, :valid?
     way.add_nd_num(node_c.id)
-    assert way.valid?
+    assert_predicate way, :valid?
   end
 
   def test_from_xml_no_id
index 0800e1541aa5da7e49570894f2ff477cac5922bc..efb1070653de48b766cdaf7c5dc6221cdd5bbf97 100644 (file)
@@ -54,7 +54,7 @@ class ReportDiaryEntryTest < ApplicationSystemTestCase
 
     issue.reload
     assert_not issue.resolved?
-    assert issue.open?
+    assert_predicate issue, :open?
   end
 
   def test_missing_report_params
index e1409a5dc4a0d36dcd5adf6b3add0b197b2c6ecf..341ca4d77f417515853ff7566c9a992231515265 100644 (file)
@@ -23,7 +23,7 @@ class CharactersValidatorTest < ActiveSupport::TestCase
 
     valid.each do |v|
       c.chars = v
-      assert c.valid?, "'#{v}' should be valid"
+      assert_predicate c, :valid?, "'#{v}' should be valid"
     end
   end
 
@@ -48,7 +48,7 @@ class CharactersValidatorTest < ActiveSupport::TestCase
 
     valid.each do |v|
       c.chars = v
-      assert c.valid?, "'#{v}' should be valid"
+      assert_predicate c, :valid?, "'#{v}' should be valid"
     end
   end
 
index c908538d1cb55f449d0576aa80c0e80aa486c510..2607d9f2359b2a966fc42e1763b0caab1fb9d92d 100644 (file)
@@ -33,7 +33,7 @@ class WhitespaceValidatorTest < ActiveSupport::TestCase
 
     strings.each do |v|
       validator.string = v
-      assert validator.valid?, "'#{v}' should be valid"
+      assert_predicate validator, :valid?, "'#{v}' should be valid"
     end
   end
 
@@ -55,7 +55,7 @@ class WhitespaceValidatorTest < ActiveSupport::TestCase
 
     strings.each do |v|
       validator.string = v
-      assert validator.valid?, "'#{v}' should be valid"
+      assert_predicate validator, :valid?, "'#{v}' should be valid"
     end
   end
 end