Merge pull request #400 from hpk/master

Remove optional dependencies from install_requires

Author: rohe

setup.py

@@ -8,12 +8,8 @@
 
 install_requires = [
     # core dependencies
-    'decorator',
     'requests >= 1.0.0',
     'future',
-    'paste',
-    'zope.interface',
-    'repoze.who',
     'cryptography',
     'pytz',
     'pyOpenSSL',
@@ -22,6 +18,14 @@
     'six'
 ]
 
+extras_require = {
+    's2repoze': [
+        'paste',
+        'zope.interface',
+        'repoze.who'
+    ]
+}
+
 version = ''
 with open('src/saml2/__init__.py', 'r') as fd:
     version = re.search(r'^__version__\s*=\s*[\'"]([^\'"]*)[\'"]',
@@ -56,5 +60,6 @@
     scripts=["tools/parse_xsd2.py", "tools/make_metadata.py",
              "tools/mdexport.py", "tools/merge_metadata.py"],
     install_requires=install_requires,
+    extras_require=extras_require,
     zip_safe=False,
 )

tests/test_40_sigver.py

@@ -18,8 +18,7 @@
 from saml2.samlp import response_from_string
 from saml2.s_utils import factory, do_attribute_statement
 
-#from pyasn1.codec.der import decoder
-
+import pytest
 from py.test import raises
 
 from pathutils import full_path
@@ -69,6 +68,10 @@ def _eq(l1, l2):
 wKe3ACFXBvqGQN0IbcH49hu0FKhYFM/GPDJcIHFBsiyMBXChpye9vBaTNEBCtU3K
 jjyG0hRT2mAQ9h+bkPmOvlEo/aH0xR68Z9hw4PF13w=="""
 
+try:
+    from pyasn1.codec.der import decoder
+except ImportError:
+    decoder = None
 
 
 def test_cert_from_instance_1():
@@ -81,16 +84,18 @@ def test_cert_from_instance_1():
     assert certs[0] == CERT1
 
 
-# def test_cert_from_instance_ssp():
-#     xml_response = open(SIMPLE_SAML_PHP_RESPONSE).read()
-#     response = samlp.response_from_string(xml_response)
-#     assertion = response.assertion[0]
-#     certs = sigver.cert_from_instance(assertion)
-#     assert len(certs) == 1
-#     assert certs[0] == CERT_SSP
-#     der = base64.b64decode(certs[0])
-#     print(str(decoder.decode(der)).replace('.', "\n."))
-#     assert decoder.decode(der)
+@pytest.mark.skipif(not decoder,
+                    reason="pyasn1 is not installed")
+def test_cert_from_instance_ssp():
+    xml_response = open(SIMPLE_SAML_PHP_RESPONSE).read()
+    response = samlp.response_from_string(xml_response)
+    assertion = response.assertion[0]
+    certs = sigver.cert_from_instance(assertion)
+    assert len(certs) == 1
+    assert certs[0] == CERT_SSP
+    der = base64.b64decode(certs[0])
+    print(str(decoder.decode(der)).replace('.', "\n."))
+    assert decoder.decode(der)
 
 
 class FakeConfig():

tests/test_60_sp.py

@@ -2,12 +2,17 @@
 # -*- coding: utf-8 -*-
 
 import base64
+import pytest
 from saml2.authn_context import INTERNETPROTOCOLPASSWORD
 from saml2.saml import NAMEID_FORMAT_TRANSIENT
 from saml2.samlp import NameIDPolicy
-from saml2.s2repoze.plugins.sp import make_plugin
 from saml2.server import Server
 
+try:
+    from saml2.s2repoze.plugins.sp import make_plugin
+except ImportError:
+    make_plugin = None
+
 ENV1 = {'SERVER_SOFTWARE': 'CherryPy/3.1.2 WSGI Server',
         'SCRIPT_NAME': '',
         'ACTUAL_SERVER_PROTOCOL': 'HTTP/1.1',
@@ -43,6 +48,8 @@
 }
 
 
+@pytest.mark.skipif(not make_plugin,
+                    reason="s2repoze dependencies not installed")
 class TestSP():
     def setup_class(self):
         self.sp = make_plugin("rem", saml_conf="server_conf")