http://bugs.winehq.org/show_bug.cgi?id=19529
--- Comment #5 from Juan Lang juan_lang@yahoo.com 2009-07-31 11:38:08 --- (In reply to comment #4)
In reference to this external server, I was patterning myself on the tests in wine/dlls/wldap32/tests/parse.c. I don't know any other way to test this functionality than using existing LDAP server.
Since the existing tests use the same server, I'm sure this is fine. You'd want to send the test in git format to wine-patches@winehq.org.
Also, the tests: ok( !ret, "ldap_search_extW failed 0x%08x\n", ret ); assume your patch is already in the tree, yes?
A better way might be to add the test first, with todo_wine before the tests that fail with unpatched wine. Something like: todo_wine ok( !ret, "ldap_search_extW failed 0x%08x\n", ret );
A subsequent patch that fixes the search function can remove the todo_wine.