summaryrefslogtreecommitdiff
path: root/selftest
diff options
context:
space:
mode:
authorAndrew Bartlett <abartlet@samba.org>2011-10-26 15:21:11 +1100
committerAndrew Bartlett <abartlet@samba.org>2011-10-28 13:10:27 +0200
commit8dcfe2e5c44184298b0aa5bb1a13e2108b31a9c4 (patch)
tree14a684c269b848dc36d9b553774e18e0c3501e93 /selftest
parentf54dcc86b8c04bfd3c85ad327d268f5e32dd2cfd (diff)
downloadsamba-8dcfe2e5c44184298b0aa5bb1a13e2108b31a9c4.tar.gz
samba-8dcfe2e5c44184298b0aa5bb1a13e2108b31a9c4.tar.bz2
samba-8dcfe2e5c44184298b0aa5bb1a13e2108b31a9c4.zip
selftest: Remove --target option and the ability to run 'samba4 only' tests
This simplifies the selftest system, and by default we always tested all of samba3 and samba4 in the waf build. This simply removes a rarely used option for testing only part of the system. The make test TESTS="^samba3" syntax remains unchanged, so no functionality is lost. Andrew Bartlett
Diffstat (limited to 'selftest')
-rwxr-xr-xselftest/selftest.pl12
-rw-r--r--selftest/wscript37
2 files changed, 16 insertions, 33 deletions
diff --git a/selftest/selftest.pl b/selftest/selftest.pl
index 78627c8eb6..f41ff33d53 100755
--- a/selftest/selftest.pl
+++ b/selftest/selftest.pl
@@ -26,7 +26,7 @@ selftest - Samba test runner
selftest --help
-selftest [--srcdir=DIR] [--bindir=DIR] [--exeext=EXT][--target=samba4|samba3|win|kvm] [--socket-wrapper] [--quick] [--exclude=FILE] [--include=FILE] [--one] [--prefix=prefix] [--testlist=FILE] [TESTS]
+selftest [--srcdir=DIR] [--bindir=DIR] [--exeext=EXT][--target=samba|samba3|win|kvm] [--socket-wrapper] [--quick] [--exclude=FILE] [--include=FILE] [--one] [--prefix=prefix] [--testlist=FILE] [TESTS]
=head1 DESCRIPTION
@@ -56,7 +56,7 @@ Executable extention
Change directory to run tests in. Default is 'st'.
-=item I<--target samba4|samba3|win|kvm>
+=item I<--target samba|samba3|win|kvm>
Specify test target against which to run. Default is 'samba4'.
@@ -142,7 +142,7 @@ if ($@) {
}
my $opt_help = 0;
-my $opt_target = "samba4";
+my $opt_target = "samba";
my $opt_quick = 0;
my $opt_socket_wrapper = 0;
my $opt_socket_wrapper_pcap = undef;
@@ -300,7 +300,7 @@ Usage: $Script [OPTIONS] TESTNAME-REGEX
Generic options:
--help this help page
- --target=samba[34]|win|kvm Samba version to target
+ --target=samba[3]|win|kvm Samba version to target
--testlist=FILE file to read available tests from
Paths:
@@ -490,10 +490,6 @@ if ($opt_target eq "samba") {
$testenv_default = "all";
require target::Samba;
$target = new Samba($bindir, \%binary_mapping, $ldap, $srcdir, $exeext, $server_maxtime);
-} elsif ($opt_target eq "samba4") {
- $testenv_default = "all";
- require target::Samba4;
- $target = new Samba4($bindir, \%binary_mapping, $ldap, $srcdir, $exeext, $server_maxtime);
} elsif ($opt_target eq "samba3") {
if ($opt_socket_wrapper and `$bindir/smbd -b | grep SOCKET_WRAPPER` eq "") {
die("You must include --enable-socket-wrapper when compiling Samba in order to execute 'make test'. Exiting....");
diff --git a/selftest/wscript b/selftest/wscript
index 558cb2abd2..c34658a1a7 100644
--- a/selftest/wscript
+++ b/selftest/wscript
@@ -30,9 +30,6 @@ def set_options(opt):
gr.add_option('--tests',
help=("wildcard pattern of tests to run"),
action="store", dest='TESTS', default='')
- gr.add_option('--target',
- help=("target of samba3 or samba4"),
- action="store", dest='TARGET', default=None)
gr.add_option('--filtered-subunit',
help=("output (xfail) filtered subunit"),
action="store_true", dest='FILTERED_SUBUNIT', default=False)
@@ -188,30 +185,20 @@ def cmd_testonly(opt):
if not os.path.isdir(env.SELFTEST_PREFIX):
os.makedirs(env.SELFTEST_PREFIX, int('755', 8))
- if Options.options.TARGET:
- env.SELFTEST_TARGET = Options.options.TARGET
- else:
- env.SELFTEST_TARGET = "samba"
-
- if env.SELFTEST_TARGET == "samba4":
- env.SELFTEST_DIR = "${srcdir}/source4/selftest"
- elif env.SELFTEST_TARGET == "samba3":
- env.SELFTEST_DIR = "${srcdir}/source3/selftest"
-
- if env.SELFTEST_TARGET == "samba":
- for f in ["knownfail", "slow", "quick", "skip" ]:
- combine_files(env.srcdir + "/source4/selftest/" + f,
- env.srcdir + "/source3/selftest/" + f,
- env.SELFTEST_PREFIX + "/" + f)
-
- env.SELFTEST_DIR = env.SELFTEST_PREFIX
- env.TESTLISTS = ('--testlist="${PYTHON} ${srcdir}/source3/selftest/tests.py|" ' +
- '--testlist="${PYTHON} ${srcdir}/source4/selftest/tests.py|"')
- else:
- env.TESTLISTS = '--testlist="${SELFTEST_DIR}/tests.py|"'
+ env.SELFTEST_TARGET = "samba"
+
+ for f in ["knownfail", "slow", "quick", "skip" ]:
+ combine_files(env.srcdir + "/source4/selftest/" + f,
+ env.srcdir + "/source3/selftest/" + f,
+ env.SELFTEST_PREFIX + "/" + f)
+
+ env.SELFTEST_DIR = env.SELFTEST_PREFIX
+
+ env.TESTLISTS = ('--testlist="${PYTHON} ${srcdir}/source3/selftest/tests.py|" ' +
+ '--testlist="${PYTHON} ${srcdir}/source4/selftest/tests.py|"')
# We use the full path rather than relative path because it cause problems on some plateforms (ie. solaris 8).
- cmd = '(${PERL} ${srcdir}/selftest/selftest.pl --target=${SELFTEST_TARGET} --prefix=${SELFTEST_PREFIX} --srcdir=${srcdir} --exclude=${SELFTEST_DIR}/skip ${TESTLISTS} ${OPTIONS} --socket-wrapper ${TESTS} && touch ${SELFTEST_PREFIX}/st_done) | ${FILTER_OPTIONS} | tee ${SELFTEST_PREFIX}/subunit'
+ cmd = '(${PERL} ${srcdir}/selftest/selftest.pl --target=samba --prefix=${SELFTEST_PREFIX} --srcdir=${srcdir} --exclude=${SELFTEST_DIR}/skip ${TESTLISTS} ${OPTIONS} --socket-wrapper ${TESTS} && touch ${SELFTEST_PREFIX}/st_done) | ${FILTER_OPTIONS} | tee ${SELFTEST_PREFIX}/subunit'
if os.environ.get('RUN_FROM_BUILD_FARM') is None and not Options.options.FILTERED_SUBUNIT:
cmd += ' | ${FORMAT_TEST_OUTPUT}'