URL: https://github.com/freeipa/freeipa/pull/1480 Author: felipevolpone Title: #1480: IntegrationTests now collects logs from all test methods Action: opened
PR body: """ `logs_dict` should not be cleared. It's filled once per class and it should not be cleared after running the first test. https://pagure.io/freeipa/issue/7310 https://pagure.io/freeipa/issue/7335 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/1480/head:pr1480 git checkout pr1480
From c75de1008381de45affa9a853989d6278e3b6996 Mon Sep 17 00:00:00 2001 From: Felipe Barreto <fbarr...@redhat.com> Date: Thu, 18 Jan 2018 09:10:06 -0200 Subject: [PATCH] IntegrationTests now collects logs from all test methods logs_dict should not be cleared. It's filled once per class and it should not be cleared after running the first test. https://pagure.io/freeipa/issue/7310 https://pagure.io/freeipa/issue/7335 --- ipatests/pytest_plugins/integration/__init__.py | 2 -- 1 file changed, 2 deletions(-) diff --git a/ipatests/pytest_plugins/integration/__init__.py b/ipatests/pytest_plugins/integration/__init__.py index bee669b0dd..2c107b926b 100644 --- a/ipatests/pytest_plugins/integration/__init__.py +++ b/ipatests/pytest_plugins/integration/__init__.py @@ -179,8 +179,6 @@ def collect_logs(name, logs_dict, logfile_dir=None, beakerlib_plugin=None): else: shutil.rmtree(topdirname) - logs_dict.clear() - @pytest.fixture(scope='class') def class_integration_logs():
_______________________________________________ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org