diff options
| author | Stefan Metzmacher <metze@samba.org> | 2021-12-20 17:02:12 +0100 |
|---|---|---|
| committer | Stefan Metzmacher <metze@samba.org> | 2021-12-20 16:10:31 +0000 |
| commit | bd804e0eef85ed4e05f9a3b7afbd29b1ba4a5d97 (patch) | |
| tree | c941c5a5c9d9b14f7d26d977e0acc93afac7248e /python | |
| parent | 00c2425c2c10c324868f0c5e7283da7714c009eb (diff) | |
| download | samba-bd804e0eef85ed4e05f9a3b7afbd29b1ba4a5d97.tar.gz | |
Revert "python:tests: Don't require an emtpy 'authorization-data' to be present"
This reverts commit 36325f1ee907d38c978229da67de3844f969cd33.
This was not the latest version from:
https://gitlab.com/samba-team/samba/-/merge_requests/2304
The correct version follows...
Signed-off-by: Stefan Metzmacher <metze@samba.org>
Diffstat (limited to 'python')
| -rw-r--r-- | python/samba/tests/krb5/raw_testcase.py | 15 |
1 files changed, 2 insertions, 13 deletions
diff --git a/python/samba/tests/krb5/raw_testcase.py b/python/samba/tests/krb5/raw_testcase.py index 8b6eec3c40d..d11f628d7b6 100644 --- a/python/samba/tests/krb5/raw_testcase.py +++ b/python/samba/tests/krb5/raw_testcase.py @@ -2490,19 +2490,8 @@ class RawKerberosTest(TestCaseInTempDir): if self.strict_checking: self.assertElementEqual(ticket_private, 'caddr', []) if expect_pac is not None: - if expect_pac: - self.assertElementPresent(ticket_private, - 'authorization-data', - expect_empty=not expect_pac) - else: - # It is more correct to not have an authorization-data - # present than an empty one. - # - # https://github.com/krb5/krb5/pull/1225#issuecomment-995104193 - v = self.getElementValue(ticket_private, - 'authorization-data') - if v is not None: - self.assertEqual(0, len(v)) + self.assertElementPresent(ticket_private, 'authorization-data', + expect_empty=not expect_pac) encpart_session_key = None if encpart_private is not None: |