test(e2e): expand Podman e2e test parity with Docker#283
Conversation
…ith Docker Port variables substitution, variable defaults, remoteEnv null, custom image, custom image skip build, extra devcontainer merge/override, postStartCommand restart, waitFor deferred lifecycle, postAttach nonblocking, postAttach every attach, initializeCommand parallel, secrets-file injection, and multi devcontainer selection tests to both rootless and rootful Podman contexts.
✅ Deploy Preview for devsydev canceled.
|
|
Important Review skippedDraft detected. Please check the settings in the CodeRabbit UI or the ⚙️ Run configurationConfiguration used: Organization UI Review profile: CHILL Plan: Pro Run ID: You can disable this status message by setting the Use the checkbox below for a quick retry:
Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Replace repeated "windows" string literals with a named constant.
|
Superseded by PR #284 which was merged for DEVSY-100 (Podman e2e test parity). This PR was an earlier attempt from a different worktree branch. |
…ring (#283) (#287) Per the devcontainer spec, setting workspaceMount to "" should suppress the default workspace mount. Previously the string type could not distinguish between omitted and explicitly empty, so "" fell through to the default bind mount. Change WorkspaceMount from string to *string so nil means omitted (use default mount) and *"" means suppress mount entirely.
1 participant
Summary
ginkgo.Itblocks) ine2e/tests/up/provider_podman.go