scriptoriumadmin/skillhub
1
0
Fork 0
mirror of https://github.com/iflytek/skillhub.git synced 2026-11-14 09:01:16 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 5

test(chart): pass analytics JSON through a values file

Browse source 
Helm --set-string parses {...} as a list, so the enabled-analytics
contract case failed schema validation in CI. Use a values file, the
same way the operator guide documents the setting.

Refs #989

Signed-off-by: 千乘妍 (Xiaoyaner) <258399167+xiaoyaner0201@users.noreply.github.com>
This commit is contained in:
千乘妍 (Xiaoyaner) 2026-10-30 00:25:38 +08:00
parent 7eab5c6efb
commit 6cee2fedd5
1 changed files with 9 additions and 3 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

12
charts/skillhub/tests/configuration-contracts.sh
View file

@ -476,9 +476,15 @@ assert_rejected subpath-reserved-analytics-nested --set-string web.basePath=/_sk
render analytics-default "$CHART_DIR" >"$TMP_DIR/analytics-default.yaml"
grep -Fq 'web-analytics-config: ""' "$TMP_DIR/analytics-default.yaml" || fail "analytics must default off"
render analytics-enabled "$CHART_DIR" \
--set-string web.analytics.config='{"enabled":true}' \
--set-string web.analytics.upstream=https://stats.example.com >"$TMP_DIR/analytics-enabled.yaml"
# --set treats {...} as a list, so pass the JSON string through a values file,
# the same way the operator guide documents it.
cat >"$TMP_DIR/analytics-values.yaml" <<'VALUES'
web:
analytics:
config: '{"enabled":true,"adapter":"umami","consent":"granted","adapterConfig":{"websiteId":"4398de8e-40df-4483-9eea-ed213643525f"}}'
upstream: https://stats.example.com
VALUES
render analytics-enabled "$CHART_DIR" -f "$TMP_DIR/analytics-values.yaml" >"$TMP_DIR/analytics-enabled.yaml"
for key in SKILLHUB_WEB_ANALYTICS_CONFIG SKILLHUB_WEB_ANALYTICS_UPSTREAM; do
grep -Fq "name: $key" "$TMP_DIR/analytics-enabled.yaml" || fail "missing analytics environment"
done