Stop creating some throw-away data-structures for a method call

Signed-off-by: Niels Thykier <niels@thykier.net>

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
 

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