Akihiro Sagawa : winmm/tests: Add more MIDI stream position tests.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 2ae051720904a10ab3d14655ecde87012ead9aa6
URL:    https://source.winehq.org/git/wine.git/?a=commit;h=2ae051720904a10ab3d14655ecde87012ead9aa6

Author: Akihiro Sagawa <sagawa.aki at gmail.com>
Date:   Wed Jul 11 22:33:21 2018 +0900

winmm/tests: Add more MIDI stream position tests.

Signed-off-by: Akihiro Sagawa <sagawa.aki at gmail.com>
Signed-off-by: Andrew Eikum <aeikum at codeweavers.com>
Signed-off-by: Alexandre Julliard <julliard at winehq.org>

---

 dlls/winmm/tests/midi.c | 33 +++++++++++++++++++++++++++++++++
 1 file changed, 33 insertions(+)

diff --git a/dlls/winmm/tests/midi.c b/dlls/winmm/tests/midi.c
index c46286b..c550dfa 100644
--- a/dlls/winmm/tests/midi.c
+++ b/dlls/winmm/tests/midi.c
@@ -509,6 +509,11 @@ static void test_position(HMIDISTRM hm, UINT typein, UINT typeout)
     case TIME_MIDI:
         trace("Stream position song pointer %u\n", mmtime.u.midi.songptrpos);
         break;
+    case TIME_SMPTE:
+        trace("Stream position %02u:%02u:%02u.%02u/%02u\n",
+              mmtime.u.smpte.hour, mmtime.u.smpte.min, mmtime.u.smpte.sec,
+              mmtime.u.smpte.frame, mmtime.u.smpte.fps);
+        break;
     }
 }
 
@@ -573,6 +578,8 @@ static DWORD get_position(HMIDISTRM hm, UINT type)
         return mmtime.u.ms;
     case TIME_TICKS:
         return mmtime.u.ticks;
+    case TIME_MIDI:
+        return mmtime.u.midi.songptrpos;
     default:
         return MAXDWORD;
     }
@@ -885,6 +892,10 @@ static void test_midiStream(UINT udev, HWND hwnd)
     ok(ret > expected && ret < expected + MARGIN, "expected greater than %ums, got %ums\n", expected, ret);
     expected = ret;
 
+    ret = get_position(hm, TIME_TICKS);
+    todo_wine ok(ret > strmNopsWithDelta[0].dwDeltaTime && ret < strmNopsWithDelta[1].dwDeltaTime,
+       "TIME_TICKS position is continuous, got %u\n", ret);
+
     /* shouldn't set time division property while playing */
     midiprop.tdiv.cbStruct  = sizeof(midiprop.tdiv);
     midiprop.tdiv.dwTimeDiv = 24;
@@ -914,6 +925,16 @@ static void test_midiStream(UINT udev, HWND hwnd)
     ok(ret >= strmNopsWithDelta[1].dwDeltaTime && ret < strmNopsWithDelta[1].dwDeltaTime + 3,
        "expected %u ticks, got %u\n", strmNopsWithDelta[1].dwDeltaTime, ret);
 
+    midiprop.tdiv.cbStruct = sizeof(midiprop.tdiv);
+    rc = midiStreamProperty(hm, (void*)&midiprop, MIDIPROP_GET | MIDIPROP_TIMEDIV);
+    ok(!rc, "midiStreamProperty(GET|TIMEDIV, dev=%d) rc=%s\n", udev, mmsys_error(rc));
+    ok(midiprop.tdiv.dwTimeDiv == 24, "expected 24, got %u\n", midiprop.tdiv.dwTimeDiv);
+
+    /* TIME_MIDI value is a quarter of TIME_TICKS, rounded */
+    expected = (ret + midiprop.tdiv.dwTimeDiv/8) / (midiprop.tdiv.dwTimeDiv/4);
+    ret = get_position(hm, TIME_MIDI);
+    todo_wine ok(ret == expected, "expected song pointer %u, got %u\n", expected, ret);
+
     ok(records.count == 2, "expected 2 MM_MOM_DONE messages, got %d\n", records.count);
 
     /* Time between midiStreamPause and midiStreamRestart isn't counted.
@@ -932,6 +953,18 @@ static void test_midiStream(UINT udev, HWND hwnd)
     ret = get_position(hm, TIME_MS);
     ok(ret == 0, "expected 0ms, got %ums\n", ret);
 
+    midiprop.tdiv.cbStruct  = sizeof(midiprop.tdiv);
+    midiprop.tdiv.dwTimeDiv = 0xe204; /* 30 fps, 4 ticks/frame */
+    rc = midiStreamProperty(hm, (void*)&midiprop, MIDIPROP_SET | MIDIPROP_TIMEDIV);
+    ok(!rc, "midiStreamProperty(SET|TIMEDIV, dev=%d) rc=%s\n", udev, mmsys_error(rc));
+
+    test_position(hm, TIME_MS,      TIME_MS);
+    test_position(hm, TIME_TICKS,   TIME_TICKS);
+    test_position(hm, TIME_MIDI,    TIME_MS);
+    todo_wine test_position(hm, TIME_SMPTE,   TIME_SMPTE);
+    test_position(hm, TIME_SAMPLES, TIME_MS);
+    test_position(hm, TIME_BYTES,   TIME_MS);
+
     rc = midiStreamClose(hm);
     ok(!rc, "midiStreamClose(dev=%d) rc=%s\n", udev, mmsys_error(rc));
     CloseHandle(records.done);