Fixup tests, refactor args

bin/client.rb

@@ -5,9 +5,10 @@ require 'securerandom'
 
 class PipelineClient
 
-  attr_reader :context, :socket
+  attr_reader :address, :context, :socket
 
-  def initialize
+  def initialize(address="tcp://localhost:5555")
+    @address = address
     @context = ZMQ::Context.new(1)
     open_socket
     at_exit do
@@ -18,7 +19,7 @@ class PipelineClient
   def open_socket
     @socket = context.socket(ZMQ::REQ)
     @socket.setsockopt(ZMQ::LINGER, 0)
-    @socket.connect("tcp://localhost:5555")
+    @socket.connect(address)
   end
 
   def close_socket

lib/pipeline/rpc/worker/analyze_action.rb

@@ -15,7 +15,7 @@ module Pipeline::Rpc::Worker
     end
 
     def prepare_folder(iteration_folder)
-      location = @request["iteration_folder"]
+      location = @request["s3_uri"]
       location_uri = URI(location)
       bucket = location_uri.host
       path = location_uri.path[1..-1]

lib/pipeline/rpc/worker/container_action.rb

@@ -21,7 +21,7 @@ module Pipeline::Rpc::Worker
 
       language_slug = request["track_slug"]
       exercise_slug = request["exercise_slug"]
-      solution_slug = request["solution_slug"]
+      job_slug = request["id"]
       container_version = request["container_version"]
 
       unless environment.released?(language_slug, container_version)

test/pipeline/build/analyzer_build_test.rb

@@ -5,7 +5,8 @@ module Pipeline::Build
   class AnalyzerBuildTest < Minitest::Test
 
     def setup
-      @cmd = AnalyzerBuild.new("v0.1.1", "demotrack")
+      @repo = mock()
+      @cmd = AnalyzerBuild.new("v0.1.1", "demotrack", @repo)
     end
 
     def test_call_invokes_each_phase

test/pipeline/util/img_wrapper_test.rb

@@ -5,7 +5,8 @@ module Pipeline::Util
   class ImgWrapperTest < Minitest::Test
 
     def setup
-      @img = ImgWrapper.new
+      @logs = LogCollector.new
+      @img = ImgWrapper.new @logs
       @img.binary_path = "/path/to/img"
     end