On Tue, Apr 17, 2012 at 3:48 PM, Johan Corveleyn <jcor...@gmail.com> wrote: > On Tue, Apr 17, 2012 at 10:42 AM, Philip Martin > <philip.mar...@wandisco.com> wrote: >> Johan Corveleyn <jcor...@gmail.com> writes: >> >>> Apparently, the test is comparing the 'Repository Root' from the >>> output of 'svn info' with the expected repository url. When I run 'svn >>> info' manually on ...\svn-test-work\working_copies\basic_tests-29, I >>> see the following: >>> >>> [[[ >>> Path: >>> R:\test\subversion\tests\cmdline\svn-test-work\working_copies\basic_tests-29 >>> Working Copy Root Path: >>> R:\test\subversion\tests\cmdline\svn-test-work\working_copies\basic_tests-29 >>> URL: >>> file:///R%7C/test/subversion/tests/cmdline/svn-test-work/local_tmp/repos >>> Repository Root: >>> file:///R%7C/test/subversion/tests/cmdline/svn-test-work/local_tmp/repos >>> ]]] >>> >>> So the '%7C' instead of ':' seems to be the problem (BTW, %7C is the >>> pipe symbol, which seems quite strange here). >>> >>> When I use the same svn binary to do a new checkout of the same >>> repository, over file://, I don't get this problem. So that indicates >>> it's a problem with the test suite. >> >> tests.log should show the exact command used to do the checkout. Is it >> the same as the command you are running manually? > > Gahh! For some reason tests.log doesn't show much detail: > > [[[ > ... > PASS: basic_tests.py 26: basic info command > PASS: basic_tests.py 27: ignore files matching local ignores in added dirs > PASS: basic_tests.py 28: add ignored files in added dirs > 2012-04-17 22:38:44 [WARNING] Bad or missing repository root > 2012-04-17 22:38:44 [WARNING] > Traceback (most recent call last): > File > "C:\research\svn\client_build\trunk2\subversion\tests\cmdline\svntest\main.py", > line 1332, in run > rc = self.pred.run(sandbox) > File > "C:\research\svn\client_build\trunk2\subversion\tests\cmdline\svntest\testcase.py", > line 176, in run > return self.func(sandbox) > File > "C:\research\svn\client_build\trunk2\subversion\tests\cmdline\basic_tests.py", > line 1709, in repos_root > check_repos_root(output) > File > "C:\research\svn\client_build\trunk2\subversion\tests\cmdline\basic_tests.py", > line 1703, in check_repos_root > raise svntest.Failure > Failure > FAIL: basic_tests.py 29: check that repos root gets set on checkout > PASS: basic_tests.py 30: checks peg revision on filename with @ sign > PASS: basic_tests.py 31: info on file not existing in HEAD > ... > ]]] > > I tried adding -v to win-tests.py, but that didn't increase the > verbosity level of the logging. > > What am I missing? How can I get the usual (old) verbosity back in the log > file? > > I remember reading commits and some discussion about using the python > logging framework from the test-suite ... but don't know the details. > Is this supposed to be backward compatible, and show the same > log-level as there used to be?
The logging framework has a significant amount of granularity, but in replacing print() with logger.foo(), we may have erred on the side of being a bit too conservative in the amount of stuff printed by default. My guess is that the information you are looking for is currently hidden by logger.info(), but should be logger.warn(). -Hyrum -- uberSVN: Apache Subversion Made Easy http://www.uberSVN.com/
