URL: https://github.com/freeipa/freeipa/pull/1291 Author: tiran Title: #1291: [Backport][ipa-4-6] Py3: Fix vault tests Action: opened
PR body: """ Manual backport of PR #1284 """ To pull the PR as Git branch: git remote add ghfreeipa https://github.com/freeipa/freeipa git fetch ghfreeipa pull/1291/head:pr1291 git checkout pr1291
From 1093cd9c31478d09ca225784d6d31d547af496b7 Mon Sep 17 00:00:00 2001 From: Christian Heimes <chei...@redhat.com> Date: Tue, 14 Nov 2017 16:42:38 +0100 Subject: [PATCH] Py3: Fix vault tests * Bump PKI to 10.5.1-2, which fixes an issue with KRA under Python 3 * Correct encoding of secret https://pagure.io/freeipa/issue/7033 Signed-off-by: Christian Heimes <chei...@redhat.com> Reviewed-By: Rob Crittenden <rcrit...@redhat.com> --- .travis.yml | 3 +-- freeipa.spec.in | 13 +++++++------ ipatests/test_xmlrpc/test_vault_plugin.py | 26 ++++++++++++++++---------- 3 files changed, 24 insertions(+), 18 deletions(-) diff --git a/.travis.yml b/.travis.yml index 8875b7760d..2be56a2ed7 100644 --- a/.travis.yml +++ b/.travis.yml @@ -46,8 +46,7 @@ env: test_ipapython test_ipaserver test_integration/test_ipalib_util.py - test_xmlrpc/test_[l-uw-z]*.py" - # FIXME: add vault tests once PKI finally fixes vault + test_xmlrpc/test_[l-z]*.py" install: - pip install --upgrade pip - pip3 install --upgrade pip diff --git a/freeipa.spec.in b/freeipa.spec.in index f410a4c3aa..b50b443739 100644 --- a/freeipa.spec.in +++ b/freeipa.spec.in @@ -188,7 +188,7 @@ BuildRequires: python2-dns >= 1.15 BuildRequires: jsl BuildRequires: python2-yubico # pki Python package -BuildRequires: pki-base-python2 +BuildRequires: pki-base-python2 >= 10.5.1-2 BuildRequires: python2-pytest-multihost BuildRequires: python2-pytest-sourceorder # 0.4.2: Py3 fix https://bugzilla.redhat.com/show_bug.cgi?id=1476150 @@ -225,7 +225,7 @@ BuildRequires: python3-qrcode-core >= 5.0.0 BuildRequires: python3-dns >= 1.15 BuildRequires: python3-yubico # pki Python package -BuildRequires: pki-base-python3 +BuildRequires: pki-base-python3 >= 10.5.1-2 BuildRequires: python3-pytest-multihost BuildRequires: python3-pytest-sourceorder # 0.4.2: Py3 fix https://bugzilla.redhat.com/show_bug.cgi?id=1476150 @@ -323,8 +323,9 @@ Requires(post): systemd-units Requires: selinux-policy >= %{selinux_policy_version} Requires(post): selinux-policy-base >= %{selinux_policy_version} Requires: slapi-nis >= %{slapi_nis_version} -Requires: pki-ca >= 10.4.0-1 -Requires: pki-kra >= 10.4.0-1 +# 10.5.1-2 contains Python 3 vault fix +Requires: pki-ca >= 10.5.1-2 +Requires: pki-kra >= 10.5.1-2 Requires(preun): systemd-units Requires(postun): systemd-units Requires: policycoreutils >= 2.1.12-5 @@ -388,7 +389,7 @@ Requires: python2-dbus Requires: python2-dns >= 1.15 Requires: python2-kdcproxy >= 0.3 Requires: rpm-libs -Requires: pki-base-python2 +Requires: pki-base-python2 >= 10.5.1-2 Requires: python2-augeas %description -n python2-ipaserver @@ -422,7 +423,7 @@ Requires: python3-dns >= 1.15 Requires: python3-kdcproxy >= 0.3 Requires: python3-augeas Requires: rpm-libs -Requires: pki-base-python3 +Requires: pki-base-python3 >= 10.5.1-2 %description -n python3-ipaserver IPA is an integrated solution to provide centrally managed Identity (users, diff --git a/ipatests/test_xmlrpc/test_vault_plugin.py b/ipatests/test_xmlrpc/test_vault_plugin.py index 7023d6a9be..e4348c19c0 100644 --- a/ipatests/test_xmlrpc/test_vault_plugin.py +++ b/ipatests/test_xmlrpc/test_vault_plugin.py @@ -22,9 +22,12 @@ """ import nose -from ipalib import api -from ipatests.test_xmlrpc.xmlrpc_test import Declarative, fuzzy_string import pytest +import six + +from ipalib import api +from ipatests.test_xmlrpc.xmlrpc_test import Declarative, fuzzy_bytes + vault_name = u'test_vault' service_name = u'HTTP/server.example.com' @@ -35,12 +38,15 @@ asymmetric_vault_name = u'asymmetric_test_vault' # binary data from \x00 to \xff -secret = ''.join(chr(c) for c in range(0, 256)) +if six.PY2: + secret = b''.join(chr(c) for c in range(0, 256)) +else: + secret = bytes(range(0, 256)) password = u'password' other_password = u'other_password' -public_key = """ +public_key = b""" -----BEGIN PUBLIC KEY----- MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAnT61EFxUOQgCJdM0tmw/ pRRPDPGchTClnU1eBtiQD3ItKYf1+weMGwGOSJXPtkto7NlE7Qs8WHAr0UjyeBDe @@ -52,7 +58,7 @@ -----END PUBLIC KEY----- """ -private_key = """ +private_key = b""" -----BEGIN RSA PRIVATE KEY----- MIIEpAIBAAKCAQEAnT61EFxUOQgCJdM0tmw/pRRPDPGchTClnU1eBtiQD3ItKYf1 +weMGwGOSJXPtkto7NlE7Qs8WHAr0UjyeBDek/zeB6nSVdk47OdaW1AHrJL+44r2 @@ -82,7 +88,7 @@ -----END RSA PRIVATE KEY----- """ -other_public_key = """ +other_public_key = b""" -----BEGIN PUBLIC KEY----- MIIBIjANBgkqhkiG9w0BAQEFAAOCAQ8AMIIBCgKCAQEAv7E/QLVyKjrgDctZ50U7 rmtL7Ks1QLoccp9WvZJ6WI1rYd0fX5FySS4dI6QTNZc6qww8NeNuZtkoxT9m1wkk @@ -94,7 +100,7 @@ -----END PUBLIC KEY----- """ -other_private_key = """ +other_private_key = b""" -----BEGIN RSA PRIVATE KEY----- MIIEpgIBAAKCAQEAv7E/QLVyKjrgDctZ50U7rmtL7Ks1QLoccp9WvZJ6WI1rYd0f X5FySS4dI6QTNZc6qww8NeNuZtkoxT9m1wkkRl/3wK7fWNLenH/+VHOaTQc20exg @@ -678,7 +684,7 @@ def setup_class(cls): 'result': { 'cn': [standard_vault_name], 'ipavaulttype': [u'symmetric'], - 'ipavaultsalt': [fuzzy_string], + 'ipavaultsalt': [fuzzy_bytes], 'owner_user': [u'admin'], 'username': u'admin', }, @@ -724,7 +730,7 @@ def setup_class(cls): 'objectclass': [u'top', u'ipaVault'], 'cn': [symmetric_vault_name], 'ipavaulttype': [u'symmetric'], - 'ipavaultsalt': [fuzzy_string], + 'ipavaultsalt': [fuzzy_bytes], 'owner_user': [u'admin'], 'username': u'admin', }, @@ -784,7 +790,7 @@ def setup_class(cls): 'result': { 'cn': [symmetric_vault_name], 'ipavaulttype': [u'symmetric'], - 'ipavaultsalt': [fuzzy_string], + 'ipavaultsalt': [fuzzy_bytes], 'owner_user': [u'admin'], 'username': u'admin', },
_______________________________________________ FreeIPA-devel mailing list -- freeipa-devel@lists.fedorahosted.org To unsubscribe send an email to freeipa-devel-le...@lists.fedorahosted.org