Signed-off-by: Francois Gouget fgouget@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);
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"
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