Skip to content

Add test coverage for detected commands running in sandbox#207

Open
hanabel1 wants to merge 2 commits intomainfrom
add-sandbox-image-tests
Open

Add test coverage for detected commands running in sandbox#207
hanabel1 wants to merge 2 commits intomainfrom
add-sandbox-image-tests

Conversation

@hanabel1
Copy link
Copy Markdown
Contributor

@hanabel1 hanabel1 commented Apr 8, 2026

  • adds unit tests for DetectCommands covering every toolchain. The function didn't have any tests before.
  • adds TestDetectPackageManager covering each lockfile type and monorepo subdirectory detection
  • Adds TestDetectCommandsAndRunRemoteSSH which wires detected commands through runRemote into the fake ssh server, making sure that a Go project's go test ./..., a node project's npm test, and a Taskfile project's task test are correctly formatted and sent to the sandbox
  • Add's sandbox image smoke tests verifying the tools chunk relies on are present in Dockerfile.sandbox.

@hanabel1 hanabel1 assigned hanabel1 and unassigned hanabel1 Apr 8, 2026
@hanabel1 hanabel1 changed the title Add sandbox image smoke tests for installed commands Add test coverage for detected commands running in sandbox Apr 8, 2026
@hanabel1 hanabel1 marked this pull request as ready for review April 8, 2026 22:25
Copy link
Copy Markdown
Contributor

@danmux danmux left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed — approving.

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.

2 participants