From 981d391d4d9e65ccf86119ee9b312c2298d785d7 Mon Sep 17 00:00:00 2001 From: Evgeny Latyshev Date: Tue, 26 Sep 2023 21:40:14 +0700 Subject: [PATCH] Fix accessory name check in `with_accessory` --- lib/kamal/cli/accessory.rb | 3 ++- test/cli/accessory_test.rb | 4 ++++ 2 files changed, 6 insertions(+), 1 deletion(-) diff --git a/lib/kamal/cli/accessory.rb b/lib/kamal/cli/accessory.rb index e6bce05f..64cd97cd 100644 --- a/lib/kamal/cli/accessory.rb +++ b/lib/kamal/cli/accessory.rb @@ -226,7 +226,8 @@ class Kamal::Cli::Accessory < Kamal::Cli::Base private def with_accessory(name) - if accessory = KAMAL.accessory(name) + if KAMAL.config.accessory(name) + accessory = KAMAL.accessory(name) yield accessory, accessory_hosts(accessory) else error_on_missing_accessory(name) diff --git a/test/cli/accessory_test.rb b/test/cli/accessory_test.rb index f6e6045b..444b40d2 100644 --- a/test/cli/accessory_test.rb +++ b/test/cli/accessory_test.rb @@ -79,6 +79,10 @@ class CliAccessoryTest < CliTestCase assert_match "docker ps --filter label=service=app-mysql", run_command("details", "mysql") end + test "details with non-existent accessory" do + assert_equal "No accessory by the name of 'hello' (options: mysql and redis)", stderred { run_command("details", "hello") } + end + test "details with all" do run_command("details", "all").tap do |output| assert_match "docker ps --filter label=service=app-mysql", output