imjasonh/snoop
1
0
Fork 0
mirror of https://github.com/imjasonh/snoop synced 2026-07-18 06:35:10 +00:00
Code Activity

Fix arithmetic increment to work with set -e

Browse source 
((VAR++)) returns the old value (0), which causes set -e to exit.
Changed to VAR=$((VAR + 1)) which always succeeds.
This commit is contained in:
Jason Hall 2026-01-14 14:05:27 -05:00
parent a389697828
commit 849312ec12
1 changed files with 6 additions and 6 deletions
Download patch file Download diff file Expand all files Collapse all files

12
test/kind/run-tests.sh
View file

@ -80,7 +80,7 @@ run_test() {
echo "Deploying test workload to namespace: $test_namespace..."
if ! kubectl apply -f "$temp_manifest" 2>&1 | tee "$RESULTS_DIR/${test_name}-deploy.log"; then
echo -e "${RED}❌ FAILED: Deployment failed${NC}"
((FAILED++))
FAILED=$((FAILED + 1))
FAILED_TESTS+=("$test_name: deployment failed")
return 1
fi
@ -96,7 +96,7 @@ run_test() {
echo ""
echo "App logs:"
kubectl logs -l "app=$app_label" -n "$test_namespace" -c app --tail=20 || echo "(no logs)"
((FAILED++))
FAILED=$((FAILED + 1))
FAILED_TESTS+=("$test_name: pod not ready")
kubectl delete namespace "$test_namespace" --wait=false >/dev/null 2>&1 || true
return 1
@ -136,7 +136,7 @@ run_test() {
echo ""
echo "Checking if report file exists in pod..."
kubectl exec -n "$test_namespace" "$POD_NAME" -c snoop -- ls -la /data/ || echo "(ls failed)"
((FAILED++))
FAILED=$((FAILED + 1))
FAILED_TESTS+=("$test_name: report not found")
kubectl delete namespace "$test_namespace" --wait=false >/dev/null 2>&1 || true
return 1
@ -155,7 +155,7 @@ run_test() {
echo ""
echo "Report content:"
cat "$REPORT_FILE" | jq . || cat "$REPORT_FILE"
((FAILED++))
FAILED=$((FAILED + 1))
FAILED_TESTS+=("$test_name: validation failed")
kubectl delete namespace "$test_namespace" --wait=false >/dev/null 2>&1 || true
return 1
@ -165,7 +165,7 @@ run_test() {
FILE_COUNT=$(jq '.files | length' "$REPORT_FILE")
if [ "$FILE_COUNT" -lt "$expected_min_files" ]; then
echo -e "${RED}❌ FAILED: Too few files captured ($FILE_COUNT < $expected_min_files)${NC}"
((FAILED++))
FAILED=$((FAILED + 1))
FAILED_TESTS+=("$test_name: insufficient files")
kubectl delete namespace "$test_namespace" --wait=false >/dev/null 2>&1 || true
return 1
@ -179,7 +179,7 @@ run_test() {
echo ""
echo "Sample files captured:"
(jq -r '.files[] | " " + .' "$REPORT_FILE" | head -10) || echo " (none shown)"
((PASSED++))
PASSED=$((PASSED + 1))
# Cleanup (async - don't block)
echo "Cleaning up..."