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

Use a wrapper to properly test CTRL-C event on Windows #359

Merged
merged 8 commits into from Aug 22, 2022
Merged

Use a wrapper to properly test CTRL-C event on Windows #359

merged 8 commits into from Aug 22, 2022

Conversation

paescuj
Copy link
Collaborator

@paescuj paescuj commented Aug 17, 2022

I feel like it's important to test the behavior of concurrently when receiving a CTRL-C (similar to SIGINT) signal on Windows, so we can be sure no regressions are introduced and work on new features with a better feeling. Therefore using ctrlc-wrapper to properly simulate this event.

@coveralls
Copy link

coveralls commented Aug 17, 2022

Coverage Status

Coverage remained the same at 100.0% when pulling 7f9be6c on paescuj:sigint-test-windows into e469c04 on open-cli-tools:main.

@paescuj paescuj marked this pull request as ready for review August 17, 2022 18:48
@paescuj paescuj marked this pull request as draft August 18, 2022 09:15
@paescuj paescuj changed the title Add a wrapper to properly test CTRL-C (SIGINT) on Windows Add a wrapper to properly test CTRL-C event on Windows Aug 19, 2022
@paescuj paescuj marked this pull request as ready for review August 19, 2022 14:18
@paescuj paescuj changed the title Add a wrapper to properly test CTRL-C event on Windows Use a wrapper to properly test CTRL-C event on Windows Aug 20, 2022
@paescuj paescuj merged commit 2e8eaa5 into open-cli-tools:main Aug 22, 2022
@paescuj paescuj deleted the sigint-test-windows branch August 22, 2022 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants