Merge remote-tracking branch 'upstream/pull/5499'

[rails.git] / test / models / note_test.rb

diff --git a/test/models/note_test.rb b/test/models/note_test.rb
index e3831b2c022fd4d8b9f70163cda58cca0ec44df5..ba87911e33a8d79be67e2793958bdb5f8c20290b 100644 (file)
--- a/test/models/note_test.rb
+++ b/test/models/note_test.rb
@@ -1,23 +1,20 @@
-# -*- coding: utf-8 -*-

 require "test_helper"
 
 class NoteTest < ActiveSupport::TestCase
 require "test_helper"
 
 class NoteTest < ActiveSupport::TestCase

-  fixtures :users
-

   def test_status_valid
   def test_status_valid

-    ok = %w(open closed hidden)
-    bad = %w(expropriated fubared)
+    ok = %w[open closed hidden]
+    bad = %w[expropriated fubared]

 
     ok.each do |status|
       note = create(:note)
       note.status = status
 
     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|
       note = create(:note)
       note.status = status
     end
 
     bad.each do |status|
       note = create(:note)
       note.status = status

-      assert !note.valid?, "#{status} is valid when it shouldn't be"
+      assert_not_predicate note, :valid?, "#{status} is valid when it shouldn't be"

     end
   end
 
     end
   end
 


@@ -31,7 +28,7 @@ class NoteTest < ActiveSupport::TestCase
   end
 
   def test_reopen
   end
 
   def test_reopen

-    note = create(:note, :status => "closed", :closed_at => Time.now)
+    note = create(:note, :closed)

     assert_equal "closed", note.status
     assert_not_nil note.closed_at
     note.reopen
     assert_equal "closed", note.status
     assert_not_nil note.closed_at
     note.reopen


@@ -40,14 +37,23 @@ class NoteTest < ActiveSupport::TestCase
   end
 
   def test_visible?
   end
 
   def test_visible?

-    assert_equal true, create(:note, :status => "open").visible?
-    assert_equal true, create(:note, :status => "closed").visible?
-    assert_equal false, create(:note, :status => "hidden").visible?
+    assert_predicate create(:note, :status => "open"), :visible?
+    assert_predicate create(:note, :closed), :visible?
+    assert_not_predicate create(:note, :status => "hidden"), :visible?

   end
 
   def test_closed?
   end
 
   def test_closed?

-    assert_equal true, create(:note, :status => "closed", :closed_at => Time.now).closed?
-    assert_equal false, create(:note, :status => "open", :closed_at => nil).closed?
+    assert_predicate create(:note, :closed), :closed?
+    assert_not_predicate create(:note, :status => "open", :closed_at => nil), :closed?
+  end
+
+  def test_description
+    comment = create(:note_comment)
+    assert_equal comment.body, comment.note.description
+
+    user = create(:user)
+    comment = create(:note_comment, :author => user)
+    assert_equal comment.body, comment.note.description

   end
 
   def test_author
   end
 
   def test_author


@@ -59,6 +65,15 @@ class NoteTest < ActiveSupport::TestCase
     assert_equal user, comment.note.author
   end
 
     assert_equal user, comment.note.author
   end
 

+  def test_author_id
+    comment = create(:note_comment)
+    assert_nil comment.note.author_id
+
+    user = create(:user)
+    comment = create(:note_comment, :author => user)
+    assert_equal user.id, comment.note.author_id
+  end
+

   def test_author_ip
     comment = create(:note_comment)
     assert_nil comment.note.author_ip
   def test_author_ip
     comment = create(:note_comment)
     assert_nil comment.note.author_ip


@@ -66,4 +81,12 @@ class NoteTest < ActiveSupport::TestCase
     comment = create(:note_comment, :author_ip => IPAddr.new("192.168.1.1"))
     assert_equal IPAddr.new("192.168.1.1"), comment.note.author_ip
   end
     comment = create(:note_comment, :author_ip => IPAddr.new("192.168.1.1"))
     assert_equal IPAddr.new("192.168.1.1"), comment.note.author_ip
   end

+
+  # Ensure the lat/lon is formatted as a decimal e.g. not 4.0e-05
+  def test_lat_lon_format
+    note = build(:note, :latitude => 0.00004 * GeoRecord::SCALE, :longitude => 0.00008 * GeoRecord::SCALE)
+
+    assert_equal "0.0000400", note.lat.to_s
+    assert_equal "0.0000800", note.lon.to_s
+  end

 end
 end