[wintrust/tests] Add a few tests

Paul Vriens paul.vriens.wine at gmail.com
Wed Jan 7 15:31:52 CST 2009


Hi,

While looking at bug 16013 and checking the mentioned crypt tests I found some
oddities.

It started with the fact that we didn't remove the winetest.cat catalogfile as
we passed the temporary filename to CryptCATAdminRemoveCatalog. I then saw that
we didn't remove the unique catalog file as well.

On top of that I found that although CryptCATAdminRemoveCatalog succeeds
sometimes (from a return value POV) it doesn't remove the file if it's not correct.

As I wanted to check the existence of the catalog file I've made two global
variables catroot and catroot2 so we didn't need to do things twice.

I agree that the whole A->W and W->A conversions are ugly but it was either that
or introducing a strrchrW somewhere (copied from unicode.h most likely).

Changelog
   Add a few tests

-- 
Cheers,

Paul.

-------------- next part --------------
A non-text attachment was scrubbed...
Name: 0004-Add-a-few-tests.patch
Type: text/x-diff
Size: 5843 bytes
Desc: not available
Url : http://www.winehq.org/pipermail/wine-patches/attachments/20090107/4d1bbd89/attachment-0001.patch 


More information about the wine-patches mailing list