[PATCH 5/5] dnsapi/tests: Add a query test.

[Hans Leidekker]

Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
---
 dlls/dnsapi/tests/query.c | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/dlls/dnsapi/tests/query.c b/dlls/dnsapi/tests/query.c
index 237dfd6ae7c..74669005a33 100644
--- a/dlls/dnsapi/tests/query.c
+++ b/dlls/dnsapi/tests/query.c
@@ -38,6 +38,11 @@ static void test_DnsQuery(void)
     DNS_RECORDW *rec;
     DNS_STATUS status;
 
+    rec = NULL;
+    status = DnsQuery_W(L"winehq.org", DNS_TYPE_A, DNS_QUERY_STANDARD, NULL, &rec, NULL);
+    ok(status == ERROR_SUCCESS, "DnsQuery_W(%s) => %d\n", wine_dbgstr_w(name), status);
+    DnsRecordListFree(rec, DnsFreeRecordList);
+
     status = DnsQuery_W(L"", DNS_TYPE_SRV, DNS_QUERY_STANDARD, NULL, &rec, NULL);
     ok(status == DNS_ERROR_RCODE_NAME_ERROR || status == DNS_INFO_NO_RECORDS || status == ERROR_INVALID_NAME /* XP */, "got %u\n", status);
 
-- 
2.30.2

[Marvin]

Hi,

While running your changed tests, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?

Full results can be found at:
https://testbot.winehq.org/JobDetails.pl?Key=87912

Your paranoid android.


=== w8adm (32 bit report) ===

dnsapi:
query.c:43: Test failed: DnsQuery_W(L"\ffff\f658#\d5eb\74d7\00cb") => 1460
query.c:51: Test failed: got 1460

[Dmitry Timoshkov]

Hans Leidekker <hans(a)codeweavers.com> wrote:
> +    rec = NULL;
> +    status = DnsQuery_W(L"winehq.org", DNS_TYPE_A, DNS_QUERY_STANDARD, NULL, &rec, NULL);
> +    ok(status == ERROR_SUCCESS, "DnsQuery_W(%s) => %d\n", wine_dbgstr_w(name), status);
> +    DnsRecordListFree(rec, DnsFreeRecordList);

You probably already noticed (from the testbot) that error message prints
a not initialized buffer 'name'.

-- 
Dmitry.