Renamed LEC to LECP. It is really a proxy.

2 files changed, 13 insertions, 15 deletions

diff --git a/python/tests/LibertyEnabledClient.py b/python/tests/LibertyEnabledClientProxy.py
index f25f9c59..12871ca9 100644
--- a/python/tests/LibertyEnabledClient.py
+++ b/python/tests/LibertyEnabledClientProxy.py
@@ -27,7 +27,7 @@ import lasso
 from websimulator import *
 
 
-class LibertyEnabledClient(WebClient):
+class LibertyEnabledClientProxy(WebClient):
     # A service provider MAY provide a list of identity providers it recognizes by including the
     # <lib:IDPList> element in the <lib:AuthnRequestEnvelope>. The format and processing rules for
     # the identity provider list MUST be as defined in [LibertyProtSchema].
@@ -69,9 +69,7 @@ class LibertyEnabledClient(WebClient):
         #     'LIBV=urn:liberty:iff:2003-08,http://projectliberty.org/specs/v1'))
         'Accept': ','.join((httpRequestHeaders['Accept'], 'application/vnd.liberty-request+xml'))
         })
-    # FIXME: Lasso should provide a way for Liberty-enabled client to create a "lassoServer"
-    # without metadata, instead of using 'singleSignOnServiceUrl'.
-    idpSingleSignOnServiceUrl = None
+    idpSite = None # The identity provider, this LECP will use to authenticate users.
     lassoServerDump = None
     principal = None
 
diff --git a/python/tests/login_tests.py b/python/tests/login_tests.py
index 54827aee..2979f1ba 100644
--- a/python/tests/login_tests.py
+++ b/python/tests/login_tests.py
@@ -36,7 +36,7 @@ import lasso
 
 import builtins
 from IdentityProvider import IdentityProvider
-from LibertyEnabledClient import LibertyEnabledClient
+from LibertyEnabledClientProxy import LibertyEnabledClientProxy
 from ServiceProvider import ServiceProvider
 from websimulator import *
 
@@ -67,17 +67,17 @@ class LoginTestCase(unittest.TestCase):
         # Frederic Peters has no account on identity provider.
         return site
 
-    def generateLibertyEnabledClient(self, internet):
-        client = LibertyEnabledClient(internet)
+    def generateLibertyEnabledClientProxy(self, internet):
+        clientProxy = LibertyEnabledClientProxy(internet)
         lassoServer = lasso.Server.new()
         lassoServer.add_provider(
             '../../examples/data/idp-metadata.xml',
             '../../examples/data/idp-public-key.pem',
             '../../examples/data/ca-crt.pem')
-        client.lassoServerDump = lassoServer.dump()
-        failUnless(client.lassoServerDump)
+        clientProxy.lassoServerDump = lassoServer.dump()
+        failUnless(clientProxy.lassoServerDump)
         lassoServer.destroy()
-        return client
+        return clientProxy
         
     def generateSpSite(self, internet):
         site = ServiceProvider(internet, 'https://service-provider/')
@@ -254,19 +254,19 @@ class LoginTestCase(unittest.TestCase):
         principal = Principal(internet, 'Romain Chantereau')
         principal.keyring[idpSite.url] = 'Chantereau'
         principal.keyring[spSite.url] = 'Romain'
-        lec = self.generateLibertyEnabledClient(internet)
-        lec.idpSite = idpSite
+        lecp = self.generateLibertyEnabledClientProxy(internet)
+        lecp.idpSite = idpSite
 
         # Try LECP, but the principal is not authenticated on identity-provider. So, LECP must
         # fail.
-        httpResponse = lec.login(principal, spSite, '/login')
+        httpResponse = lecp.login(principal, spSite, '/login')
         failUnlessEqual(httpResponse.statusCode, 401)
 
         # Now we authenticate principal, before testing LECP. So, LECP must succeed.
         httpResponse = principal.sendHttpRequestToSite(spSite, 'GET', '/login')
         failUnlessEqual(httpResponse.statusCode, 200)
-        idpSite.createSession(lec)
-        httpResponse = lec.login(principal, spSite, '/login')
+        idpSite.createSession(lecp)
+        httpResponse = lecp.login(principal, spSite, '/login')
         failUnlessEqual(httpResponse.statusCode, 200)