diff options
author | Ran Benita <ran@unusedvar.com> | 2021-01-31 01:49:25 +0200 |
---|---|---|
committer | Ran Benita <ran@unusedvar.com> | 2021-02-06 20:46:08 +0200 |
commit | f674f6da9fec826bc8c4e08781e7309322fc12cc (patch) | |
tree | 10fdb72013df25f8e89f5c8512511f858a7a208f /src/_pytest/runner.py | |
parent | 80c223474c98fd59a07776994e672e934866c7d5 (diff) | |
download | pytest-f674f6da9fec826bc8c4e08781e7309322fc12cc.tar.gz |
runner: add a safety assert to SetupState.prepare
This ensures that the teardown for the previous item was done properly
for this (next) item, i.e. there are no leftover teardowns.
Diffstat (limited to 'src/_pytest/runner.py')
-rw-r--r-- | src/_pytest/runner.py | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/src/_pytest/runner.py b/src/_pytest/runner.py index ae76a2472..124cf531a 100644 --- a/src/_pytest/runner.py +++ b/src/_pytest/runner.py @@ -479,15 +479,18 @@ class SetupState: def prepare(self, item: Item) -> None: """Setup objects along the collector chain to the item.""" + needed_collectors = item.listchain() + # If a collector fails its setup, fail its entire subtree of items. # The setup is not retried for each item - the same exception is used. for col, (finalizers, prepare_exc) in self.stack.items(): + assert col in needed_collectors, "previous item was not torn down properly" if prepare_exc: raise prepare_exc - needed_collectors = item.listchain() for col in needed_collectors[len(self.stack) :]: assert col not in self.stack + # Push onto the stack. self.stack[col] = ([col.teardown], None) try: col.setup() |