You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@subversion.apache.org by Branko Čibej <br...@wandisco.com> on 2012/12/14 20:15:05 UTC

FAIL: basic_tests.py 48: basic auth test

Is anyone else seeing this? Running with current trunk and

$ make davautocheck PARALLEL=1 TESTS=subversion/tests/cmdline/basic_tests.py


This is the only failure in the test suite.

-- Brane

-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com


Re: FAIL: basic_tests.py 48: basic auth test

Posted by Ben Reser <be...@reser.org>.
On Fri, Dec 14, 2012 at 12:54 PM, Branko Čibej <br...@wandisco.com> wrote:
> That's another interesting bit: davautocheck happily deletes the HTTP
> directory without asking. Or rather it asks, but just answers "yes" to
> itself. Digging ...

That bit works fine here.  Though it does have a 32 second timeout
which gives the default answer if you don't answer in that time frame.

Re: FAIL: basic_tests.py 48: basic auth test

Posted by Branko Čibej <br...@wandisco.com>.
On 14.12.2012 21:28, Ben Reser wrote:
> The odd part there is that it's actually the first --non-interactive
> checkout that's failing in that test. What's in the logs for httpd?
> Anything interesting? 

That's another interesting bit: davautocheck happily deletes the HTTP
directory without asking. Or rather it asks, but just answers "yes" to
itself. Digging ...

-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com


Re: FAIL: basic_tests.py 48: basic auth test

Posted by Branko Čibej <br...@wandisco.com>.
On 14.12.2012 22:12, Branko Čibej wrote:
> On 14.12.2012 21:28, Ben Reser wrote:
>> The odd part there is that it's actually the first --non-interactive
>> checkout that's failing in that test. 
> Sigh. I keep forgetting that I configure with
>
>     --disable-plaintext-password-storage.
>
> The test depends on storing the credentials from one command to the
> next, and since --non-interactive implies it can't even ask about
> storing the password in the keychain, it eventually fails.
>
> The test should probably be split in two parts, one that does and one
> that doesn't depend on that configure option, so that we can
> conditioinally skip the failing part.

For now, I fixed this in r1422171 by adding a conditioinal @XFail
decorator to the test. The actual predicate implementation is not
perfect, but it'll do until something better comes along.


-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com


Re: FAIL: basic_tests.py 48: basic auth test

Posted by Branko Čibej <br...@wandisco.com>.
On 14.12.2012 21:28, Ben Reser wrote:
> The odd part there is that it's actually the first --non-interactive
> checkout that's failing in that test. 

Sigh. I keep forgetting that I configure with

    --disable-plaintext-password-storage.

The test depends on storing the credentials from one command to the
next, and since --non-interactive implies it can't even ask about
storing the password in the keychain, it eventually fails.

The test should probably be split in two parts, one that does and one
that doesn't depend on that configure option, so that we can
conditioinally skip the failing part.

-- Brane

-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com


Re: FAIL: basic_tests.py 48: basic auth test

Posted by Ben Reser <be...@reser.org>.
On Fri, Dec 14, 2012 at 11:54 AM, Branko Čibej <br...@wandisco.com> wrote:
> On 14.12.2012 20:38, Ben Reser wrote:
>> On Fri, Dec 14, 2012 at 11:15 AM, Branko Čibej <br...@wandisco.com> wrote:
>>> Is anyone else seeing this? Running with current trunk and
>>>
>>> $ make davautocheck PARALLEL=1 TESTS=subversion/tests/cmdline/basic_tests.py
>>>
>>>
>>> This is the only failure in the test suite.
>> Not happening here.  What's the output of the test that's failing?
>
> W: ../trunk/subversion/svn/checkout-cmd.c:168: (apr_err=215004)
> W: ../trunk/subversion/libsvn_client/checkout.c:103: (apr_err=215004)
> W: ../trunk/subversion/libsvn_client/ra.c:497: (apr_err=215004)
> W: ../trunk/subversion/libsvn_client/ra.c:363: (apr_err=215004)
> W: ../trunk/subversion/libsvn_ra/ra_loader.c:482: (apr_err=215004)
> W: svn: E215004: Unable to connect to a repository at URL 'http://localhost:5475/svn-test-work/local_tmp/repos'
> W: ../trunk/subversion/libsvn_ra_serf/util.c:819: (apr_err=215004)
> W: ../trunk/subversion/libsvn_ra_serf/util.c:776: (apr_err=215004)
> W: ../trunk/subversion/libsvn_ra_serf/util.c:1752: (apr_err=215004)
> W: svn: E215004: No more credentials or we tried too manytimes.
> W: Authentication failed
> W:
> Traceback (most recent call last):
>   File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/main.py", line 1424, in run
>     rc = self.pred.run(sandbox)
>   File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/testcase.py", line 176, in run
>     return self.func(sandbox)
>   File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/basic_tests.py", line 2517, in basic_auth_test
>     '--username', 'jrandom', '--non-interactive', '--config-dir', config_dir)
>   File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/main.py", line 354, in run_command
>     None, *varargs)
>   File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/main.py", line 538, in run_command_stdin
>     raise Failure
> Failure
> FAIL:  basic_tests.py 48: basic auth test

The odd part there is that it's actually the first --non-interactive
checkout that's failing in that test.

What's in the logs for httpd?  Anything interesting?

Re: FAIL: basic_tests.py 48: basic auth test

Posted by Branko Čibej <br...@wandisco.com>.
On 14.12.2012 20:38, Ben Reser wrote:
> On Fri, Dec 14, 2012 at 11:15 AM, Branko Čibej <br...@wandisco.com> wrote:
>> Is anyone else seeing this? Running with current trunk and
>>
>> $ make davautocheck PARALLEL=1 TESTS=subversion/tests/cmdline/basic_tests.py
>>
>>
>> This is the only failure in the test suite.
> Not happening here.  What's the output of the test that's failing?

W: ../trunk/subversion/svn/checkout-cmd.c:168: (apr_err=215004)
W: ../trunk/subversion/libsvn_client/checkout.c:103: (apr_err=215004)
W: ../trunk/subversion/libsvn_client/ra.c:497: (apr_err=215004)
W: ../trunk/subversion/libsvn_client/ra.c:363: (apr_err=215004)
W: ../trunk/subversion/libsvn_ra/ra_loader.c:482: (apr_err=215004)
W: svn: E215004: Unable to connect to a repository at URL 'http://localhost:5475/svn-test-work/local_tmp/repos'
W: ../trunk/subversion/libsvn_ra_serf/util.c:819: (apr_err=215004)
W: ../trunk/subversion/libsvn_ra_serf/util.c:776: (apr_err=215004)
W: ../trunk/subversion/libsvn_ra_serf/util.c:1752: (apr_err=215004)
W: svn: E215004: No more credentials or we tried too manytimes.
W: Authentication failed
W: 
Traceback (most recent call last):
  File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/main.py", line 1424, in run
    rc = self.pred.run(sandbox)
  File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/testcase.py", line 176, in run
    return self.func(sandbox)
  File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/basic_tests.py", line 2517, in basic_auth_test
    '--username', 'jrandom', '--non-interactive', '--config-dir', config_dir)
  File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/main.py", line 354, in run_command
    None, *varargs)
  File "/Users/brane/src/svn/repos/trunk/subversion/tests/cmdline/svntest/main.py", line 538, in run_command_stdin
    raise Failure
Failure
FAIL:  basic_tests.py 48: basic auth test


-- 
Branko Čibej
Director of Subversion | WANdisco | www.wandisco.com


Re: FAIL: basic_tests.py 48: basic auth test

Posted by Ben Reser <be...@reser.org>.
On Fri, Dec 14, 2012 at 11:15 AM, Branko Čibej <br...@wandisco.com> wrote:
> Is anyone else seeing this? Running with current trunk and
>
> $ make davautocheck PARALLEL=1 TESTS=subversion/tests/cmdline/basic_tests.py
>
>
> This is the only failure in the test suite.

Not happening here.  What's the output of the test that's failing?