Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Updated the helm-unittest versions to latest. #961

Closed
wants to merge 2 commits into from
Closed

Updated the helm-unittest versions to latest. #961

wants to merge 2 commits into from

Conversation

abhayporwals
Copy link

What does this PR do?

This PR will update the older version of helm-unittest to the latest.

@abhayporwals abhayporwals requested review from a team as code owners November 27, 2023 18:31
@abhayporwals
Copy link
Author

Hey @NotMyFault, Kindly take a look and let me know why these test cases are failing.

@torstenwalter
Copy link
Member

Error: unknown flag: --helm3. The command to run the unit test is specified in

- helm unittest --helm3 --strict -f 'unittests/*.yaml' {{ .Path }}

I bet it's using helm3 by default and that parameter is no longer necessary. Not sure what else has been changed. You can use the same command locally to run the tests. That simplifies the workflow as you might have to adapt some tests to get them working with the newer plugin version.

@kvanzuijlen
Copy link
Contributor

@timja can be closed

@timja timja closed this Feb 1, 2024
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.

Update helm-unittests from 0.2.11 to 0.3.x
4 participants