Only perform the screen resolution changing tests in interactive mode.

Francois Gouget fgouget at free.fr
Fri Sep 29 06:03:40 CDT 2006


---
This test is the reason why I have not been running the Wine conformance 
tests for the past year:
 * In my almost stock xorg.conf file, I have 6 different resolutions, 6 
different color depths, and with all that it causes this test to spend 
more than 30 minutes changing screen resolutions. This means my 21 inch 
CRT monitors clicks and resyncs for half an hour and that's just no 
good.
 * IMHO, conformance tests should not be too disruptive. That was mostly 
true for a while, though now we have quite a few tests that pop up 
focused windows :-/
 * Finally the test shows some test screen in each resolution, but if no 
one's watching there's not much point.

So I think it is more reasonable to only run this test in interactive 
mode when someone can actively monitor it, and actually check that each 
test screen is displayed properly.


 dlls/ddraw/tests/ddrawmodes.c |   10 +++++++++-
 1 files changed, 9 insertions(+), 1 deletions(-)

diff --git a/dlls/ddraw/tests/ddrawmodes.c b/dlls/ddraw/tests/ddrawmodes.c
index fd00bb6..5020bec 100644
--- a/dlls/ddraw/tests/ddrawmodes.c
+++ b/dlls/ddraw/tests/ddrawmodes.c
@@ -1,6 +1,13 @@
 /*
  * Unit tests for ddraw functions
  *
+ *
+ * Part of this test involves changing the screen resolution. But this is
+ * really disrupting if the user is doing something else and is not very nice
+ * to CRT screens. Plus, ideally it needs someone watching it to check that
+ * each mode displays correctly.
+ * So this is only done if the test is being run in interactive mode.
+ *
  * Copyright (C) 2003 Sami Aario
  *
  * This library is free software; you can redistribute it and/or
@@ -342,7 +349,8 @@ START_TEST(ddrawmodes)
     if (!createdirectdraw())
         return;
     enumdisplaymodes();
-    testdisplaymodes();
+    if (winetest_interactive)
+        testdisplaymodes();
     flushdisplaymodes();
     releasedirectdraw();
 
-- 
1.4.1.1




More information about the wine-patches mailing list