Skip to content

Commit

Permalink
added conversion translation section, and fixed mocker reference
Browse files Browse the repository at this point in the history
  • Loading branch information
ESadek-MO committed Nov 1, 2024
1 parent 7bca716 commit 8c52324
Showing 1 changed file with 21 additions and 1 deletion.
22 changes: 21 additions & 1 deletion docs/src/developers_guide/contributing_pytest_conversions.rst
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("...")``. Note, ``mocker.patch("...")`` is NOT a context manager.
``mocker.patch("...")``.
#. 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 All @@ -54,3 +54,23 @@ Conversion Checklist
#. Check the file against https://github.com/astral-sh/ruff , using ``pip install ruff`` ->
``ruff check --select PT <file>``.

Common Translations
-------------------

.. list-table::
:widths: 50 50
:header-rows: 1

* - ``unittest`` method
- ``pytest`` equivalent
* - ``assertTrue(x)``
- ``assert x``
* - ``assertFalse(x)``
- ``assert not x``
* - ``assertRegex(x, y)``
- ``assert re.match(y, x)``
* - ``assertRaisesRegex(cls, msg_re)``
- ``with pytest.raises(cls, match=msg_re):``
* - ``mock.patch(...)``
- ``mocker.patch(...)``

0 comments on commit 8c52324

Please sign in to comment.