move version_line into GemVersion

Author: indirect

lib/compact_index.rb

@@ -69,38 +69,9 @@ def self.versions(versions_file, gems = nil, args = {})
   #   1.0.1 requirement:<2.0&>1.0|checksum:abc1
   #   1.0.2 requirement:<2.0&>1.0,requirement2:=1.1|checksum:abc2,ruby:>1.0,rubygems:>2.0
   #   ```
-  def self.info(params)
-    output = String.new("---\n")
-    params.each do |version|
-      output << version_line(version) << "\n"
+  def self.info(versions)
+    versions.inject("---\n") do |output, version|
+      output << version.to_line << "\n"
     end
-    output
-  end
-
-private
-
-  def self.version_line(version)
-    if version[:dependencies]
-      version[:dependencies]
-      deps = version[:dependencies].map do |d|
-        [
-          d[:gem],
-          d.version_and_platform.split(", ").sort.join("&")
-        ].join(":")
-      end
-    else
-      deps = []
-    end
-
-    line = version.number_and_platform.dup
-    line << " "
-    line << deps.join(",")
-    line << "|"
-
-    line << "checksum:#{version[:checksum]}"
-    line << ",ruby:#{version[:ruby_version]}" if version[:ruby_version]
-    line << ",rubygems:#{version[:rubygems_version]}" if version[:rubygems_version]
-
-    line
   end
 end

lib/compact_index/gem_version.rb

@@ -9,5 +9,22 @@ def number_and_platform
         "#{number}-#{platform}"
       end
     end
+
+    def to_line
+      line = number_and_platform << " " << deps_line << "|checksum:#{checksum}"
+      line << ",ruby:#{ruby_version}" if ruby_version && ruby_version != ">= 0"
+      line << ",rubygems:#{rubygems_version}" if rubygems_version && rubygems_version != ">= 0"
+      line
+    end
+
+  private
+
+    def deps_line
+      return "" if dependencies.nil?
+      dependencies.map do |d|
+        [d[:gem], d.version_and_platform.split(", ").sort.join("&")].join(":")
+      end.join(",")
+    end
+
   end
 end