[PATCH v3 1/5] mmdevapi/tests/render: Check that incompatible formats are rejected by IsFormatSupported().

[Giovanni Mascellani]

From: Giovanni Mascellani

---
 dlls/mmdevapi/tests/render.c | 13 ++++++++-----
 1 file changed, 8 insertions(+), 5 deletions(-)

diff --git a/dlls/mmdevapi/tests/render.c b/dlls/mmdevapi/tests/render.c
index 236a4ff71c8..08579741676 100644
--- a/dlls/mmdevapi/tests/render.c
+++ b/dlls/mmdevapi/tests/render.c
@@ -553,11 +553,14 @@ static void test_formats(AUDCLNT_SHAREMODE mode)
                   mode == AUDCLNT_SHAREMODE_SHARED ? "shared " : "exclus.",
                   fmt.nSamplesPerSec, fmt.wBitsPerSample, fmt.nChannels);

-        /* Change GetMixFormat wBitsPerSample only => S_OK */
-        if (mode == AUDCLNT_SHAREMODE_SHARED
-            && fmt.nSamplesPerSec == pwfx->nSamplesPerSec
-            && fmt.nChannels == pwfx->nChannels)
-            ok(hr == S_OK, "Varying BitsPerSample %u\n", fmt.wBitsPerSample);
+        /* In shared mode you can only change bit width, not sampling rate or channel count. */
+        if (mode == AUDCLNT_SHAREMODE_SHARED)
+        {
+            BOOL compatible = fmt.nSamplesPerSec == pwfx->nSamplesPerSec && fmt.nChannels == pwfx->nChannels;
+            HRESULT expected = compatible ? S_OK : S_FALSE;
+            todo_wine_if(expected == S_FALSE)
+            ok(hr == expected, "Got %lx expected %lx\n", hr, expected);
+        }

ok((hr == S_FALSE)^(pwfx2 == NULL), "hr %lx<->suggest %p\n", hr, pwfx2);
         if (pwfx2 == (WAVEFORMATEX*)0xDEADF00D)