kanister icon indicating copy to clipboard operation
kanister copied to clipboard

Add Helm Test to cover additional attributes introduced in values.yaml used during kanister operator installation

Open mabhi opened this issue 9 months ago • 1 comments

Change Overview

Now that support has been added PR for tolerations and nodeSelector in the kanister helm chart, they need to be tested via helm-tests. With additional support for dry-run with PR we can run helm install dry run on these charts and rendered output can be validated. This PR specifically does the above requirement by adding helm tests to validate the properties are populated with the provided helm values.

Pull request type

Please check the type of change your PR introduces:

  • [ ] :construction: Work in Progress
  • [ ] :rainbow: Refactoring (no functional changes, no api changes)
  • [ ] :hamster: Trivial/Minor
  • [ ] :bug: Bugfix
  • [ ] :sunflower: Feature
  • [ ] :world_map: Documentation
  • [x] :robot: Test

Issues

  • fixes #issue-number

Test Plan

  • [ ] :muscle: Manual
  • [x] :zap: Unit test
  • [ ] :green_heart: E2E

mabhi avatar May 10 '24 08:05 mabhi

@mabhi can you please correct the title of the PR.

viveksinghggits avatar May 10 '24 13:05 viveksinghggits