[SCM] Samba Shared Repository - branch master updated
Jelmer Vernooij
jelmer at samba.org
Sat Sep 11 15:19:39 MDT 2010
Hi Matthias,
On Sat, 2010-09-11 at 22:55 +0200, Matthias Dieter Wallnöfer wrote:
> first this patch originally wasn't by me - I've only integrated it. But
> it is really bad style to use such "manual" testing code? And there are
> some more places where this is used - always in the same file.
Well, you took responsibility for it when you pushed it...
Either way, we should be moving away from printing test results manually
and rather try to use the convenience functions in more places. I
realize there are still some places where they're not used yet but we
should be reducing that number, not adding to it.
Cheers,
Jelmer
> Jelmer Vernooij wrote:
> > Hi Matthias,
> >
> > On Sat, 2010-09-11 at 19:15 +0200, Matthias Dieter Wallnöfer wrote:
> >
> >> @@ -173,11 +174,19 @@ testit "wbinfo -D against $TARGET" $wbinfo -D $DOMAIN || failed=`expr $failed +
> >>
> >> testit "wbinfo -i against $TARGET" $wbinfo -i "$DOMAIN/$USERNAME" || failed=`expr $failed + 1`
> >>
> >> -testit "wbinfo --uid-info against $TARGET" $wbinfo --uid-info $admin_uid
> >> +testit "wbinfo --uid-info against $TARGET" $wbinfo --uid-info $admin_uid || failed=`expr $failed + 1`
> >>
> >> -# this does not work
> >> -knownfail "wbinfo --group-info against $TARGET" $wbinfo --group-info "S-1-22-2-0"
> >> -knownfail "wbinfo --gid-info against $TARGET" $wbinfo --gid-info 30001
> >> +echo "test: wbinfo --group-info against $TARGET"
> >> +rawgid=`$wbinfo --group-info "Domain admins" | sed 's/.*:\([0-9][0-9]*\):/\1/'`
> >> +if test x$? = x0; then
> >> + echo "success: wbinfo --group-info against $TARGET"
> >> +else
> >> + echo "failure: wbinfo --group-info against $TARGET"
> >> + failed=`expr $failed + 1`
> >> +fi
> >>
> > Is there any reason for manually printing here rather than using the
> > testit function?
> >
> > Cheers,
> >
> > Jelmer
> >
> >
>
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: This is a digitally signed message part
URL: <http://lists.samba.org/pipermail/samba-technical/attachments/20100911/5bb50c0a/attachment.pgp>
More information about the samba-technical
mailing list