Skip to content

Commit

Permalink
Merge pull request #2980 from nicoddemus/immutable-fix-parameters
Browse files Browse the repository at this point in the history
Add CHANGELOG entry about pytest.fixture "params" being now immutable
  • Loading branch information
nicoddemus authored Nov 30, 2017
2 parents 4de433e + fdfc194 commit a9dd37f
Showing 1 changed file with 8 additions and 0 deletions.
8 changes: 8 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,14 @@ Deprecations and Removals
with the boolean ``Node._skipped_by_mark``. (`#2767
<https://github.com/pytest-dev/pytest/issues/2767>`_)

- The ``params`` list passed to ``pytest.fixture`` is now for
all effects considered immutable and frozen at the moment of the ``pytest.fixture``
call. Previously the list could be changed before the first invocation of the fixture
allowing for a form of dynamic parametrization (for example, updated from command-line options),
but this was an unwanted implementation detail which complicated the internals and prevented
some internal cleanup. See issue `#2959 <https://github.com/pytest-dev/pytest/issues/2959>`_
for details and a recommended workaround.

Features
--------

Expand Down

0 comments on commit a9dd37f

Please sign in to comment.