Hi all,

this was discussed before on IRC, but here is an update and a question.
currently pytest-trunk implements a convenient new monkeypatch.replace method:

    # example usage
    monkeypatch.replace("os.path.abspath", lambda x: "/abc")

This effectively performs:

    import os.path
    monkeypatch.setattr(os.path, "abspath", lambda x: "/abc")

However, i just wanted to use it and couldn't remember the "replace"
name and don't find it very telling anymore.  I often do something
like this::

    monkeypatch.setattr("os.path.abspath", lambda x: "/abc")
   
which currently ends up as an error.  Would it be too awkward to
just make the latter work and forget about introducing a new "replace"
method name?  IMO it's rather intuitive to just make setattr accept
the new signature, although the docstring will need a bit of explaining.

cheers,
holger

_______________________________________________
Pytest-dev mailing list
[email protected]
https://mail.python.org/mailman/listinfo/pytest-dev

Reply via email to