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

helm unittest are failing #100

Open
3 tasks
MichaelSp opened this issue Mar 3, 2024 · 1 comment
Open
3 tasks

helm unittest are failing #100

MichaelSp opened this issue Mar 3, 2024 · 1 comment

Comments

@MichaelSp
Copy link
Contributor

MichaelSp commented Mar 3, 2024

Is it just me, or are all unit-tests failing?

$ helm unittest charts/docker-mailserver
 FAIL  	charts/docker-mailserver/tests/configmap_test.yaml
	- Execution Error:
		1 error(s) decoding:

* 'Pattern' expected type 'string', got unconvertible type 'int', value: '10993'


### Error:  1 error(s) decoding:

* 'Pattern' expected type 'string', got unconvertible type 'int', value: '10993'

but even after fixing that simple error:

Charts:      1 failed, 0 passed, 1 total
Test Suites: 5 failed, 1 passed, 6 total
Tests:       13 failed, 1 errored, 5 passed, 18 total
Snapshot:    15 passed, 15 total
Time:        71.612ms

looks pretty bad :(

Also, snapshots are not committed.

I would recommend:

  • fix the tests
  • commit the snapshots
  • create a workflow that runs unittests on each PR
@cfis
Copy link
Collaborator

cfis commented Mar 4, 2024

Yeah I am sure that is true. I didn't update them when making a bunch of changes a month ago. I am not sure if they are worth updating or should be removed, but I agree they should not fail.

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

No branches or pull requests

2 participants