From 211275fae6895cdedbbe25ca3e41d0a856c1a370 Mon Sep 17 00:00:00 2001 From: Douglas Bagnall Date: Thu, 28 Jun 2018 16:19:31 +1200 Subject: [PATCH] samba-tool showrepl tests: test all-good with --pull-summary We test the all-good case with --pull-summary, which is the only one we can be reasonably certain about. Signed-off-by: Douglas Bagnall Reviewed-by: Andrew Bartlett Autobuild-User(master): Douglas Bagnall Autobuild-Date(master): Thu Jun 28 09:23:10 CEST 2018 on sn-devel-144 --- selftest/flapping.d/samba_tool_drs_showrepl | 1 - .../torture/drs/python/samba_tool_drs_showrepl.py | 12 ++++++------ 2 files changed, 6 insertions(+), 7 deletions(-) delete mode 100644 selftest/flapping.d/samba_tool_drs_showrepl diff --git a/selftest/flapping.d/samba_tool_drs_showrepl b/selftest/flapping.d/samba_tool_drs_showrepl deleted file mode 100644 index ffacea8f6c1..00000000000 --- a/selftest/flapping.d/samba_tool_drs_showrepl +++ /dev/null @@ -1 +0,0 @@ -^samba4.drs.samba_tool_drs_showrepl.python.+samba_tool_drs_showrepl.SambaToolDrsShowReplTests.test_samba_tool_showrepl_pull_summary_all_good diff --git a/source4/torture/drs/python/samba_tool_drs_showrepl.py b/source4/torture/drs/python/samba_tool_drs_showrepl.py index 2fe8ab5b638..40949553eed 100644 --- a/source4/torture/drs/python/samba_tool_drs_showrepl.py +++ b/source4/torture/drs/python/samba_tool_drs_showrepl.py @@ -222,8 +222,8 @@ class SambaToolDrsShowReplTests(drs_base.DrsBaseTestCase): src, dest = dc, remote self._net_drs_replicate(dest, src, forced=True) - def test_samba_tool_showrepl_summary_all_good(self): - """Tests 'samba-tool drs showrepl --summary' command.""" + def test_samba_tool_showrepl_pull_summary_all_good(self): + """Tests 'samba-tool drs showrepl --pull-summary' command.""" # To be sure that all is good we need to force replication # with everyone (because others might have it turned off), and # turn replication on for them in case they suddenly decide to @@ -236,20 +236,20 @@ class SambaToolDrsShowReplTests(drs_base.DrsBaseTestCase): self._force_all_reps(samdb1, self.dc1, 'inbound') self._force_all_reps(samdb1, self.dc1, 'outbound') - out = self.check_output("samba-tool drs showrepl --summary %s %s" % + out = self.check_output("samba-tool drs showrepl --pull-summary %s %s" % (self.dc1, self.cmdline_creds)) self.assertStringsEqual(out, "[ALL GOOD]\n") - out = self.check_output("samba-tool drs showrepl --summary " + out = self.check_output("samba-tool drs showrepl --pull-summary " "--color=yes %s %s" % (self.dc1, self.cmdline_creds)) self.assertStringsEqual(out, "\033[1;32m[ALL GOOD]\033[0m\n") # --verbose output is still quiet when all is good. - out = self.check_output("samba-tool drs showrepl --summary -v %s %s" % + out = self.check_output("samba-tool drs showrepl --pull-summary -v %s %s" % (self.dc1, self.cmdline_creds)) self.assertStringsEqual(out, "[ALL GOOD]\n") - out = self.check_output("samba-tool drs showrepl --summary -v " + out = self.check_output("samba-tool drs showrepl --pull-summary -v " "--color=yes %s %s" % (self.dc1, self.cmdline_creds)) self.assertStringsEqual(out, "\033[1;32m[ALL GOOD]\033[0m\n")