[PATCH] Add libsmbclient list shares test
Jeremy Allison
jra at samba.org
Tue Aug 28 22:38:54 UTC 2018
On Wed, Aug 29, 2018 at 10:19:30AM +1200, Andrew Bartlett wrote:
> On Tue, 2018-08-28 at 15:14 -0700, Jeremy Allison via samba-technical
> wrote:
> >
> > Ah, looking at the full log I see:
> >
> > Listing: smb://Administrator:locDCpass1@addc/tmp
> >
> > - which explains why it's listing the contents
> > of /tmp. Now, how did list_shares get invoked
> > with /tmp appended to the smb url ?
>
> To be clear [tmp] is one of the standard shares in all our selftest
> environments. It doesn't map to the host /tmp, just somewhere under
> our per virtual server selftest area.
Yeah, I know - I was just using /tmp as a shorthand.
What I'm trying to debug is the fact that the new
code has:
source4/selftest/tests.py:
libsmbclient = smbtorture4_testsuites("libsmbclient.")
for t in libsmbclient:
url = "smb://$USERNAME:$PASSWORD@$SERVER/tmp"
if "list_shares" in t:
url = "smb://$USERNAME:$PASSWORD@$SERVER"
so for the 'list_shares' test it should have
removed the /tmp from the end, and didn't.
More information about the samba-technical
mailing list