Skip to content

Feature/helm unit test#6

Merged
thenav56 merged 5 commits into
mainfrom
feature/helm-unit-test
Apr 29, 2026
Merged

Feature/helm unit test#6
thenav56 merged 5 commits into
mainfrom
feature/helm-unit-test

Conversation

@susilnem

@susilnem susilnem commented Feb 27, 2026

Copy link
Copy Markdown
Member

Addresses

Command to run tests

helm unittest chart -f "tests/**/*_test.yaml"

Changes

  • Add helm unit test
  • Add CI check for Helm unit test

@susilnem susilnem force-pushed the feature/helm-unit-test branch from 0c4d8c1 to 5c46816 Compare February 27, 2026 12:27
@susilnem susilnem marked this pull request as ready for review February 27, 2026 12:28
@susilnem susilnem requested a review from thenav56 February 27, 2026 12:28
@susilnem susilnem linked an issue Feb 27, 2026 that may be closed by this pull request
@susilnem susilnem force-pushed the feature/helm-unit-test branch 2 times, most recently from 4598f53 to 26f1ead Compare April 19, 2026 02:46
@susilnem susilnem force-pushed the feature/helm-unit-test branch 2 times, most recently from 5582bd2 to fd5a0bc Compare April 19, 2026 02:52
@susilnem susilnem force-pushed the feature/helm-unit-test branch from fd5a0bc to 7b43ccc Compare April 23, 2026 06:19
@thenav56 thenav56 merged commit 8dcdb4c into main Apr 29, 2026
1 check passed
@thenav56 thenav56 deleted the feature/helm-unit-test branch April 29, 2026 07:46
thenav56 added a commit that referenced this pull request Jun 12, 2026
Rename table: add ingress.* -> api.ingress.* (block move, sub-keys
unchanged) and ingress.className required -> api.ingress.className
optional.

Section-by-section: new ingress: -> api.ingress: block with before/after.

Behavioral changes #6: explain that omitted className falls through to
the cluster's default IngressClass, including the failure mode if no
default exists (Ingress created but traffic silently unrouted).

Verification protocol: add "absent spec.ingressClassName" to expected
diff when consumer drops className.

Co-Authored-By: Claude Opus 4.7 <noreply@anthropic.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

add helm unittest

2 participants