fix(cli): remove unused _ = workspaceID, fix repoRoot() path for test binary

- workspace.go: remove no-op _ = workspaceID (workspaceID consumed in runHTTP call)
- molecule_test.go: replace hardcoded /workspace/repos/clone-cli path with
  dynamic exe-based resolution so tests pass when built by CI

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Molecule AI · sdk-dev 2026-04-22 18:35:43 +00:00
parent 1471cad4c1
commit a1dacff202
2 changed files with 6 additions and 4 deletions

View File

@ -193,10 +193,13 @@ func mockServer(t *testing.T, basePath string) *httptest.Server {
}
// repoRoot returns the repo root directory.
// Assumes this file is at cmd/molecule/molecule_test.go.
// ../../.. from cmd/molecule/ -> the module root.
func repoRoot() string {
// test file is at cmd/molecule/molecule_test.go
// ../.. from molecule/ goes to cmd, ../../.. goes to clone-cli
return filepath.Dir(filepath.Dir(filepath.Dir("/workspace/repos/clone-cli/cmd/molecule/")))
// Use os.Args[0] as a stable anchor: the test binary path.
// We walk up 3 levels from the test binary location.
exe, _ := os.Executable()
return filepath.Join(filepath.Dir(exe), "..", "..", "..")
}
// mol returns the path to the CLI binary, building it if needed.

View File

@ -284,6 +284,5 @@ func runWorkspaceDelegate(cmd *cobra.Command, args []string) error {
} else {
fmt.Printf("Delegation sent to %q.\n", targetID)
}
_ = workspaceID
return nil
}