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

Proc-test for Windows compensates for scoop #418

Merged
merged 1 commit into from
May 17, 2022

Conversation

lread
Copy link
Collaborator

@lread lread commented May 17, 2022

Also now closing driver session which closes the window that was always
left open at the end of test runs.

Fixes #382

Also now closing driver session which closes the window that was always
left open at the end of test runs.

Fixes clj-commons#382
@lread lread requested review from borkdude and slipset as code owners May 17, 2022 00:02
@borkdude
Copy link
Contributor

@lread What about using babashka fs and/or process here? It has a cross-platform way of detecting binaries: fs/which.

@lread
Copy link
Collaborator Author

lread commented May 17, 2022

Hiya @borkdude!
Big yes, I will definitely be adding in babashka/fs and also maybe babashka/process.
In this first stage, I'm just trying to make sure Etaoin tests are passing on all supported platforms while minimizing changes. I am almost there!

@lread lread merged commit 4dde749 into clj-commons:master May 17, 2022
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.

tests: test-prevent-process-fork fails on Windows
2 participants