Refactor names
This commit is contained in:
@@ -2,7 +2,7 @@ require "mandate"
|
||||
require "propono"
|
||||
|
||||
require "ext/propono"
|
||||
require "builder/build_container"
|
||||
require "builder/build_analyzer"
|
||||
require "builder/listen_for_new_analyzers"
|
||||
require "builder/publish_message"
|
||||
|
||||
|
||||
@@ -1,5 +1,5 @@
|
||||
module Builder
|
||||
class BuildContainer
|
||||
class BuildAnalyzer
|
||||
include Mandate
|
||||
|
||||
initialize_with :track_slug, :tag
|
||||
@@ -7,7 +7,7 @@ module Builder
|
||||
track_slug = message[:track_slug]
|
||||
tag = message[:tag]
|
||||
|
||||
Builder::BuildContainer.(track_slug, tag)
|
||||
Builder::BuildAnalyzer.(track_slug, tag)
|
||||
end
|
||||
end
|
||||
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
require 'test_helper'
|
||||
|
||||
module Builder
|
||||
class BuildContainerTest < Minitest::Test
|
||||
class BuildAnalyzerTest < Minitest::Test
|
||||
|
||||
def test_calls_system_and_propono_with_the_correct_params_and
|
||||
track_slug = "ruby"
|
||||
@@ -13,14 +13,14 @@ module Builder
|
||||
image_name: "exercism-analyzer-#{track_slug}:#{tag}"
|
||||
)
|
||||
|
||||
Builder::BuildContainer.(track_slug, tag)
|
||||
Builder::BuildAnalyzer.(track_slug, tag)
|
||||
end
|
||||
|
||||
def test_does_not_publish_if_kernel_fails
|
||||
Kernel.expects(:system).returns(false)
|
||||
PublishMessage.expects(:call).never
|
||||
|
||||
Builder::BuildContainer.("foo", "v1.3.4")
|
||||
Builder::BuildAnalyzer.("foo", "v1.3.4")
|
||||
end
|
||||
end
|
||||
end
|
||||
@@ -1,7 +1,7 @@
|
||||
require 'test_helper'
|
||||
|
||||
module Builder
|
||||
class BuildContainerTest < Minitest::Test
|
||||
class ListenForNewAnalyzersTest < Minitest::Test
|
||||
|
||||
def test_proxies_message_correctly
|
||||
track_slug = "python"
|
||||
@@ -11,7 +11,7 @@ module Builder
|
||||
propono_client.expects(:listen).with(:analyzer_ready_to_build).yields(message)
|
||||
Propono.expects(:configure_client).returns(propono_client)
|
||||
|
||||
BuildContainer.expects(:call).with(track_slug, tag)
|
||||
BuildAnalyzer.expects(:call).with(track_slug, tag)
|
||||
Builder::ListenForNewAnalyzers.()
|
||||
end
|
||||
end
|
||||
|
||||
Reference in New Issue
Block a user