From: Tom Hughes <tom@compton.nu>
Date: Wed, 22 Mar 2017 20:43:39 +0000 (+0000)
Subject: Add explicit tests for lat/lon methods on node models
X-Git-Tag: live~3452
X-Git-Url: https://git.openstreetmap.org/rails.git/commitdiff_plain/8a565061e3b0cb76101a9ec42315fa0474811b54?hp=9ae3df1a785fb2e71c7431fc69a1c2ac4204ba8c

Add explicit tests for lat/lon methods on node models
---

diff --git a/test/models/node_test.rb b/test/models/node_test.rb
index 6a19e242f..4384c6287 100644
--- a/test/models/node_test.rb
+++ b/test/models/node_test.rb
@@ -48,6 +48,19 @@ class NodeTest < ActiveSupport::TestCase
     assert_equal false, node.valid?
   end
 
+  def test_lat_lon
+    node = build(:node, :latitude => 12.345 * OldNode::SCALE, :longitude => 34.567 * OldNode::SCALE)
+
+    assert_in_delta 12.345, node.lat, 0.0000001
+    assert_in_delta 34.567, node.lon, 0.0000001
+
+    node.lat = 54.321
+    node.lon = 76.543
+
+    assert_in_delta 54.321 * OldNode::SCALE, node.latitude, 0.000001
+    assert_in_delta 76.543 * OldNode::SCALE, node.longitude, 0.000001
+  end
+
   # Check that you can create a node and store it
   def test_create
     changeset = create(:changeset)
diff --git a/test/models/old_node_test.rb b/test/models/old_node_test.rb
index 3fa4bac3b..8b0540957 100644
--- a/test/models/old_node_test.rb
+++ b/test/models/old_node_test.rb
@@ -46,6 +46,19 @@ class OldNodeTest < ActiveSupport::TestCase
     assert_equal false, node.valid?
   end
 
+  def test_lat_lon
+    node = build(:old_node, :latitude => 12.345 * OldNode::SCALE, :longitude => 34.567 * OldNode::SCALE)
+
+    assert_in_delta 12.345, node.lat, 0.0000001
+    assert_in_delta 34.567, node.lon, 0.0000001
+
+    node.lat = 54.321
+    node.lon = 76.543
+
+    assert_in_delta 54.321 * OldNode::SCALE, node.latitude, 0.000001
+    assert_in_delta 76.543 * OldNode::SCALE, node.longitude, 0.000001
+  end
+
   def test_node_tags
     node_v1 = create(:old_node, :version => 1)
     node_v2 = create(:old_node, :node_id => node_v1.node_id, :version => 2)