Removed author_{id, ip} methods from Note model

Removed author_id, author_ip methods from Note model because they are not used anymore. Also, removed associated unit tests.

diff --git a/app/models/note.rb b/app/models/note.rb
index 17b57c0fa689ff5d6691d71b05ce6539a5770688..376516e9b4e8841a0ef9f132102d41ec2fd04c36 100644 (file)
--- a/app/models/note.rb
+++ b/app/models/note.rb
@@ -99,16 +99,6 @@ class Note < ApplicationRecord
     comments.first.author
   end
 
-  # Return the note's author ID, derived from the first comment
-  def author_id
-    comments.first.author_id
-  end
-
-  # Return the note's author IP address, derived from the first comment
-  def author_ip
-    comments.first.author_ip
-  end
-
   private
 
   # Fill in default values for new notes

diff --git a/test/models/note_test.rb b/test/models/note_test.rb
index ba87911e33a8d79be67e2793958bdb5f8c20290b..8b0439ac22dc0e5487f6017e872f6883b50fc4cc 100644 (file)
--- a/test/models/note_test.rb
+++ b/test/models/note_test.rb
@@ -65,23 +65,6 @@ class NoteTest < ActiveSupport::TestCase
     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
-
-    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)