From b37575f000c96d9471b29c4078c6c626c85337a2 Mon Sep 17 00:00:00 2001 From: Niels Thykier Date: Wed, 31 Oct 2018 21:31:55 +0000 Subject: [PATCH] Stop creating some throw-away data-structures for a method call Signed-off-by: Niels Thykier --- britney.py | 2 +- britney2/utils.py | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/britney.py b/britney.py index 5236ab2..3e3963f 100755 --- a/britney.py +++ b/britney.py @@ -1460,7 +1460,7 @@ class Britney(object): # on a subset of all nobreak architectures). # This forgivness is only done if the package is already in testing AND it is broken # in testing on this architecture already. Anything else would be a regression - if inst_tester.any_of_these_are_in_testing({pkg_id}) and not inst_tester.is_installable(pkg_id): + if inst_tester.is_pkg_in_testing(pkg_id) and not inst_tester.is_installable(pkg_id): # It is a regression. excuse.policy_verdict = PolicyVerdict.REJECTED_PERMANENTLY diff --git a/britney2/utils.py b/britney2/utils.py index 4e39fa7..2fe324a 100644 --- a/britney2/utils.py +++ b/britney2/utils.py @@ -140,7 +140,7 @@ def undo_changes(lundo, inst_tester, suite_info, all_binary_packages): # If this happens, pkg_id must be a cruft item that # was *not* migrated. assert source_data.version != all_binary_packages[pkg_id].version - assert not inst_tester.any_of_these_are_in_testing((pkg_id,)) + assert not inst_tester.is_pkg_in_testing(pkg_id) inst_tester.remove_testing_binary(pkg_id) # STEP 3