docs: validate and update E2E test docs#4834
Conversation
✅ Deploy Preview for zarf-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
560fc30 to
2e67c9c
Compare
|
@brandtkeller @AustinAbro321 Can I get some early feedback on the //Verify cluster connectivity section I added in 20_zarf_init_test.go. Originally was trying to work in main_test.go to verify cluster, but skipK8sEnvVar did not work as I expected it to. I left my original thought process in main_test.go commented out. This is what the error in 20_zarf_init_test.go would look like, so it would fail early. More so just wondering if putting it in 20_zarf_init_test.go is good or if you would truly prefer it in main_test.go.
|
This is a great fallback but I would love to hear more about what limitations or constraints you ran into regarding I would point out that the current solution is limited to specific test commands. The out of the box |
2e67c9c to
7b44029
Compare

Description
This PR aims to:
Related Issue
Relates to #2477
Checklist before merging