Skip to content

Commit 5e04bc0

Browse files
committed
Another workaround for pytest-dev/pytest#1062 - don't call hook unless strictly necessary (never is for xdist).
1 parent aba5037 commit 5e04bc0

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

src/pytest_benchmark/session.py

Lines changed: 7 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -208,11 +208,13 @@ def handle_loading(self):
208208
def finish(self):
209209
self.handle_saving()
210210
self.handle_loading()
211-
self.groups = self.config.hook.pytest_benchmark_group_stats(
212-
config=self.config,
213-
benchmarks=self.prepare_benchmarks(),
214-
group_by=self.group_by
215-
)
211+
prepared_benchmarks = list(self.prepare_benchmarks())
212+
if prepared_benchmarks:
213+
self.groups = self.config.hook.pytest_benchmark_group_stats(
214+
config=self.config,
215+
benchmarks=self.prepare_benchmarks(),
216+
group_by=self.group_by
217+
)
216218

217219
def display(self, tr):
218220
if not self.groups:

0 commit comments

Comments
 (0)