[DOCS] helm: fix documentation drift in README-testing.md#1688
[DOCS] helm: fix documentation drift in README-testing.md#1688EItanya merged 5 commits intokagent-dev:mainfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Note
Copilot was unable to run its full agentic suite in this review.
This PR updates helm/README-testing.md to match the current Helm chart unittest suite layout and to align CLI examples with helm-unittest v1.0.3.
Changes:
- Updated the documented test directory structure and test file names to reflect the current
helm/kagent/testscontents. - Revised the “Test Coverage” section to remove non-existent test categories and add missing ones.
- Corrected the example helm-unittest CLI usage for providing a values file.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Update test file names to match actual files (e.g. deployment_test.yaml → controller-deployment_test.yaml), remove non-existent integration_test.yaml, add missing test entries (postgresql, security-context, toolserver, ui-deployment, ui-service), fix incorrect CLI flags (--set → removed, -f → -v for values files), and correct global.tag to tag. Signed-off-by: TOMOFUMI-KONDO <ugax2kontomo0314@gmail.com>
TOMOFUMI-KONDO
force-pushed
the
docs/helm-test-doc
branch
from
bc6bf1f to
faefc52
Compare
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
@EItanya @ilackarms @yuval-k |
|
@EItanya pull_request:
branches: [main, release/v0.7.x]
paths-ignore:
- "**/*.md"Should I fix the workflow triggers not to ignore markdown files? |
TOMOFUMI-KONDO
requested review from
iplay88keys,
jmhbh and
supreme-gg-gg
as code owners
fixed some documentation drift in helm/README-testing.md
Test Structure,Test CoverageExample Test Scenarios