Re: [PATCH 2/2] propsys/test: Add VariantToStringWithDefault tests.

[Nikolay Sivov]

On 3/25/22 04:44, Mohamad Al-Jaf wrote:
> - /* VT_BSTR */
> -
> - V_VT(&var) = VT_BSTR;
> - V_BSTR(&var) = NULL;
> - result = VariantToStringWithDefault(&var, default_value);
> - ok(!lstrcmpW(result, wstr_empty), "Unexpected value %s\n", wine_dbgstr_w(result));
> -
> - V_VT(&var) = VT_BSTR;
> - V_BSTR(&var) = SysAllocString(wstr_empty);
> - result = VariantToStringWithDefault(&var, default_value);
> - ok(!lstrcmpW(result, wstr_empty), "Unexpected value %s\n", wine_dbgstr_w(result));
> - SysFreeString(V_BSTR(&var));
> -
> - V_VT(&var) = VT_BSTR;
> - V_BSTR(&var) = SysAllocString(wstr_space);
> - result = VariantToStringWithDefault(&var, default_value);
> - ok(!lstrcmpW(result, wstr_space), "Unexpected value %s\n", wine_dbgstr_w(result));
> - SysFreeString(V_BSTR(&var));
> -
> - V_VT(&var) = VT_BSTR;
> - V_BSTR(&var) = SysAllocString(wstr_test2);
> - result = VariantToStringWithDefault(&var, default_value);
> - ok(!lstrcmpW(result, wstr_test2), "Unexpected value %s\n", wine_dbgstr_w(result));
> - SysFreeString(V_BSTR(&var));

What you need to test is returned pointer value.