Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

410/gexec gopackage type #414

Closed
wants to merge 4 commits into from
Closed

410/gexec gopackage type #414

wants to merge 4 commits into from

Conversation

holyhope
Copy link
Contributor

No description provided.

Adds following methods:
- CompileTest(packagePath string, args ...string) (compiledPath string, err error)
- CompileTestWithEnvironment(packagePath string, env []string, args ...string) (compiledPath string, err error)
- CompileTestIn(gopath string, packagePath string, args ...string) (compiledPath string, err error)

Signed-off-by: Pierre Péronnet <pierre.peronnet@ovhcloud.com>
This solve the issue in test where suite scoped firefly binary
was removed by CleanupBuildArtifacts() in build_test.go

Signed-off-by: Pierre Péronnet <pierre.peronnet@ovhcloud.com>
Adds following methods:
- GetAndCompileTest(packagePath string, args ...string) (compiledPath string, err error)
- GetAndCompileTestWithEnvironment(packagePath string, env []string, args ...string) (compiledPath string, err error)
- GetAndCompileTestIn(gopath string, packagePath string, args ...string) (compiledPath string, err error)

Signed-off-by: Pierre Péronnet <pierre.peronnet@ovhcloud.com>
Signed-off-by: Pierre Péronnet <pierre.peronnet@ovhcloud.com>
@holyhope
Copy link
Contributor Author

New implementation of #411 for #410

@onsi onsi closed this Feb 25, 2021
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.

None yet

2 participants