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

Support selective patching #388

Open
wants to merge 8 commits into
base: master
Choose a base branch
from

Conversation

tzoiker
Copy link

@tzoiker tzoiker commented Mar 4, 2021

Motivated by #384 - implicit monotonic time patching may (and does) cause hard-to-debug problems in various settings.

  • Add Target enum describing functions that can be patched;
  • Pass targets parameter to freeze_time to describe functions to be patched;
  • Make targets exclude monotonic time by default;
  • Add freeze_time_with_monotonic helper to include monotonic time patching.

@Pavkazzz
Copy link

@spulec Can we merge this one?
Looks good

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.

2 participants