[PATCH] Add libsmbclient list shares test
Jeremy Allison
jra at samba.org
Tue Aug 28 23:02:17 UTC 2018
On Wed, Aug 29, 2018 at 12:58:12AM +0200, Stefan Metzmacher wrote:
> Hi Jeremy,
>
> >> 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.
>
> The name is "libsmbclient.list_shares", not just "list_shares".
Yes I already found and fixed that. It still fails
due to the libsmbclient tests being invoked from
inside source3/selftest/tests.py, whereas Andreas's
change only fixed the smburl in source4/selftest/tests.py
Whoever thought (a) writing this stuff in python was a good
idea and (b) naming things identially inside source3 and
source4 is responsible for a *LOT* of wasted engineering
time and effort..
More information about the samba-technical
mailing list