[Freeipa-devel] [PATCH 62] Tweak the session auth to reflect developer consensus.

[Rob Crittenden]

John Dennis wrote:
> rebased patch against current ipa-2-2 branch
>
>


I'm seeing two test failures with patches 62, 63 and 64 applied:

======================================================================
ERROR: tests.test_ipaserver.test_rpcserver.test_session.test_route
----------------------------------------------------------------------
Traceback (most recent call last):
   File "/usr/lib/python2.7/site-packages/nose/case.py", line 187, in 
runTest
     self.test(*self.arg)
   File 
"/home/rcrit/redhat/freeipa-review/tests/test_ipaserver/test_rpcserver.py", 
line 107, in test_route
     assert inst.route(d, None) == ('from 1', ['/ipa/foo', '/stuff'])
   File "/home/rcrit/redhat/freeipa-review/ipaserver/rpcserver.py", line 
231, in route
     return not_found(environ, start_response)
   File "/home/rcrit/redhat/freeipa-review/ipaserver/rpcserver.py", line 
105, in not_found
     start_response(status, response_headers)
TypeError: 'NoneType' object is not callable

======================================================================
FAIL: Test the `ipalib.errors.public_errors` module variable.
----------------------------------------------------------------------
Traceback (most recent call last):
   File "/usr/lib/python2.7/site-packages/nose/case.py", line 187, in 
runTest
     self.test(*self.arg)
   File 
"/home/rcrit/redhat/freeipa-review/tests/test_ipalib/test_errors.py", 
line 336, in test_public_errors
     assert m is not None, "need '**ERRNO**' in %s docstring" % 
klass.__name__
AssertionError: need '**ERRNO**' in InvalidSessionPassword docstring