Use --set-string for testMode to force string type

Change from --set to --set-string for testMode to ensure it's always
a string and has higher precedence over values from GCloud secrets.

Update template to use toString filter and string comparison:
  {{- if or .Values.localDeploy (eq (.Values.testMode | toString) "true") }}

This handles the case where GCloud secrets might set testMode to false
or null, which would not be properly overridden by --set.

Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>

Author: davdhacs

chart/infra-server/templates/secrets.yaml

@@ -116,7 +116,7 @@ data:
 {{ end }}
 {{- end }}
 
-{{- if or .Values.localDeploy .Values.testMode }}
+{{- if or .Values.localDeploy (eq (.Values.testMode | toString) "true") }}
 ---
 apiVersion: v1
 kind: Secret

scripts/deploy/helm.sh

@@ -93,9 +93,10 @@ deploy() {
     fi
 
     # Add dynamic values AFTER --values - so they override GCloud secret values
+    # Use --set-string for testMode to ensure it overrides any value from GCloud secrets
     helm_cmd+=(--set tag="${TAG}")
     helm_cmd+=(--set environment="${ENVIRONMENT}")
-    helm_cmd+=(--set testMode="${TEST_MODE}")
+    helm_cmd+=(--set-string testMode="${TEST_MODE}")
 
     # Add sessionSecret if provided (for local and test deployments)
     if [[ -n "${SESSION_SECRET}" ]]; then