Rémi Bernon : dmime/tests: Check reference time and allow 0.5 tick difference.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 6447b11b17d3481945d40e079a345cbe18fd1053
URL:    https://gitlab.winehq.org/wine/wine/-/commit/6447b11b17d3481945d40e079a345cb...

Author: Rémi Bernon <rbernon(a)codeweavers.com>
Date:   Fri Dec 15 12:10:10 2023 +0100

dmime/tests: Check reference time and allow 0.5 tick difference.

Renaming check_music_time to check_reference_time, as it's truly
reference time values that are passed.

Wine-Bug: https://bugs.winehq.org/show_bug.cgi?id=55637

---

 dlls/dmime/tests/dmime.c | 25 +++++++++++++------------
 1 file changed, 13 insertions(+), 12 deletions(-)

diff --git a/dlls/dmime/tests/dmime.c b/dlls/dmime/tests/dmime.c
index b4b6d848fdc..5fff5ea02de 100644
--- a/dlls/dmime/tests/dmime.c
+++ b/dlls/dmime/tests/dmime.c
@@ -62,7 +62,7 @@ static void check_interface_(unsigned int line, void *iface_ptr, REFIID iid, BOO
 
 static double scale_music_time(MUSIC_TIME time, double tempo)
 {
-    return (600000000.0 * time) / (tempo * 768.0);
+    return (600000000.0 * time) / (tempo * DMUS_PPQ);
 }
 
 #define check_dmus_note_pmsg(a, b, c, d, e, f, g) check_dmus_note_pmsg_(__LINE__, a, b, c, d, e, f, g)
@@ -2812,10 +2812,11 @@ static void test_performance_graph(void)
     IDirectMusicTool_Release(tool);
 }
 
-#define check_music_time(a, b) check_music_time_(__LINE__, a, b)
-static void check_music_time_(int line, MUSIC_TIME time, MUSIC_TIME expect)
+#define check_reference_time(a, b) check_reference_time_(__LINE__, a, b)
+static void check_reference_time_(int line, REFERENCE_TIME time, double expect)
 {
-    ok_(__FILE__, line)(abs(time - expect) <= 1, "got %ld, expected %ld\n", time, expect);
+    ok_(__FILE__, line)(llabs(time - (REFERENCE_TIME)expect) <= scale_music_time(1, 120) / 2.0,
+                        "got %I64u, expected %f\n", time, expect);
 }
 
 static void test_performance_time(void)
@@ -2867,15 +2868,15 @@ static void test_performance_time(void)
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 1, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(1, 120));
+    check_reference_time(time - init_time, scale_music_time(1, 120));
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 1000, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(1000, 120));
+    check_reference_time(time - init_time, scale_music_time(1000, 120));
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 2000, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(2000, 120));
+    check_reference_time(time - init_time, scale_music_time(2000, 120));
 
     music_time = 0xdeadbeef;
     hr = IDirectMusicPerformance_ReferenceToMusicTime(performance, init_time, &music_time);
@@ -4250,23 +4251,23 @@ static void test_tempo_track_play(void)
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 1, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(1, 120));
+    check_reference_time(time - init_time, scale_music_time(1, 120));
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 100, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(100, 120));
+    check_reference_time(time - init_time, scale_music_time(100, 120));
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 150, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(100, 120) + scale_music_time(50, 80));
+    check_reference_time(time - init_time, scale_music_time(100, 120) + scale_music_time(50, 80));
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 200, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(100, 120) + scale_music_time(100, 80));
+    check_reference_time(time - init_time, scale_music_time(100, 120) + scale_music_time(100, 80));
     time = 0xdeadbeef;
     hr = IDirectMusicPerformance_MusicToReferenceTime(performance, 400, &time);
     ok(hr == S_OK, "got %#lx\n", hr);
-    check_music_time(time - init_time, scale_music_time(100, 120) + scale_music_time(200, 80) + scale_music_time(100, 20));
+    check_reference_time(time - init_time, scale_music_time(100, 120) + scale_music_time(200, 80) + scale_music_time(100, 20));
 
     music_time = 0xdeadbeef;
     hr = IDirectMusicPerformance_ReferenceToMusicTime(performance, init_time, &music_time);