summaryrefslogtreecommitdiff
path: root/testprogs
Commit message (Collapse)AuthorAgeFilesLines
* testprocs/blackbox: Add tests for net ads setspn (add|delete|list)Noel Power2018-03-021-0/+20
| | | | | | Signed-off-by: Noel Power <noel.power@suse.com> Reviewed-by: Jeremy Allison <jra@samba.org> Reviewed-by: Andreas Schneider <asn@samba.org>
* Fix autobuild for user names starting with cChristof Schmitt2018-03-021-8/+8
| | | | | | | | | | | | | | | | | | | | | | | | This is similar to the fix from commit 0b1ba00: Now commit 475a7616 introduced tests that create user names in the form DOMAIN\USER and pass them through shell functions. There "echo" is used to print he username, resulting in /bin/dash on sn-devel to interpret e.g. \c which skips any further output. The result are test exceptions like: failure: samba4.blackbox.trust_ntlm.Test01 rpcclient lookupnames with LOCALADMEMBER\cs(ad_member:local) [ Exception: Exception: LOCALADMEMBER ] time: 2018-02-26 23:00:46.688800Z Fix this by replacing the echo with printf %s. This surfaced for test_rpcclient_grep, but apply the same change to all functions in common_test_fns.inc for consistency. Signed-off-by: Christof Schmitt <cs@samba.org> Reviewed-by: Andrew Bartlett <abartlet@samba.org> Autobuild-User(master): Christof Schmitt <cs@samba.org> Autobuild-Date(master): Fri Mar 2 10:49:09 CET 2018 on sn-devel-144
* Remove some bashisms from the test scriptsTimur I. Bakeyev2018-03-011-3/+3
| | | | | | Signed-off-by: Timur I. Bakeyev <timur@iXsystems.com> Reviewed-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andreas Schneider <asn@samba.org>
* tests/dbcheck: Provision using the old schema and ignore displayNameGarming Sam2018-03-011-2/+5
| | | | | | | These tests are currently not run on Ubuntu due to bashisms in the test. Signed-off-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andreas Schneider <asn@samba.org>
* test_trust_ntlm.sh: add lookup name testsStefan Metzmacher2018-02-211-18/+59
| | | | | | | Bug: https://bugzilla.samba.org/show_bug.cgi?id=13286 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org>
* Fix incorrect `ldbsearch` invocationTimur I. Bakeyev2018-02-191-1/+1
| | | | | | | | | Signed-off-by: Timur I. Bakeyev <timur@iXsystems.com> Reviewed-by: Uri Simchoni <uri@samba.org> Reviewed-by: Björn Jacke <bjacke@samba.org> Autobuild-User(master): Uri Simchoni <uri@samba.org> Autobuild-Date(master): Mon Feb 19 12:31:38 CET 2018 on sn-devel-144
* Remove some bashisms from the test scriptsTimur I. Bakeyev2018-02-191-2/+1
| | | | | | Signed-off-by: Timur I. Bakeyev <timur@iXsystems.com> Reviewed-by: Uri Simchoni <uri@samba.org> Reviewed-by: Christof Schmitt <cs@samba.org>
* selftest/dbcheck: add a test for corrupt forward links restorationRalph Boehme2018-02-051-0/+78
| | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=13228 Signed-off-by: Ralph Boehme <slow@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org>
* testprogs:blackbox: add regression test for unsorted links in ↵Stefan Metzmacher2018-01-221-0/+24
| | | | | | | | | tombstones-expunge.sh BUG: https://bugzilla.samba.org/show_bug.cgi?id=13228 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org>
* upgradeprovision: Mark tests as passing again (using functional prep)Garming Sam2017-12-201-0/+6
| | | | | Signed-off-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* functionalprep.sh: Add a test to show that functional prep works on old ↵Garming Sam2017-12-201-0/+39
| | | | | | | databases Signed-off-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* functionalprep.sh: New test for ensuring that the prep works correctlyGarming Sam2017-12-201-0/+76
| | | | | Signed-off-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* provision: Use the official MS 2008R2 schema by defaultAndrew Bartlett2017-12-142-9/+51
| | | | | | | | This fixes us to have the official adminDescription etc. While both schema were provided by Microsoft this is a better quality one, but still under the same licence. Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Garming Sam <garming@catalyst.net.nz>
* provision: RODC revision level should be at 2Garming Sam2017-12-141-1/+1
| | | | | | | | | | | | | | | | This number had been mistakenly updated alongside the standard forest updates revision. This version number appears to be independent of the other revision levels. Also add the change to a new .ldf file, which can be used to apply the schema change to an existing Samba 4.7 (or earlier) instance. Update the provision/upgrade test to do just this (otherwise it complains about differences between a new provision and an older Samba 4.0.0 instance). Signed-off-by: Garming Sam <garming@catalyst.net.nz> Signed-off-by: Tim Beale <timbeale@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* selftest: Add basic test for schema upgradeTim Beale2017-12-141-0/+80
| | | | | | | | | | | This tests that we can provision using both the 2008 and 2012 schema, that we can upgrade a 2008 Samba instance to use the 2012 schema, and that when we do that the result (more or less) matches a straight 2012 provision. Signed-off-by: Tim Beale <timbeale@catalyst.net.nz> Reviewed-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* upgradeprovision: Change test to always use 2008 R2 schemaTim Beale2017-12-141-1/+1
| | | | | | | | | | | | | | | | | This tool (and the corresponding test) is designed to migrate a Samba DC from a pre-4.0.0 release up to a more recent schema (i.e. Windows 2008R2). Going further than 2008R2 turns this test into a bit of a nightmare. We now have a better adprep/'samba-tool domain schemaupgrade' option for upgrading from 2008R2 to a more recent schema. It seems to make most sense to leave this tests just running against 2008R2 schema provisions and add new tests to migrate from 2008R2 to 2012R2. Signed-off-by: Tim Beale <timbeale@catalyst.net.nz> Reviewed-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* selftest: Fix upgradeprovision test by importing new objects for schema 45Andrew Bartlett2017-12-141-0/+4
| | | | | | | | | | | | | | The recent schema changes mean that the upgradeprovision test starts failing. This is because it's using an old 4.0.0 schema (that doesn't have these schema changes), but it's comparing it against a fresh provision (which does have the changes). We can avoid this failure by using the 'samba-tool domain schemaupgrade' to bring the old 4.0.0 schema in line with a fresh provision. Note that the 'upgradeprovision --full' test doesn't need this change as it seems to more aggressively copy over any schema differences with a fresh provision. Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Garming Sam <garming@catalyst.net.nz>
* schema: Re-work extended rights handling in provision (prep for 2012R2)Andrew Bartlett2017-12-141-1/+1
| | | | | | | | | | | | | | | | | | | | Add the changes needed to provision a 2012 DC (mostly this just affects the Extended Rights objects) by moving to the new extended-rights.ldif The localizationDisplayId is not documented in MS-ATDS so these values are moved to provision_configuation_modify.ldif and applied after the display-specifiers.ldif We don't enable the 2012R2 mode yet. The ${INC2012} variable just gets replaced with '#' so the lines get commented out and not applied. This approach allows us to support provisioning both a 2008R2 DC or a 2012R2 DC (so that we can test we can upgrade a 2008 DC to 2012). Signed-off-by: Tim Beale <timbeale@catalyst.net.nz> Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Garming Sam <garming@catalyst.net.nz>
* testprogs: Fix a typo in the net ads testNoel Power2017-11-301-1/+1
| | | | | | | | | Signed-off-by: Noel Power <noel.power@suse.com> Reviewed-by: Andreas Schneider <asn@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org> Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org> Autobuild-Date(master): Thu Nov 30 01:47:24 CET 2017 on sn-devel-144
* testprogs: Test net ads keytab listNoel Power2017-11-291-0/+13
| | | | | | | | | | Test that correct keytab is picked up. BUG: https://bugzilla.samba.org/show_bug.cgi?id=13166 Signed-off-by: Noel Power <noel.power@suse.com> Reviewed-by: Andreas Schneider <asn@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org>
* selftest: Add more corruption cases for runtime and dbcheckAndrew Bartlett2017-11-243-3/+86
| | | | | | | | These tests now confirm we can handle these issues at runtime as well as at dbcheck Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org>
* selftest: add dbcheck tests for duplicate linksStefan Metzmacher2017-11-241-0/+39
| | | | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=13095 Pair-Programmed-With: Andrew Bartlett <abartlet@samba.org> Signed-off-by: Stefan Metzmacher <metze@samba.org> Signed-off-by: Andrew Bartlett <abartlet@samba.org>
* selftest: Additional check for a backlink pointing at a deleted objectAndrew Bartlett2017-11-242-0/+9
| | | | | Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org>
* selftest: Split out creation of complex (often invalid) linksAndrew Bartlett2017-11-242-199/+200
| | | | | | | | This will allow us to test other run-time behaviour with broken databases. Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org>
* selftest: Split out dbcheck runs from dangling_multi_valued testAndrew Bartlett2017-11-241-3/+7
| | | | | Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org>
* selftest: add more dbcheck testsAndrew Bartlett2017-11-241-12/+81
| | | | | | | | This validates some more combinations and ensures that the changes in 962a1b32201fce0a49c6be55943d4fbb57ed781e are tested. Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Stefan Metzmacher <metze@samba.org>
* testprogs: Update the kpasswd setpassword testAndreas Schneider2017-10-271-5/+2
| | | | | | | | | | This has been fixed with b81ca4f9dcbb378a95fb3ac31bfd9a1cbe505d7d Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Ralph Boehme <slow@samba.org> Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org> Autobuild-Date(master): Fri Oct 27 15:22:43 CEST 2017 on sn-devel-144
* selftest: sort dbcheck output to avoid sort order impacting resultsAndrew Bartlett2017-09-201-1/+3
| | | | | | | The GUID index code will change the returned results order Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Garming Sam <garming@catalyst.net.nz>
* selftest: reindex in dbcheck-oldrelease after modifying the backend DBAndrew Bartlett2017-09-141-0/+3
| | | | | | | | Modifying the backend DB is not a supported operation, but helps us create test situations. Signed-off-by: Andrew Bartlett <abartlet@samba.org> Reviewed-by: Garming Sam <garming@catalyst.net.nz>
* replmd: Try to add forward-link for unknown cross-partition linksTim Beale2017-08-181-0/+8
| | | | | | | | | | | | | | | | | | | | | | | | | Previously Samba would just drop cross-partition links where the link target object is unknown. Instead, what we want to do is try to add the forward link for the GUID specified. We can't add the backlink because we don't know the target, however, dbcheck should be able to fix any missing backlinks. The new behaviour should now mean dbcheck will detect the problem and be able to fix it. It's still not ideal, but it's better than dropping the link completely. I've updated the log so that it has higher severity and tells the user what they need to do to fix it. These changes now mean that the selftests now detect an error - instead of completely dropping the serverReference, we now have a missing backlink. I've updated the selftests to fix up any missing serverReference backlinks before running dbcheck. Signed-off-by: Tim Beale <timbeale@catalyst.net.nz> Reviewed-by: Garming Sam <garming@samba.org> Reviewed-by: Andrew Bartlett <abartlet@samba.org> BUG: https://bugzilla.samba.org/show_bug.cgi?id=12972
* lib: talloc: Use the system <talloc.h> include.Jeremy Allison2017-08-171-1/+1
| | | | | | | | Signed-off-by: Jeremy Allison <jra@samba.org> Reviewed-by: Andreas Schneider <asn@samba.org> Autobuild-User(master): Jeremy Allison <jra@samba.org> Autobuild-Date(master): Thu Aug 17 00:53:48 CEST 2017 on sn-devel-144
* blackbox: Add test for 'net ads changetrustpw'Andreas Schneider2017-08-111-0/+4
| | | | | | | | | | BUG: BUG: https://bugzilla.samba.org/show_bug.cgi?id=12956 Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Richard Sharpe <realrichardsharpe@gmail.com> Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org> Autobuild-Date(master): Fri Aug 11 22:09:27 CEST 2017 on sn-devel-144
* testprogs/blackbox: don't use hardcoded values in test_net_ads_dns.shStefan Metzmacher2017-06-131-3/+7
| | | | | Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Andreas Schneider <asn@samba.org>
* testprogs: Add 'net rpc user' test against AD DCAndreas Schneider2017-05-111-0/+56
| | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs: Ignore escape characters when printing test nameChristof Schmitt2017-05-051-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | Long story: This was triggered by the addition of the test_trust_ntlm.sh script in commits 3caca9b and 2de1994. test_trust_ntlm.sh creates a variable CREDS="$REALM\\$USERNAME%$PASSWORD" that is then used as part of the test name. subunit.sh uses echo to print the name that is then picked up by subunithelper.py. test_trust_ntlm.sh also uses /bin/sh as shell which can be a POSIX compliant shell like dash. This combination broke 'make test' for any username starting with the letter c. In this case CREDS contains the escape sequence \c that is defined to stop producing further output at this point. dash implements this feature and the echo in subunit.sh as a result skips the output after \c, including skipping the newline. This means that the data received by subunithelper.py contains the timestamp from the next line in the test name, which then breaks the testcase tracking. Fix this by replacing the echo in subunit.sh with a printf that does not trigger the special handling of escape characters. Signed-off-by: Christof Schmitt <cs@samba.org> Reviewed-by: Volker Lendecke <vl@samba.org> Autobuild-User(master): Volker Lendecke <vl@samba.org> Autobuild-Date(master): Fri May 5 23:44:16 CEST 2017 on sn-devel-144
* testprogs: Add MIT Kerberos specific kpasswd blackbox testAndreas Schneider2017-04-291-0/+231
| | | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs: Add test with exported keytab from samba-toolAndreas Schneider2017-04-291-0/+127
| | | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs: Add a kinit trust test for MIT KDCAndreas Schneider2017-04-291-0/+139
| | | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs: Add test_kinit_mit.sh testAndreas Schneider2017-04-291-0/+310
| | | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs: Fix usage printout of bogus blackbox testAndreas Schneider2017-04-291-1/+1
| | | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs: Fix test_chgdcpass blackbox test with MITAndreas Schneider2017-04-291-2/+14
| | | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs: Add common kinit functionAndreas Schneider2017-04-291-0/+16
| | | | | | Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* lib/util: add a test for samba_runcmd_send()Ralph Boehme2017-04-201-0/+15
| | | | | Signed-off-by: Ralph Boehme <slow@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org>
* testprogs/blackbox: add test_trust_ntlm.shStefan Metzmacher2017-04-101-0/+164
| | | | | | | | | | | This verifies that various domain/realm and username combinations map to the correct user. BUG: https://bugzilla.samba.org/show_bug.cgi?id=2976 BUG: https://bugzilla.samba.org/show_bug.cgi?id=12709 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* testprogs/blackbox: add test_rpcclient_*_grep helper functionsStefan Metzmacher2017-04-101-0/+52
| | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12709 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* testprogs/blackbox: use subunit_ helper functions in test_smbclient_*Stefan Metzmacher2017-04-101-8/+8
| | | | | | | BUG: https://bugzilla.samba.org/show_bug.cgi?id=12709 Signed-off-by: Stefan Metzmacher <metze@samba.org> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* testprogs: Correctly expand shell parametersAndreas Schneider2017-03-151-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The old behaviour is: for var in $* do echo "$var" done And you get this: $ sh test.sh 1 2 '3 4' 1 2 3 4 Changing it to: for var in "$@" do echo "$var" done will correctly expand to: $ sh test.sh 1 2 '3 4' 1 2 3 4 Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Jeremy Allison <jra@samba.org> Autobuild-User(master): Andreas Schneider <asn@cryptomilk.org> Autobuild-Date(master): Wed Mar 15 05:26:17 CET 2017 on sn-devel-144
* testprogs: Test 'net ads join' with a dedicated keytabAndreas Schneider2017-03-141-0/+9
| | | | | | | | This checks that a 'net ads join' can create the keytab and make sure we will not regress in future. Signed-off-by: Andreas Schneider <asn@samba.org> Reviewed-by: Andrew Bartlet <abartlet@samba.org>
* tests/dbcheck-links: remove spurious sleepingGarming Sam2017-03-131-1/+0
| | | | | Signed-off-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
* dbcheck: Improve dbcheck to find (and may fix) dangling msDS-RevealedUsersGarming Sam2017-03-131-0/+77
| | | | | | | | We cannot add missing backlinks because of the duplicate checking. There seems to be no trivial way to add the bypass. Signed-off-by: Garming Sam <garming@catalyst.net.nz> Reviewed-by: Andrew Bartlett <abartlet@samba.org>
View Lorry Controller Status