Fix some ruby 3 compatibility issues

openstreetmap · Apr 22, 2021 · 70d7d8d · 70d7d8d
1 parent 816a245
commit 70d7d8d
Show file tree

Hide file tree

Showing 4 changed files with 14 additions and 14 deletions.
diff --git a/test/models/diary_entry_test.rb b/test/models/diary_entry_test.rb
@@ -9,16 +9,16 @@ def setup
   def test_diary_entry_validations
     diary_entry_valid({})
     diary_entry_valid({ :title => "" }, :valid => false)
-    diary_entry_valid(:title => "a" * 255)
+    diary_entry_valid({ :title => "a" * 255 })
     diary_entry_valid({ :title => "a" * 256 }, :valid => false)
     diary_entry_valid({ :body => "" }, :valid => false)
-    diary_entry_valid(:latitude => 90)
+    diary_entry_valid({ :latitude => 90 })
     diary_entry_valid({ :latitude => 90.00001 }, :valid => false)
-    diary_entry_valid(:latitude => -90)
+    diary_entry_valid({ :latitude => -90 })
     diary_entry_valid({ :latitude => -90.00001 }, :valid => false)
-    diary_entry_valid(:longitude => 180)
+    diary_entry_valid({ :longitude => 180 })
     diary_entry_valid({ :longitude => 180.00001 }, :valid => false)
-    diary_entry_valid(:longitude => -180)
+    diary_entry_valid({ :longitude => -180 })
     diary_entry_valid({ :longitude => -180.00001 }, :valid => false)
   end
 

diff --git a/test/models/trace_test.rb b/test/models/trace_test.rb
@@ -77,15 +77,15 @@ def test_tagged
   def test_validations
     trace_valid({})
     trace_valid({ :user_id => nil }, :valid => false)
-    trace_valid(:name => "a" * 255)
+    trace_valid({ :name => "a" * 255 })
     trace_valid({ :name => "a" * 256 }, :valid => false)
     trace_valid({ :description => nil }, :valid => false)
-    trace_valid(:description => "a" * 255)
+    trace_valid({ :description => "a" * 255 })
     trace_valid({ :description => "a" * 256 }, :valid => false)
-    trace_valid(:visibility => "private")
-    trace_valid(:visibility => "public")
-    trace_valid(:visibility => "trackable")
-    trace_valid(:visibility => "identifiable")
+    trace_valid({ :visibility => "private" })
+    trace_valid({ :visibility => "public" })
+    trace_valid({ :visibility => "trackable" })
+    trace_valid({ :visibility => "identifiable" })
     trace_valid({ :visibility => "foo" }, :valid => false)
   end
 

diff --git a/test/models/tracetag_test.rb b/test/models/tracetag_test.rb
@@ -5,8 +5,8 @@ def test_validations
     tracetag_valid({})
     tracetag_valid({ :tag => nil }, :valid => false)
     tracetag_valid({ :tag => "" }, :valid => false)
-    tracetag_valid(:tag => "a")
-    tracetag_valid(:tag => "a" * 255)
+    tracetag_valid({ :tag => "a" })
+    tracetag_valid({ :tag => "a" * 255 })
     tracetag_valid({ :tag => "a" * 256 }, :valid => false)
     tracetag_valid({ :tag => "a/b" }, :valid => false)
     tracetag_valid({ :tag => "a;b" }, :valid => false)

diff --git a/test/test_helper.rb b/test/test_helper.rb
@@ -160,7 +160,7 @@ def signed_request(method, uri, options = {})
 
       request.sign!(oauth)
 
-      method(method).call(request.signed_uri, options)
+      method(method).call(request.signed_uri, **options)
     end
 
     ##