Function TestGetNpmBinPathSetup

pkg/workflow/engine_helpers_test.go:274–316

(t *testing.T)

}

func TestGetNpmBinPathSetup(t *testing.T) {
	pathSetup := GetNpmBinPathSetup()

	// Should require RUNNER_TOOL_CACHE instead of guessing fallback paths.
	if !strings.Contains(pathSetup, "RUNNER_TOOL_CACHE must be set") {
		t.Errorf("PATH setup should require RUNNER_TOOL_CACHE, got: %s", pathSetup)
	}
	if strings.Contains(pathSetup, ":-/opt/hostedtoolcache") {
		t.Errorf("PATH setup should not fall back to /opt/hostedtoolcache, got: %s", pathSetup)
	}
	if strings.Contains(pathSetup, "/home/runner/work/_tool") {
		t.Errorf("PATH setup should not hard-code /home/runner/work/_tool, got: %s", pathSetup)
	}

	// Should search for bin directories
	if !strings.Contains(pathSetup, "-name bin") {
		t.Errorf("PATH setup should search for bin directories, got: %s", pathSetup)
	}
	if !strings.Contains(pathSetup, "-maxdepth 5") {
		t.Errorf("PATH setup should search deep enough for setup-node toolcache bins, got: %s", pathSetup)
	}

	// Should preserve existing PATH
	if !strings.Contains(pathSetup, "$PATH") {
		t.Errorf("PATH setup should include $PATH, got: %s", pathSetup)
	}

	// Should re-prepend GOROOT/bin after the find to preserve correct Go version ordering
	// (find returns alphabetically, so go/1.23 can shadow go/1.25)
	if !strings.Contains(pathSetup, "$GOROOT") {
		t.Errorf("PATH setup should re-prepend GOROOT/bin after find, got: %s", pathSetup)
	}

	// GOROOT re-prepend should come AFTER the find command
	findIdx := strings.Index(pathSetup, `find "$GH_AW_TOOL_CACHE"`)
	gorootIdx := strings.Index(pathSetup, "$GOROOT")
	if findIdx < 0 {
		t.Errorf("PATH setup should search GH_AW_TOOL_CACHE, got: %s", pathSetup)
	}
	if gorootIdx < findIdx {
		t.Errorf("GOROOT re-prepend should come after find command, got: %s", pathSetup)
	}
}

// TestGetNpmBinPathSetup_GorootOrdering verifies that GOROOT/bin takes precedence
// over alphabetically-ordered Go versions in hostedtoolcache.