oleaut32/tests: add tests for OleLoadPictureFile(Ex) and OleLoadPicturePath (resend) [2/2]

Jeremy Drake wine at jdrake.com
Thu Jan 15 00:28:26 CST 2009


On Wed, 14 Jan 2009, Alexandre Julliard wrote:

> Yes, it's better to avoid it, because then you may be testing things
> that you know the function is doing internally but that may not actually
> matter. Tests have to treat the target dll as a black box, and if you
> looked inside then it's no longer a true black box. Sorry about that,
> but we really have to play it safe, for obvious reasons.

I understand.  But, can we at least get the [1/2] patch in, that adds
stubs for OleLoadPictureFile and OleLoadPictureFileEx that just trace
FIXMEs and return E_NOTIMPL?  There's definitely nothing in there that I
could have learned from disassembly, and it would at least keep my app
from crashing.



More information about the wine-devel mailing list