Resolve 34 Rubocop Lint/AmbiguousOperator conflicts

diff --git a/.rubocop_todo.yml b/.rubocop_todo.yml
index e3154ac45491a902f2b37ea051947a67072e555c..653d7e1575bb954e7d885012e589aed088543e78 100644 (file)
--- a/.rubocop_todo.yml
+++ b/.rubocop_todo.yml
@@ -6,14 +6,6 @@
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 
 # Note that changes in the inspected code, or installation of new
 # versions of RuboCop, may require this file to be generated again.
 

-# Offense count: 34
-Lint/AmbiguousOperator:
-  Exclude:
-    - 'test/controllers/amf_controller_test.rb'
-    - 'test/controllers/changeset_controller_test.rb'
-    - 'test/lib/bounding_box_test.rb'
-    - 'test/lib/country_test.rb'
-

 # Offense count: 32
 # Configuration parameters: AllowSafeAssignment.
 Lint/AssignmentInCondition:
 # Offense count: 32
 # Configuration parameters: AllowSafeAssignment.
 Lint/AssignmentInCondition:

diff --git a/test/controllers/amf_controller_test.rb b/test/controllers/amf_controller_test.rb
index b8cfd97cd8ab743ee78a69f6bd8eff6b7ae151ba..ec11b38e89bc33ec3ec7be3b4a8078aa1ac1c356 100644 (file)
--- a/test/controllers/amf_controller_test.rb
+++ b/test/controllers/amf_controller_test.rb
@@ -73,7 +73,7 @@ class AmfControllerTest < ActionController::TestCase
     assert_response :success
     amf_parse_response
     result = amf_result("/1")
     assert_response :success
     amf_parse_response
     result = amf_result("/1")

-    assert_equal -4, result[0]
+    assert_equal(-4, result[0])

     assert_equal "way", result[1]
     assert_equal id, result[2]
     assert(result[3].nil? && result[4].nil? && result[5].nil? && result[6].nil?)
     assert_equal "way", result[1]
     assert_equal id, result[2]
     assert(result[3].nil? && result[4].nil? && result[5].nil? && result[6].nil?)


@@ -154,7 +154,7 @@ class AmfControllerTest < ActionController::TestCase
     assert_response :success
     amf_parse_response
     way = amf_result("/1")
     assert_response :success
     amf_parse_response
     way = amf_result("/1")

-    assert_equal -4, way[0]
+    assert_equal(-4, way[0])

     assert_equal "way", way[1]
     assert_equal 0, way[2]
     assert(way[3].nil?) && way[4].nil? && way[5].nil? && way[6].nil?
     assert_equal "way", way[1]
     assert_equal 0, way[2]
     assert(way[3].nil?) && way[4].nil? && way[5].nil? && way[6].nil?


@@ -370,7 +370,7 @@ class AmfControllerTest < ActionController::TestCase
       assert_response :success
       amf_parse_response
       returned_way = amf_result("/1")
       assert_response :success
       amf_parse_response
       returned_way = amf_result("/1")

-      assert_equal -1, returned_way[0]
+      assert_equal(-1, returned_way[0])

       assert returned_way[3].nil?
       assert returned_way[4].nil?
       assert returned_way[5].nil?
       assert returned_way[3].nil?
       assert returned_way[4].nil?
       assert returned_way[5].nil?


@@ -391,7 +391,7 @@ class AmfControllerTest < ActionController::TestCase
       assert_response :success
       amf_parse_response
       returned_way = amf_result("/1")
       assert_response :success
       amf_parse_response
       returned_way = amf_result("/1")

-      assert_equal -1, returned_way[0]
+      assert_equal(-1, returned_way[0])

       assert returned_way[3].nil?
       assert returned_way[4].nil?
       assert returned_way[5].nil?
       assert returned_way[3].nil?
       assert returned_way[4].nil?
       assert returned_way[5].nil?


@@ -408,7 +408,7 @@ class AmfControllerTest < ActionController::TestCase
       assert_response :success
       amf_parse_response
       returned_way = amf_result("/1")
       assert_response :success
       amf_parse_response
       returned_way = amf_result("/1")

-      assert_equal -1, returned_way[0]
+      assert_equal(-1, returned_way[0])

       assert returned_way[3].nil?
       assert returned_way[4].nil?
       assert returned_way[5].nil?
       assert returned_way[3].nil?
       assert returned_way[4].nil?
       assert returned_way[5].nil?


@@ -498,7 +498,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 2, result.length
     result = amf_result("/1")
 
     assert_equal 2, result.length

-    assert_equal -1, result[0]
+    assert_equal(-1, result[0])

     assert_match(/must be logged in/, result[1])
 
     blocked_user = create(:user)
     assert_match(/must be logged in/, result[1])
 
     blocked_user = create(:user)


@@ -510,7 +510,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 2, result.length
     result = amf_result("/1")
 
     assert_equal 2, result.length

-    assert_equal -1, result[0]
+    assert_equal(-1, result[0])

     assert_match(/access to the API has been blocked/, result[1])
   end
 
     assert_match(/access to the API has been blocked/, result[1])
   end
 


@@ -546,7 +546,7 @@ class AmfControllerTest < ActionController::TestCase
 
     # find by name fails as it uses mysql text search syntax...
     assert_equal 2, result.length
 
     # find by name fails as it uses mysql text search syntax...
     assert_equal 2, result.length

-    assert_equal -2, result[0]
+    assert_equal(-2, result[0])

   end
 
   def test_findrelations_by_id
   end
 
   def test_findrelations_by_id


@@ -636,7 +636,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 3, result.length
     result = amf_result("/1")
 
     assert_equal 3, result.length

-    assert_equal -4, result[0]
+    assert_equal(-4, result[0])

     assert_equal "node", result[1]
     assert_equal 999999, result[2]
   end
     assert_equal "node", result[1]
     assert_equal 999999, result[2]
   end


@@ -671,7 +671,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 3, result.length
     result = amf_result("/1")
 
     assert_equal 3, result.length

-    assert_equal -4, result[0]
+    assert_equal(-4, result[0])

     assert_equal "node", result[1]
     assert_equal node.node_id, result[2]
 
     assert_equal "node", result[1]
     assert_equal node.node_id, result[2]
 


@@ -682,7 +682,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 3, result.length
     result = amf_result("/1")
 
     assert_equal 3, result.length

-    assert_equal -4, result[0]
+    assert_equal(-4, result[0])

     assert_equal "node", result[1]
     assert_equal 999999, result[2]
   end
     assert_equal "node", result[1]
     assert_equal 999999, result[2]
   end


@@ -864,7 +864,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 2, result.size
     result = amf_result("/1")
 
     assert_equal 2, result.size

-    assert_equal -1, result[0], "Expected to get the status FAIL in the amf"
+    assert_equal(-1, result[0], "Expected to get the status FAIL in the amf")

     assert_equal "One of the tags is invalid. Linux users may need to upgrade to Flash Player 10.1.", result[1]
   end
 
     assert_equal "One of the tags is invalid. Linux users may need to upgrade to Flash Player 10.1.", result[1]
   end
 


@@ -904,7 +904,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 3, result.size
     result = amf_result("/1")
 
     assert_equal 3, result.size

-    assert_equal -4, result[0]
+    assert_equal(-4, result[0])

     assert_equal "node", result[1]
     assert_equal nd.id, result[2]
   end
     assert_equal "node", result[1]
     assert_equal nd.id, result[2]
   end


@@ -922,7 +922,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 3, result.size
     result = amf_result("/1")
 
     assert_equal 3, result.size

-    assert_equal -4, result[0]
+    assert_equal(-4, result[0])

     assert_equal "node", result[1]
     assert_equal 999999, result[2]
   end
     assert_equal "node", result[1]
     assert_equal 999999, result[2]
   end


@@ -940,7 +940,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 2, result.size
     result = amf_result("/1")
 
     assert_equal 2, result.size

-    assert_equal -2, result[0]
+    assert_equal(-2, result[0])

     assert_match(/Node is not in the world/, result[1])
   end
 
     assert_match(/Node is not in the world/, result[1])
   end
 


@@ -966,8 +966,8 @@ class AmfControllerTest < ActionController::TestCase
     assert_equal 8, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]
     assert_equal 8, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]

-    assert_equal -1, result[2]
-    assert_not_equal -1, result[3]
+    assert_equal(-1, result[2])
+    assert_not_equal(-1, result[3])

     assert_equal({}, result[4])
     assert_equal 1, result[5]
     assert_equal({}, result[6])
     assert_equal({}, result[4])
     assert_equal 1, result[5]
     assert_equal({}, result[6])


@@ -988,8 +988,8 @@ class AmfControllerTest < ActionController::TestCase
     assert_equal 8, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]
     assert_equal 8, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]

-    assert_equal -1, result[2]
-    assert_not_equal -1, result[3]
+    assert_equal(-1, result[2])
+    assert_not_equal(-1, result[3])

     assert_equal({}, result[4])
     assert_equal 1, result[5]
     assert_equal({}, result[6])
     assert_equal({}, result[4])
     assert_equal 1, result[5]
     assert_equal({}, result[6])


@@ -1011,8 +1011,8 @@ class AmfControllerTest < ActionController::TestCase
     assert_equal 8, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]
     assert_equal 8, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]

-    assert_equal -1, result[2]
-    assert_not_equal -1, result[3]
+    assert_equal(-1, result[2])
+    assert_not_equal(-1, result[3])

     assert_equal({ "-1" => new_node_id }, result[4])
     assert_equal 1, result[5]
     assert_equal({ new_node_id.to_s => 1, d.to_s => 2 }, result[6])
     assert_equal({ "-1" => new_node_id }, result[4])
     assert_equal 1, result[5]
     assert_equal({ new_node_id.to_s => 1, d.to_s => 2 }, result[6])


@@ -1193,7 +1193,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 2, result.size
     result = amf_result("/1")
 
     assert_equal 2, result.size

-    assert_equal -1, result[0]
+    assert_equal(-1, result[0])

     assert_match(/Way #{way.id} is still used/, result[1])
 
     new_way = Way.find(way.id)
     assert_match(/Way #{way.id} is still used/, result[1])
 
     new_way = Way.find(way.id)


@@ -1225,8 +1225,8 @@ class AmfControllerTest < ActionController::TestCase
     assert_equal 5, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]
     assert_equal 5, result.size
     assert_equal 0, result[0]
     assert_equal "", result[1]

-    assert_equal -1, result[2]
-    assert_not_equal -1, result[3]
+    assert_equal(-1, result[2])
+    assert_not_equal(-1, result[3])

     assert_equal 1, result[4]
 
     new_relation = Relation.find(new_relation_id)
     assert_equal 1, result[4]
 
     new_relation = Relation.find(new_relation_id)


@@ -1307,7 +1307,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 2, result.size
     result = amf_result("/1")
 
     assert_equal 2, result.size

-    assert_equal -1, result[0]
+    assert_equal(-1, result[0])

     assert_match(/relation #{relation.id} is used in/, result[1])
 
     new_relation = Relation.find(relation.id)
     assert_match(/relation #{relation.id} is used in/, result[1])
 
     new_relation = Relation.find(relation.id)


@@ -1404,7 +1404,7 @@ class AmfControllerTest < ActionController::TestCase
     result = amf_result("/1")
 
     assert_equal 2, result.size
     result = amf_result("/1")
 
     assert_equal 2, result.size

-    assert_equal -2, result[0]
+    assert_equal(-2, result[0])

     assert_equal "The user doesn't own that changeset", result[1]
 
     cs = Changeset.find(cs_id)
     assert_equal "The user doesn't own that changeset", result[1]
 
     cs = Changeset.find(cs_id)

diff --git a/test/controllers/changeset_controller_test.rb b/test/controllers/changeset_controller_test.rb
index 40147ad467294df36933cfffc9cbf4551109b143..2886780a06d46f370e20f531bba5f07918d47cc0 100644 (file)
--- a/test/controllers/changeset_controller_test.rb
+++ b/test/controllers/changeset_controller_test.rb
@@ -469,9 +469,9 @@ CHANGESET
     new_rel_id = doc.find("//diffResult/relation").first["new_id"].to_i
 
     # check the old IDs are all present and negative one
     new_rel_id = doc.find("//diffResult/relation").first["new_id"].to_i
 
     # check the old IDs are all present and negative one

-    assert_equal -1, doc.find("//diffResult/node").first["old_id"].to_i
-    assert_equal -1, doc.find("//diffResult/way").first["old_id"].to_i
-    assert_equal -1, doc.find("//diffResult/relation").first["old_id"].to_i
+    assert_equal(-1, doc.find("//diffResult/node").first["old_id"].to_i)
+    assert_equal(-1, doc.find("//diffResult/way").first["old_id"].to_i)
+    assert_equal(-1, doc.find("//diffResult/relation").first["old_id"].to_i)

 
     # check the versions are present and equal one
     assert_equal 1, doc.find("//diffResult/node").first["new_version"].to_i
 
     # check the versions are present and equal one
     assert_equal 1, doc.find("//diffResult/node").first["new_version"].to_i

diff --git a/test/lib/bounding_box_test.rb b/test/lib/bounding_box_test.rb
index 46ff38c9ade19cd597cb0d08b4a9a135f9022a33..a29711b0b445dd531b485c61c8b13b985a33a344 100644 (file)
--- a/test/lib/bounding_box_test.rb
+++ b/test/lib/bounding_box_test.rb
@@ -168,8 +168,8 @@ class BoundingBoxTest < ActiveSupport::TestCase
     @bad_negative_boundary_bbox.each do |bbox_string|
       bbox = BoundingBox.from_bbox_params(:bbox => bbox_string)
       array = bbox.to_a
     @bad_negative_boundary_bbox.each do |bbox_string|
       bbox = BoundingBox.from_bbox_params(:bbox => bbox_string)
       array = bbox.to_a

-      assert_equal -180, [array[0], array[2]].min
-      assert_equal -90, [array[1], array[3]].min
+      assert_equal(-180, [array[0], array[2]].min)
+      assert_equal(-90, [array[1], array[3]].min)

     end
   end
 
     end
   end
 

diff --git a/test/lib/country_test.rb b/test/lib/country_test.rb
index 0e21a9ed8439db7541d3bd9bd63c4e5f2d4d2e39..a6fc7c144d42f6f65f51d1a06b2eaeb0b4bf5876 100644 (file)
--- a/test/lib/country_test.rb
+++ b/test/lib/country_test.rb
@@ -5,7 +5,7 @@ class CountryTest < ActiveSupport::TestCase
     gb = Country.find("GB")
     assert_not_nil gb
     assert_equal "GB", gb.code
     gb = Country.find("GB")
     assert_not_nil gb
     assert_equal "GB", gb.code

-    assert_equal -8.623555, gb.min_lon
+    assert_equal(-8.623555, gb.min_lon)

     assert_equal 59.360249, gb.max_lat
     assert_equal 1.759, gb.max_lon
     assert_equal 49.906193, gb.min_lat
     assert_equal 59.360249, gb.max_lat
     assert_equal 1.759, gb.max_lon
     assert_equal 49.906193, gb.min_lat


@@ -16,9 +16,9 @@ class CountryTest < ActiveSupport::TestCase
     assert_not_nil au
     assert_equal "AU", au.code
     assert_equal 112.911057, au.min_lon
     assert_not_nil au
     assert_equal "AU", au.code
     assert_equal 112.911057, au.min_lon

-    assert_equal -10.062805, au.max_lat
+    assert_equal(-10.062805, au.max_lat)

     assert_equal 153.639252, au.max_lon
     assert_equal 153.639252, au.max_lon

-    assert_equal -43.64397, au.min_lat
+    assert_equal(-43.64397, au.min_lat)

   end
 
   def test_xx
   end
 
   def test_xx