samba-tool:testparm: Fix error with --section-name

Pass the correct parameters into LoadparmService.dump() so that
--section-name works properly.

BUG: https://bugzilla.samba.org/show_bug.cgi?id=14143

Signed-off-by: Joseph Sutton <josephsutton@catalyst.net.nz>
Reviewed-by: Rowland Penny <rpenny@samba.org>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>

diff --git a/python/samba/netcmd/testparm.py b/python/samba/netcmd/testparm.py
index aef24bf5c8dd8cbb62f6077f6e478f55d052896d..5219b8c6ad95b75b889f1f9f4d89bc4b52897d18 100644 (file)
--- a/python/samba/netcmd/testparm.py
+++ b/python/samba/netcmd/testparm.py
@@ -103,7 +103,7 @@ class cmd_testparm(Command):
         else:
             if section_name is not None or parameter_name is not None:
                 if parameter_name is None:
-                    lp[section_name].dump(verbose)
+                    lp[section_name].dump(lp.default_service, verbose)
                 else:
                     lp.dump_a_parameter(parameter_name, section_name)
             else:

diff --git a/selftest/knownfail.d/bug-14143 b/selftest/knownfail.d/bug-14143
deleted file mode 100644 (file)
index f4f9bc0..0000000
--- a/selftest/knownfail.d/bug-14143
+++ /dev/null
@@ -1 +0,0 @@
-^samba.tests.netcmd.samba.tests.netcmd.TestParmTests.test_section