From e629f14d56b803ffbc7f3470175f2b021a8816eb Mon Sep 17 00:00:00 2001
From: Jack Nagel
Date: Sun, 13 Jan 2013 19:51:19 -0600
Subject: Remove <=> from Dependency interface

It is important that dep equality corresponds to the name attribute, but
we may want to use the Comparable interface to sort them by installation
order in the future. Code that needs to sort them alphabetically should
just use sort_by.
---
 Library/Homebrew/test/test_dependencies.rb | 43 +++++++++++++++++++++++++-----
 1 file changed, 37 insertions(+), 6 deletions(-)

(limited to 'Library/Homebrew/test')

diff --git a/Library/Homebrew/test/test_dependencies.rb b/Library/Homebrew/test/test_dependencies.rb
index ebfd8ac01..68c4043cb 100644
--- a/Library/Homebrew/test/test_dependencies.rb
+++ b/Library/Homebrew/test/test_dependencies.rb
@@ -75,17 +75,48 @@ end
 class DependableTests < Test::Unit::TestCase
   def setup
     @tags = ["foo", "bar", :build]
-    @deps = Struct.new(:tags).new(@tags)
-    @deps.extend(Dependable)
+    @dep = Struct.new(:tags).new(@tags).extend(Dependable)
   end
 
   def test_options
-    assert_equal %w{--foo --bar}.sort, @deps.options.sort
+    assert_equal %w{--foo --bar}.sort, @dep.options.sort
   end
 
   def test_interrogation
-    assert @deps.build?
-    assert !@deps.optional?
-    assert !@deps.recommended?
+    assert @dep.build?
+    assert !@dep.optional?
+    assert !@dep.recommended?
+  end
+end
+
+class DependencyTests < Test::Unit::TestCase
+  def test_accepts_single_tag
+    dep = Dependency.new("foo", "bar")
+    assert_equal %w{bar}, dep.tags
+  end
+
+  def test_accepts_multiple_tags
+    dep = Dependency.new("foo", %w{bar baz})
+    assert_equal %w{bar baz}.sort, dep.tags.sort
+  end
+
+  def test_preserves_symbol_tags
+    dep = Dependency.new("foo", :build)
+    assert_equal [:build], dep.tags
+  end
+
+  def test_accepts_symbol_and_string_tags
+    dep = Dependency.new("foo", [:build, "bar"])
+    assert_equal [:build, "bar"], dep.tags
+  end
+
+  def test_equality
+    foo1 = Dependency.new("foo")
+    foo2 = Dependency.new("foo")
+    bar = Dependency.new("bar")
+    assert_equal foo1, foo2
+    assert foo1.eql?(foo2)
+    assert_not_equal foo1, bar
+    assert !foo1.eql?(bar)
   end
 end
-- 
cgit v1.2.3