[svn.haxx.se] · SVN Dev · SVN Users · SVN Org · TSVN Dev · TSVN Users · Subclipse Dev · Subclipse Users · this month's index

[PATCH] 1.5.x branch: python/cmdline

From: Alexander Kitaev <Alexander.Kitaev_at_svnkit.com>
Date: Mon, 17 Mar 2008 13:53:38 +0100

Hello,

I'm reposting this message because first time I specified branch
incorrectly and the message was overlooked by developers because of
that. Sorry.

Attached are patches that fixes number of minor issues in Subversion
python tests suite. Patch is against 1.5.x branch.

Explanation:

1. main.py patch

Recently we started to use client/server approach for running Subversion
python tests suite against SVNKit Subversion library. We use server that
accepts Subversion commands through the TCP/IP socket and client which
is a bash script - it reads parameters, stdin, certain environment
variables and then sends everything to the server.

To send data to the server we use netcat or, alternatively, bash
/dev/tcp/host/port device. In both case complete stdin have to be read
before sending data to the server. However, main.py doesn't close stdin
of the launched process before launching another process (in particular
when running svnadmin dump repos1 | svnadmin load repos2 commands). As a
result our tests were entering a deadlock. Suggested patch doesn't
change anything in tests behavior with native Subversion - it just
closes stdin of the launched process as soon as no more data is about to
be sent to the process stdin.

Another modification included into this patch is setting
SVN_CURRENT_TEST environment variable before running certain test. This
allows us to track what test is currently running when the whole test
suite is launched (i.e. "./copy.py" without specifying exact test number
to run).

2. copy_tests.py patch

Copy 12 test expects "sleep_for_timestamp" behavior to be enabled (that
is disabled by default for tests). As a result sometimes copy 12 pass
and sometimes not. Patch enables sleep_for_timestamp for copy 12 test.

Thanks!

-- 
Alexander Kitaev,
TMate Software,
http://svnkit.com/ - Java [Sub]Versioning Library!

Index: copy_tests.py
===================================================================
--- copy_tests.py (revision 29877)
+++ copy_tests.py (working copy)
@@ -900,7 +900,9 @@
   pi_path = os.path.join(wc_dir, 'pi')
 
   svntest.actions.run_and_verify_svn(None, None, [], 'copy', E_url, wc_dir)
+ svntest.actions.do_sleep_for_timestamps()
   svntest.actions.run_and_verify_svn(None, None, [], 'copy', pi_url, wc_dir)
+ svntest.actions.no_sleep_for_timestamps()
 
   # Extra test: modify file ASAP to check there was a timestamp sleep
   svntest.main.file_append(pi_path, 'zig\n')

Index: svntest/main.py
===================================================================
--- svntest/main.py (revision 29877)
+++ svntest/main.py (working copy)
@@ -638,6 +638,7 @@
 
   dump_in, dump_out, dump_err, dump_kid = \
            open_pipe(svnadmin_binary + dump_args, 'b')
+ dump_in.close()
   load_in, load_out, load_err, load_kid = \
            open_pipe(svnadmin_binary + load_args, 'b')
   stop = time.time()
@@ -653,7 +654,6 @@
 
   dump_lines = dump_err.readlines()
   load_lines = load_out.readlines()
- dump_in.close()
   dump_out.close()
   dump_err.close()
   load_out.close()
@@ -1052,6 +1052,7 @@
     # Tests that want to use an editor should invoke svntest.main.use_editor.
     os.environ['SVN_EDITOR'] = ''
     os.environ['SVNTEST_EDITOR_FUNC'] = ''
+ os.environ['SVN_CURRENT_TEST'] = os.path.basename(sys.argv[0]) + "_" + str(self.index)
     actions.no_sleep_for_timestamps()
 
     saved_dir = os.getcwd()

---------------------------------------------------------------------
To unsubscribe, e-mail: dev-unsubscribe_at_subversion.tigris.org
For additional commands, e-mail: dev-help_at_subversion.tigris.org
Received on 2008-03-17 13:54:31 CET

This is an archived mail posted to the Subversion Dev mailing list.

This site is subject to the Apache Privacy Policy and the Apache Public Forum Archive Policy.