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

[John Dennis]

On 02/26/2012 04:41 PM, Rob Crittenden wrote:
> 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

fixed, will appear in next patch revision


-- 
John Dennis <jdennis at redhat.com>

Looking to carve out IT costs?
www.redhat.com/carveoutcosts/