From 7a02f2545a5b8d31a5b524c1d5ef73603f2cb302 Mon Sep 17 00:00:00 2001 From: Frederic Lepied Date: Wed, 19 Jul 2017 11:31:38 +0200 Subject: [PATCH] Allow to skip test_aci_rest if no xmljson is installed There is no such package in Fedora so building the Fedora package fails. See http://38.145.33.116/dlrn/e9/70/e970237a2f288c9363b0c012e53c01f20f6b6a74_ffe3b87d/rpmbuild.log Introduced by https://github.com/ansible/ansible/commit/e970237a2f288c9363b0c012e53c01f20f6b6a74 --- test/units/modules/network/aci/test_aci_rest.py | 9 ++++++--- 1 file changed, 6 insertions(+), 3 deletions(-) diff --git a/test/units/modules/network/aci/test_aci_rest.py b/test/units/modules/network/aci/test_aci_rest.py index b1a01fdb99..373a4d662a 100644 --- a/test/units/modules/network/aci/test_aci_rest.py +++ b/test/units/modules/network/aci/test_aci_rest.py @@ -25,9 +25,12 @@ from ansible.modules.network.aci.aci_rest import aci_response from nose.plugins.skip import SkipTest -from lxml import etree -if sys.version_info >= (2, 7): - from xmljson import cobra +try: + from lxml import etree + if sys.version_info >= (2, 7): + from xmljson import cobra +except ImportError: + raise SkipTest("aci Ansible modules require the lxml and xmljson Python libraries") class AciRest(unittest.TestCase):