feat: add unit tests for Helm chart #158
Conversation
|
This pull request has been linked to Shortcut Story #15699: Automated unit tests for Helm package. |
| // CoderValues is a typed Go representation of Coder's values file, | ||
| // suitable for writing tests. This provides code completion for Go | ||
| // tests. | ||
| type CoderValues struct { |
There was a problem hiding this comment.
Keeping this in sync manually is going to be an issue down the line.
Could we generate this automatically from values.yaml?
Aside: is there any decent tool for doing this directly? I found yaml2go but it appears to not be maintained. We could also go the route of YAML -> JSON -> Go, for which there is a slightly more up to date tool. If all else fails, I suppose there's Swagger.
There was a problem hiding this comment.
I discussed this with @Emyrk, there's a way you can define a JSON Schema to validate the values, and we could generate both the values.yaml and these structs from that. But in the interest of an initial implementation, I left taht out for now :)
There was a problem hiding this comment.
Nice :-) I think this will really help!
Very rough first pass, mostly a proof-of-concept. The tests aren't complete, and not part of CI yet, but the goal is to show what writing them could look like.