[PATCH] oleaut32/tests: Show the input string when VarFormatCurrency() fails.

[Francois Gouget]

Signed-off-by: Francois Gouget <fgouget(a)codeweavers.com>
---
 dlls/oleaut32/tests/varformat.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/dlls/oleaut32/tests/varformat.c b/dlls/oleaut32/tests/varformat.c
index d1951b85a61..55190574ca9 100644
--- a/dlls/oleaut32/tests/varformat.c
+++ b/dlls/oleaut32/tests/varformat.c
@@ -639,14 +639,14 @@ static void test_VarFormatCurrency(void)
     V_VT(&in) = VT_BSTR;
     V_BSTR(&in) = str;
     hr = VarFormatCurrency(&in, 1, -2, -2, -2, 0, &str2);
-    ok(hr == S_OK, "Unexpected hr %#x.\n", hr);
+    ok(hr == S_OK, "Unexpected hr %#x for %s\n", hr, wine_dbgstr_w(str));
     ok(lstrcmpW(str, str2), "Expected different string.\n");
     SysFreeString(str2);
 
     V_VT(&in) = VT_BSTR | VT_BYREF;
     V_BSTRREF(&in) = &str;
     hr = VarFormatCurrency(&in, 1, -2, -2, -2, 0, &str2);
-    ok(hr == S_OK, "Unexpected hr %#x.\n", hr);
+    ok(hr == S_OK, "Unexpected hr %#x for %s\n", hr, wine_dbgstr_w(str));
     ok(lstrcmpW(str, str2), "Expected different string.\n");
 
     SysFreeString(str);
-- 
2.20.1

[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=94457

Your paranoid android.


=== debiant2 (32 bit Arabic:Morocco report) ===

oleaut32:
varformat.c:642: Test failed: Unexpected hr 0x80020005 for L"\062f.\0645.\200f 0.000"
varformat.c:649: Test failed: Unexpected hr 0x80020005 for L"\062f.\0645.\200f 0.000"

[Francois Gouget]

On Fri, 23 Jul 2021, Marvin wrote:
[...]
> === debiant2 (32 bit Arabic:Morocco report) ===
>
> oleaut32:
> varformat.c:642: Test failed: Unexpected hr 0x80020005 for L"\062f.\0645.\200f 0.000"
> varformat.c:649: Test failed: Unexpected hr 0x80020005 for L"\062f.\0645.\200f 0.000"

Not a new failure. The issue is that Wine only supports currency symbols 
that have at most two characters. I'm progressively working towards a 
fix.

https://test.winehq.org/data/patterns.html#oleaut32:varformat

-- 
Francois Gouget <fgouget(a)codeweavers.com>