tests: add tests for removing Builder instances

Author: tomowatt

hooks/pre-exit

@@ -38,7 +38,7 @@ if [[ "${builder_remove}" == true ]]; then
       builder_remove_args+=("--keep-state")
     fi
 
-    echo "~~~ :docker: Cleaning up Builder Instance '${builder_name}"
+    echo "~~~ :docker: Cleaning up Builder Instance '${builder_name}'"
     docker buildx stop "${builder_name}"
     docker buildx rm "${builder_name}" ${builder_remove_args:+${builder_remove_args[@]}}
   else

tests/builder-instances.bats

@@ -116,3 +116,72 @@ teardown() {
     assert_success
     assert_output "~~~ :docker: Using Builder Instance '$BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_NAME'"
 }
+
+@test "Remove Builder Instance that Exists" {
+    export BUILDKITE_BUILD_NUMBER=111
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_USE=true
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_NAME=builder-name
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_REMOVE=true
+
+    stub docker \
+        "buildx inspect builder-name : exit 0" \
+        "buildx stop builder-name : exit 0" \
+        "buildx rm builder-name : exit 0"
+
+    run "$PWD"/hooks/pre-exit
+
+    assert_success
+    assert_output "~~~ :docker: Cleaning up Builder Instance 'builder-name'"
+}
+
+@test "Remove Builder Instance that Exists with keep-daemon" {
+    export BUILDKITE_BUILD_NUMBER=111
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_USE=true
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_NAME=builder-name
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_REMOVE=true
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_KEEP_DAEMON=true
+
+    stub docker \
+        "buildx inspect builder-name : exit 0" \
+        "buildx stop builder-name : exit 0" \
+        "buildx rm builder-name --keep-daemon : exit 0"
+
+    run "$PWD"/hooks/pre-exit
+
+    assert_success
+    assert_output "~~~ :docker: Cleaning up Builder Instance 'builder-name'"
+}
+
+@test "Remove Builder Instance that Exists with keep-daemon and keep-state" {
+    export BUILDKITE_BUILD_NUMBER=111
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_USE=true
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_NAME=builder-name
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_REMOVE=true
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_KEEP_DAEMON=true
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_KEEP_STATE=true
+
+    stub docker \
+        "buildx inspect builder-name : exit 0" \
+        "buildx stop builder-name : exit 0" \
+        "buildx rm builder-name --keep-daemon --keep-state : exit 0"
+
+    run "$PWD"/hooks/pre-exit
+
+    assert_success
+    assert_output "~~~ :docker: Cleaning up Builder Instance 'builder-name'"
+}
+
+@test "Remove Builder Instance that does not Exists" {
+    export BUILDKITE_BUILD_NUMBER=111
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_USE=true
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_NAME=builder-name
+    export BUILDKITE_PLUGIN_DOCKER_COMPOSE_BUILDER_REMOVE=true
+
+    stub docker \
+        "buildx inspect builder-name : exit 1"
+
+    run "$PWD"/hooks/pre-exit
+
+    assert_success
+    assert_output "~~~ :docker: Cannot remove Builder Instance 'builder-name' as does not exist"
+}