Test coverage for DownloadStrategyDetector

While at it, make it use class methods instead; no reason to instantiate
an object for this.

Eventually there should be some functional tests for the individual
strategies as well.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>

1 files changed, 19 insertions, 0 deletions

diff --git a/Library/Homebrew/test/test_download_strategies.rb b/Library/Homebrew/test/test_download_strategies.rb
new file mode 100644
index 000000000..13392272b
--- /dev/null
+++ b/Library/Homebrew/test/test_download_strategies.rb
@@ -0,0 +1,19 @@
+require 'testing_env'
+require 'download_strategy'
+require 'bottles' # XXX: hoist these regexps into constants in Pathname?
+
+class DownloadStrategyDetectorTests < Test::Unit::TestCase
+  def setup
+    @d = DownloadStrategyDetector.new
+  end
+
+  def test_detect_git_download_startegy
+    @d = DownloadStrategyDetector.detect("git://foo.com/bar.git")
+    assert_equal GitDownloadStrategy, @d
+  end
+
+  def test_default_to_curl_strategy
+    @d = DownloadStrategyDetector.detect(Object.new)
+    assert_equal CurlDownloadStrategy, @d
+  end
+end