fix: correct tests and handle a few edge cases in remote command usage

contrib/dokku_client.sh

@@ -168,11 +168,18 @@ main() {
     remote:remove)
       shift 1
       git remote remove "$@"
-      exit "$?"
+      local exit_code="$?"
+      if [[ "$(git config dokku.remote)" == "$1" ]]; then
+        git config --unset dokku.remote
+      fi
+      exit "$exit_code"
       ;;
     remote:unset)
-      git config --unset dokku.remote
-      exit "$?"
+      if [[ -n "$(git config dokku.remote)" ]]; then
+        git config --unset dokku.remote
+        exit "$?"
+      fi
+      exit 0
       ;;
   esac
 

tests/unit/client.bats

@@ -265,13 +265,12 @@ teardown() {
   echo "output: $output"
   echo "status: $status"
   assert_success
-  assert_output "dokku2"
 
   run /bin/bash -c "${BATS_TEST_DIRNAME}/../../contrib/dokku_client.sh remote:list"
   echo "output: $output"
   echo "status: $status"
   assert_success
-  assert_output "dokku\dokku2"
+  assert_output "$(printf "dokku\ndokku2")"
 
   run /bin/bash -c "${BATS_TEST_DIRNAME}/../../contrib/dokku_client.sh remote:remove dokku2"
   echo "output: $output"
@@ -289,6 +288,11 @@ teardown() {
   echo "status: $status"
   assert_success
 
+  run /bin/bash -c "${BATS_TEST_DIRNAME}/../../contrib/dokku_client.sh remote:unset"
+  echo "output: $output"
+  echo "status: $status"
+  assert_success
+
   run /bin/bash -c "${BATS_TEST_DIRNAME}/../../contrib/dokku_client.sh remote"
   echo "output: $output"
   echo "status: $status"