|
 |
89a1c0 |
commit 623d785f957acc9eccb47a9a3f88e5e167a370b6
|
|
|
89a1c0 |
Author: John Dennis <jdennis@redhat.com>
|
|
|
89a1c0 |
Date: Mon Jun 25 17:37:45 2018 -0400
|
|
|
89a1c0 |
|
|
|
89a1c0 |
fix duplicate definition of LogoutTestCase and logoutSuite
|
|
|
89a1c0 |
|
|
|
89a1c0 |
Commit 6f617027e added a duplicate definition of the LogoutTestCase
|
|
|
89a1c0 |
class containing only 1 test which shaddowed the original
|
|
|
89a1c0 |
LogoutTestCase containing 4 tests. The logoutSuite variable was also
|
|
|
89a1c0 |
shadowed and the allTests variable contained a duplicate of
|
|
|
89a1c0 |
logoutSuite causing the 2nd definition of LogoutTestCase to be run
|
|
|
89a1c0 |
twice.
|
|
|
89a1c0 |
|
|
|
89a1c0 |
Not only were the original 4 tests not being run but the entire unit
|
|
|
89a1c0 |
test in profiles_tests.py was failing under Python3. This is because
|
|
|
89a1c0 |
the unittest code in Py3 deletes a test from it's list of tests to run
|
|
|
89a1c0 |
once it's been run. The second time the logoutSuite was invoked it no
|
|
|
89a1c0 |
longer contained any tests which caused an exception to be raised
|
|
|
89a1c0 |
because there were no tests to be run.
|
|
|
89a1c0 |
|
|
|
89a1c0 |
License: MIT
|
|
|
89a1c0 |
Signed-off-by: John Dennis <jdennis@redhat.com>
|
|
|
89a1c0 |
|
|
|
89a1c0 |
diff --git a/bindings/python/tests/profiles_tests.py b/bindings/python/tests/profiles_tests.py
|
|
|
89a1c0 |
index 547c9e24..0ba1e56e 100755
|
|
|
89a1c0 |
--- a/bindings/python/tests/profiles_tests.py
|
|
|
89a1c0 |
+++ b/bindings/python/tests/profiles_tests.py
|
|
|
89a1c0 |
@@ -386,6 +386,21 @@ class LogoutTestCase(unittest.TestCase):
|
|
|
89a1c0 |
else:
|
|
|
89a1c0 |
self.fail('Logout processResponseMsg should have failed.')
|
|
|
89a1c0 |
|
|
|
89a1c0 |
+ def test05(self):
|
|
|
89a1c0 |
+ '''Test parsing of a logout request with more than one session index'''
|
|
|
89a1c0 |
+ content = '''<samlp:LogoutRequest xmlns:samlp="urn:oasis:names:tc:SAML:2.0:protocol" xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion" ID="xxxx" Version="2.0" IssueInstant="2010-06-14T22:00:00">
|
|
|
89a1c0 |
+ <saml:Issuer>me</saml:Issuer>
|
|
|
89a1c0 |
+ <saml:NameID>coin</saml:NameID>
|
|
|
89a1c0 |
+ <samlp:SessionIndex>id1</samlp:SessionIndex>
|
|
|
89a1c0 |
+ <samlp:SessionIndex>id2</samlp:SessionIndex>
|
|
|
89a1c0 |
+ <samlp:SessionIndex>id3</samlp:SessionIndex>
|
|
|
89a1c0 |
+ </samlp:LogoutRequest>'''
|
|
|
89a1c0 |
+
|
|
|
89a1c0 |
+ node = lasso.Samlp2LogoutRequest.newFromXmlNode(content)
|
|
|
89a1c0 |
+ assert isinstance(node, lasso.Samlp2LogoutRequest)
|
|
|
89a1c0 |
+ assert node.sessionIndex == 'id1'
|
|
|
89a1c0 |
+ assert node.sessionIndexes == ('id1', 'id2', 'id3')
|
|
|
89a1c0 |
+
|
|
|
89a1c0 |
class DefederationTestCase(unittest.TestCase):
|
|
|
89a1c0 |
def test01(self):
|
|
|
89a1c0 |
"""IDP initiated defederation; testing processNotificationMsg with non Liberty query."""
|
|
|
89a1c0 |
@@ -478,32 +493,15 @@ class AttributeAuthorityTestCase(unittest.TestCase):
|
|
|
89a1c0 |
assert aq.response.assertion[0].attributeStatement[0].attribute[0]
|
|
|
89a1c0 |
assert aq.response.assertion[0].attributeStatement[0].attribute[0].attributeValue[0]
|
|
|
89a1c0 |
|
|
|
89a1c0 |
-class LogoutTestCase(unittest.TestCase):
|
|
|
89a1c0 |
- def test01(self):
|
|
|
89a1c0 |
- '''Test parsing of a logout request with more than one session index'''
|
|
|
89a1c0 |
- content = '''<samlp:LogoutRequest xmlns:samlp="urn:oasis:names:tc:SAML:2.0:protocol" xmlns:saml="urn:oasis:names:tc:SAML:2.0:assertion" ID="xxxx" Version="2.0" IssueInstant="2010-06-14T22:00:00">
|
|
|
89a1c0 |
- <saml:Issuer>me</saml:Issuer>
|
|
|
89a1c0 |
- <saml:NameID>coin</saml:NameID>
|
|
|
89a1c0 |
- <samlp:SessionIndex>id1</samlp:SessionIndex>
|
|
|
89a1c0 |
- <samlp:SessionIndex>id2</samlp:SessionIndex>
|
|
|
89a1c0 |
- <samlp:SessionIndex>id3</samlp:SessionIndex>
|
|
|
89a1c0 |
- </samlp:LogoutRequest>'''
|
|
|
89a1c0 |
-
|
|
|
89a1c0 |
- node = lasso.Samlp2LogoutRequest.newFromXmlNode(content)
|
|
|
89a1c0 |
- assert isinstance(node, lasso.Samlp2LogoutRequest)
|
|
|
89a1c0 |
- assert node.sessionIndex == 'id1'
|
|
|
89a1c0 |
- assert node.sessionIndexes == ('id1', 'id2', 'id3')
|
|
|
89a1c0 |
-
|
|
|
89a1c0 |
serverSuite = unittest.makeSuite(ServerTestCase, 'test')
|
|
|
89a1c0 |
loginSuite = unittest.makeSuite(LoginTestCase, 'test')
|
|
|
89a1c0 |
logoutSuite = unittest.makeSuite(LogoutTestCase, 'test')
|
|
|
89a1c0 |
defederationSuite = unittest.makeSuite(DefederationTestCase, 'test')
|
|
|
89a1c0 |
identitySuite = unittest.makeSuite(IdentityTestCase, 'test')
|
|
|
89a1c0 |
attributeSuite = unittest.makeSuite(AttributeAuthorityTestCase, 'test')
|
|
|
89a1c0 |
-logoutSuite = unittest.makeSuite(LogoutTestCase, 'test')
|
|
|
89a1c0 |
|
|
|
89a1c0 |
allTests = unittest.TestSuite((serverSuite, loginSuite, logoutSuite, defederationSuite,
|
|
|
89a1c0 |
- identitySuite, attributeSuite, logoutSuite))
|
|
|
89a1c0 |
+ identitySuite, attributeSuite))
|
|
|
89a1c0 |
|
|
|
89a1c0 |
if __name__ == '__main__':
|
|
|
89a1c0 |
sys.exit(not unittest.TextTestRunner(verbosity = 2).run(allTests).wasSuccessful())
|