Address CodeRabbit review: fix error logging and test paths

- Change log_info to log_error in completion failure path to avoid
  misleading success marker
- Use paths with spaces in test fixtures to exercise zsh fpath quoting

Author: helizaga

lib/commands/completion.sh

@@ -14,7 +14,7 @@ _completion_asset_path() {
   done
 
   log_error "Could not find $shell completion asset under: $GTR_DIR"
-  log_info "Expected either a source checkout (completions/) or a Homebrew install layout."
+  log_error "Expected either a source checkout (completions/) or a Homebrew install layout."
   return 1
 }
 

tests/completion.bats

@@ -7,14 +7,16 @@ log_error() { printf '%s\n' "$*" >&2; }
 log_info() { printf '%s\n' "$*" >&2; }
 
 setup() {
-  TEST_GTR_DIR="$(mktemp -d)"
+  TEST_ROOT="$(mktemp -d)"
+  TEST_GTR_DIR="$TEST_ROOT/prefix with spaces"
+  mkdir -p "$TEST_GTR_DIR"
   export GTR_DIR="$TEST_GTR_DIR"
   # shellcheck disable=SC1091
   . "$PROJECT_ROOT/lib/commands/completion.sh"
 }
 
 teardown() {
-  rm -rf "$TEST_GTR_DIR"
+  rm -rf "$TEST_ROOT"
 }
 
 write_test_file() {