87 lines
3.2 KiB
Bash
Executable File
87 lines
3.2 KiB
Bash
Executable File
#!/usr/bin/env bash
|
|
|
|
# Test for issue #2103: HTTP remote cache key should include query parameters.
|
|
# Without the fix, two URLs differing only in ?ref= share the same cache and
|
|
# the second fetch silently returns stale content from the first.
|
|
|
|
issue_2103_input_dir="${cases_dir}/issue-2103/input"
|
|
issue_2103_tmp_dir=$(mktemp -d)
|
|
|
|
cleanup_issue_2103() {
|
|
kill "${server_pid}" 2>/dev/null || true
|
|
rm -rf "${issue_2103_tmp_dir}"
|
|
unset HELMFILE_CACHE_HOME HTTP_SERVER_URL
|
|
}
|
|
trap cleanup_issue_2103 EXIT
|
|
|
|
test_start "issue-2103: HTTP cache key includes query params"
|
|
|
|
# --- Start a small HTTP server that returns different YAML per ?ref= ----------
|
|
|
|
info "Building test HTTP server"
|
|
go build -o "${issue_2103_tmp_dir}/server" "${issue_2103_input_dir}/server.go" \
|
|
|| fail "Could not build test HTTP server"
|
|
|
|
"${issue_2103_tmp_dir}/server" > "${issue_2103_tmp_dir}/server_addr.txt" &
|
|
server_pid=$!
|
|
|
|
# Poll until the server writes its address (up to 10 seconds: 20 x 0.5s)
|
|
for i in $(seq 1 20); do
|
|
if ! kill -0 "${server_pid}" 2>/dev/null; then
|
|
fail "Test HTTP server failed to start"
|
|
fi
|
|
if [ -s "${issue_2103_tmp_dir}/server_addr.txt" ]; then
|
|
break
|
|
fi
|
|
sleep 0.5
|
|
done
|
|
|
|
if [ ! -s "${issue_2103_tmp_dir}/server_addr.txt" ]; then
|
|
fail "Test HTTP server did not write its address in time"
|
|
fi
|
|
|
|
server_url=$(cat "${issue_2103_tmp_dir}/server_addr.txt")
|
|
info "Test HTTP server running at ${server_url}"
|
|
|
|
# --- Fetch remote values through helmfile and verify cache has both refs ------
|
|
|
|
export HTTP_SERVER_URL="${server_url}"
|
|
export TEST_NS="${test_ns}"
|
|
export HELMFILE_CACHE_HOME="${issue_2103_tmp_dir}/cache"
|
|
|
|
info "Running helmfile template with two releases using different ?ref= values"
|
|
${helmfile} -f "${issue_2103_input_dir}/helmfile.yaml.gotmpl" template \
|
|
> "${issue_2103_tmp_dir}/template_output.txt" 2>&1 || {
|
|
helmfile_exit_code=$?
|
|
info "helmfile template output:"
|
|
cat "${issue_2103_tmp_dir}/template_output.txt"
|
|
fail "helmfile template failed with exit code ${helmfile_exit_code}"
|
|
}
|
|
|
|
# Verify that two distinct cache directories were created for the two refs.
|
|
# With the fix, the cache key includes the query params so each ref gets its own dir.
|
|
# Without the fix, both refs would share one cache dir and the second would be stale.
|
|
info "Checking cached files for distinct ref values"
|
|
|
|
cached_commit1=$(find "${issue_2103_tmp_dir}/cache" -type f -name "raw" -exec grep -l "version: commit1" {} \; || true)
|
|
cached_commit2=$(find "${issue_2103_tmp_dir}/cache" -type f -name "raw" -exec grep -l "version: commit2" {} \; || true)
|
|
|
|
if [ -n "${cached_commit1}" ] && [ -n "${cached_commit2}" ]; then
|
|
info "Found separate cached files:"
|
|
info " commit1: ${cached_commit1}"
|
|
info " commit2: ${cached_commit2}"
|
|
if [ "${cached_commit1}" = "${cached_commit2}" ]; then
|
|
fail "Issue #2103 regression: both refs cached to the same file"
|
|
fi
|
|
info "Cache keys are distinct — fix is working"
|
|
else
|
|
info "Cache directory contents:"
|
|
find "${issue_2103_tmp_dir}/cache" -type f 2>/dev/null
|
|
info "Template output:"
|
|
cat "${issue_2103_tmp_dir}/template_output.txt"
|
|
fail "Issue #2103 regression: query-param-specific values were not cached separately"
|
|
fi
|
|
|
|
trap - EXIT
|
|
test_pass "issue-2103: HTTP cache key includes query params"
|