From: Joseph Sutton <josephsutton@catalyst.net.nz>
Date: Wed, 18 Oct 2023 03:06:51 +0000 (+1300)
Subject: tests/krb5: Make ‘services’ parameter required
X-Git-Tag: talloc-2.4.2~1211
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=7ea71c4882e97c33e1714f8be461aedb57b82aae;p=thirdparty%2Fsamba.git

tests/krb5: Make ‘services’ parameter required

We use it unconditionally without a check for None.

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
---

diff --git a/python/samba/tests/krb5/s4u_tests.py b/python/samba/tests/krb5/s4u_tests.py
index d91c06c418f..b03a246e4be 100755
--- a/python/samba/tests/krb5/s4u_tests.py
+++ b/python/samba/tests/krb5/s4u_tests.py
@@ -1778,7 +1778,7 @@ class S4UKerberosTests(KDCBaseTest):
                                     checksum_keys=checksum_keys,
                                     include_checksums=include_checksums)
 
-    def add_delegation_info(self, ticket, services=None):
+    def add_delegation_info(self, ticket, *, services):
         def modify_pac_fn(pac):
             pac_buffers = pac.buffers
             self.assertNotIn(krb5pac.PAC_TYPE_CONSTRAINED_DELEGATION,