fix(helm): fix querydoc helm unittests and include them in CI workflow (#1769)

## Overview

Fix broken querydoc helm-unittest tests and add them to CI.
The tests were introduced in PR #599 (Helm refactor) but never added to
CI, so subsequent breakage from PR #683 (Official KMCP Support) went
undetected.

close #1768 

## Changes

### Fix querydoc deployment test

- Include `configmap.yaml` as rendered template during unittest so that
`include` function for a checksum annotation works. Also added
`template: deployment.yaml` directives required when multiple templates
are listed.
- Fixed the value`replicaCount` → `replicas` to match the actual key
used in deployment manifest.
- Updated the image tag to `1.1.10` in unittest to match the AppVersion
of the chart.

### Remove obsolete toolserver test

`toolserver_test.yaml` referenced `templates/toolserver.yaml` which was
deleted in PR #683 when ToolServer CRDs were replaced by Kubernetes
Services with `kagent.dev/mcp-service` labels.

### Add querydoc to CI helm-unit-tests job

Added `helm unittest helm/tools/querydoc` to the `helm-unit-tests` job
in `.github/workflows/ci.yaml` so future breakage is caught
automatically.

---------

Signed-off-by: TOMOFUMI-KONDO <ugax2kontomo0314@gmail.com>

Author: TOMOFUMI-KONDO

.github/workflows/ci.yaml

@@ -202,6 +202,7 @@ jobs:
       - name: Run helm unit tests
         run: |
           helm unittest helm/kagent
+          helm unittest helm/tools/querydoc
 
   ui-tests:
     runs-on: ubuntu-latest

helm/tools/querydoc/tests/deployment_test.yaml

@@ -1,8 +1,10 @@
 suite: test querydoc deployment
 templates:
   - deployment.yaml
+  - configmap.yaml
 tests:
   - it: should render deployment with default values
+    template: deployment.yaml
     asserts:
       - isKind:
           of: Deployment
@@ -16,23 +18,26 @@ tests:
           count: 1
 
   - it: should render deployment with custom replica count
+    template: deployment.yaml
     set:
-      replicaCount: 2
+      replicas: 2
     asserts:
       - equal:
           path: spec.replicas
           value: 2
 
   - it: should have correct container image
+    template: deployment.yaml
     asserts:
       - equal:
           path: spec.template.spec.containers[0].name
           value: querydoc
       - equal:
           path: spec.template.spec.containers[0].image
-          value: ghcr.io/kagent-dev/doc2vec/mcp:1.1.7
+          value: ghcr.io/kagent-dev/doc2vec/mcp:1.1.10
 
   - it: should use custom image tag when set
+    template: deployment.yaml
     set:
       image:
         tag: "v2.0.0"
@@ -42,18 +47,21 @@ tests:
           value: ghcr.io/kagent-dev/doc2vec/mcp:v2.0.0
 
   - it: should have correct service account name
+    template: deployment.yaml
     asserts:
       - equal:
           path: spec.template.spec.serviceAccountName
           value: RELEASE-NAME-querydoc
 
   - it: should have correct container port
+    template: deployment.yaml
     asserts:
       - equal:
           path: spec.template.spec.containers[0].ports[0].containerPort
           value: 8080
 
   - it: should have correct labels
+    template: deployment.yaml
     asserts:
       - equal:
           path: metadata.labels["app.kubernetes.io/name"]
@@ -69,11 +77,13 @@ tests:
           value: RELEASE-NAME
 
   - it: should not set pod security context by default
+    template: deployment.yaml
     asserts:
       - notExists:
           path: spec.template.spec.securityContext
 
   - it: should set pod security context when podSecurityContext is provided
+    template: deployment.yaml
     set:
       podSecurityContext:
         runAsNonRoot: true
@@ -87,22 +97,24 @@ tests:
           value: 1000
 
   - it: should set nodeSelector
+    template: deployment.yaml
     set:
-        nodeSelector:
-          role: AI
+      nodeSelector:
+        role: AI
     asserts:
       - equal:
           path: spec.template.spec.nodeSelector
           value:
             role: AI
 
   - it: should set tolerations
+    template: deployment.yaml
     set:
-        tolerations:
-          - key: role
-            operator: Equal
-            value: AI
-            effect: NoSchedule
+      tolerations:
+        - key: role
+          operator: Equal
+          value: AI
+          effect: NoSchedule
     asserts:
       - contains:
           any: true
@@ -111,4 +123,4 @@ tests:
             key: role
             value: AI
             effect: NoSchedule
-            operator: Equal
+            operator: Equal

helm/tools/querydoc/tests/toolserver_test.yaml

@@ -1,4 +1,4 @@
-replicaCount: 1
+replicas: 1
 
 image:
   registry: ghcr.io
@@ -39,7 +39,7 @@ logLevel: info
 # MCP server configuration
 config:
 
-# Secret configuration  
+# Secret configuration
 openai:
   apiKey: ""