Skip to content
This repository has been archived by the owner on Sep 2, 2024. It is now read-only.

Add some autospecs in the patches #812

Closed
DominicOram opened this issue Jul 18, 2023 · 1 comment · Fixed by #813
Closed

Add some autospecs in the patches #812

DominicOram opened this issue Jul 18, 2023 · 1 comment · Fixed by #813
Assignees

Comments

@DominicOram
Copy link
Collaborator

We have previously seen issues with tests passing but not catching errors as the function that we're patching out has changed arguments, we should add autospec to some more patches so that we would catch these kinds of issues.

Acceptance Criteria

  • autospec=True is added to patched functions
  • There are docs around this
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

1 participant