setupapi: add case sensitivity string table tests [try 2]
Mike McCormack
mike at codeweavers.com
Wed May 3 21:25:27 CDT 2006
Hi Andrew,
Sorry I didn't spot this the first time round:
Andrew Ziem wrote:
> + /* case insensitive */
> retval=pStringTableAddString(table,string,0);
> ok(retval!=-1,"Failed to add string to String Table\n");
> +
> + retval2=pStringTableAddString(table,String,0);
> + ok(retval2!=-1,"Failed to add String to String Table\n");
You get a much better regression test by checking for an exact match of
the output. Only retval2==-1 will make the above test fail. What about
-2? Assuming retval2 should be zero, how about:
ok(retval2==0,"Failed to add String to String Table\n")
> + ok(retval=retval2,"string and String have different IDs in String Table\n");
Did you mean to use == here?
Mike
More information about the wine-devel
mailing list