admin/brew
1
0
Fork 0
Code Issues Packages Projects Releases 6 Wiki Activity

Fix Requirement equality

Browse Source
This commit is contained in:
Jack Nagel 2014-11-12 17:35:16 -06:00
parent 33dfc8e8b8
commit 9dfc5e39e8
2 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

3
Library/Homebrew/requirement.rb
View File

@ -75,9 +75,10 @@ class Requirement
self.class.env_proc self.class.env_proc
end end
def eql?(other) def ==(other)
instance_of?(other.class) && name == other.name && tags == other.tags instance_of?(other.class) && name == other.name && tags == other.tags
end end
alias_method :eql?, :==
def hash def hash
name.hash ^ tags.hash name.hash ^ tags.hash

2
Library/Homebrew/test/test_requirement.rb
View File

@ -129,12 +129,14 @@ class RequirementTests < Homebrew::TestCase
def test_eql def test_eql
a, b = Requirement.new, Requirement.new a, b = Requirement.new, Requirement.new
assert_equal a, b
assert_eql a, b assert_eql a, b
assert_equal a.hash, b.hash assert_equal a.hash, b.hash
end end
def test_not_eql def test_not_eql
a, b = Requirement.new([:optional]), Requirement.new a, b = Requirement.new([:optional]), Requirement.new
refute_equal a, b
refute_eql a, b refute_eql a, b
refute_equal a.hash, b.hash refute_equal a.hash, b.hash
end end