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

Make kill switch timing testable without sleeps #29

Open
holsee opened this issue Jul 5, 2019 · 1 comment
Open

Make kill switch timing testable without sleeps #29

holsee opened this issue Jul 5, 2019 · 1 comment
Labels
enhancement New feature or request

Comments

@holsee
Copy link
Member

holsee commented Jul 5, 2019

Refactor kill_switch function into own module to allow time-based unit testing without Proces.sleep call.

@holsee holsee added the enhancement New feature or request label Jul 5, 2019
@holsee
Copy link
Member Author

holsee commented Jul 5, 2019

From the discussions which took place at the Tuesday 2nd July Meet-up:

  • We need to refactor the kill switch mechanism into its own module
  • We need a means by which this dependency can be passed into the Service process
  • A test version of the kill-switch needs to be created that will allow for introspection of the timeout value passed into it, as well as a means to manually trigger the kill switch signal.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

1 participant