Skip to content

Commit

Permalink
clarified mocker is the fixture, not .patch
Browse files Browse the repository at this point in the history
  • Loading branch information
ESadek-MO authored Nov 4, 2024
1 parent 6067c9f commit c4d2043
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -41,7 +41,7 @@ Conversion Checklist
#. Check for references to ``@tests``. These should be changed to ``@_shared_utils``.
#. Check for references to ``with mock.patch("...")``. These should be replaced with
``mocker.patch("...")``, which can be passed into functions as a fixture.
``mocker.patch("...")``. ``mocker`` is a fixture, and can be passed into functions.
#. Check for ``np.testing.assert...``. This can usually be swapped for
``_shared_utils.assert...``.
#. Check for references to ``super()``. Most test classes used to inherit from
Expand Down

0 comments on commit c4d2043

Please sign in to comment.