Merge pull request #381 from erlliam/master

HarlemSquirrel · web-flow · commit 8c4b201ef03b · 2020-10-28T23:40:01.000-04:00
Implement '==' operator for entries
diff --git a/lib/net/ldap/entry.rb b/lib/net/ldap/entry.rb
@@ -196,4 +196,8 @@ def setter?(sym)
     sym.to_s[-1] == ?=
   end
   private :setter?
+
+  def ==(other)
+    other.instance_of?(self.class) && @myhash == other.to_h
+  end
 end # class Entry
diff --git a/test/test_entry.rb b/test/test_entry.rb
@@ -54,6 +54,17 @@ def test_to_h
     duplicate.delete(:sn)
     assert_not_equal duplicate, @entry.to_h
   end
+
+  def test_equal_operator
+    entry_two = Net::LDAP::Entry.new 'cn=Barbara,o=corp'
+    assert_equal @entry, entry_two
+
+    @entry['sn'] = 'Jensen'
+    assert_not_equal @entry, entry_two
+
+    entry_two['sn'] = 'Jensen'
+    assert_equal @entry, entry_two
+  end
 end
 
 class TestEntryLDIF < Test::Unit::TestCase
