Code refactoring

Library/Homebrew/dependency_collector.rb

@@ -166,12 +166,12 @@ class DependencyCollector
   def parse_url_spec(url, tags)
     case File.extname(url)
     when ".xz"          then xz_dep_if_needed(tags)
+    when ".zip"         then zip_dep_if_needed(tags)
+    when ".bz2"         then bzip2_dep_if_needed(tags)
     when ".lha", ".lzh" then Dependency.new("lha", tags)
     when ".lz"          then Dependency.new("lzip", tags)
     when ".rar"         then Dependency.new("unrar", tags)
     when ".7z"          then Dependency.new("p7zip", tags)
-    when ".zip"         then zip_dep_if_needed(tags)
-    when ".bz2"         then bzip2_dep_if_needed(tags)
     end
   end
 end

Library/Homebrew/extend/os/mac/dependency_collector.rb

@@ -18,13 +18,13 @@ class DependencyCollector
     Dependency.new("xz", tags)
   end
 
+  def zip_dep_if_needed(tags); end
+
+  def bzip2_dep_if_needed(tags); end
+
   def ld64_dep_if_needed(*)
     # Tiger's ld is too old to properly link some software
     return if MacOS.version > :tiger
     LD64Dependency.new
   end
-
-  def zip_dep_if_needed(*); end
-
-  def bzip2_dep_if_needed(*); end
 end

Library/Homebrew/test/dependency_collector_spec.rb

@@ -140,4 +140,3 @@ describe DependencyCollector do
     end
   end
 end
-require "test/os/dependency_collector_spec"

Library/Homebrew/test/os/dependency_collector_spec.rb

@@ -1 +1 @@
-require "test/os/linux/dependency_collector" if OS.linux?
+require "test/dependency_collector"