Misha Koshelev : gdi32/tests:
Add framework for tests of drawing functions in paths,
add test for ArcTo in paths.
Alexandre Julliard
julliard at wine.codeweavers.com
Tue Jun 19 06:42:35 CDT 2007
Module: wine
Branch: master
Commit: 2c9c761b5607b19544a22caa1e5449ba0382a48d
URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c9c761b5607b19544a22caa1e5449ba0382a48d
Author: Misha Koshelev <mk144210 at bcm.edu>
Date: Tue Jun 19 01:44:22 2007 -0500
gdi32/tests: Add framework for tests of drawing functions in paths, add test for ArcTo in paths.
---
dlls/gdi32/tests/path.c | 145 +++++++++++++++++++++++++++++++++++++++++++++++
1 files changed, 145 insertions(+), 0 deletions(-)
diff --git a/dlls/gdi32/tests/path.c b/dlls/gdi32/tests/path.c
index e9a6d39..a9fab4b 100644
--- a/dlls/gdi32/tests/path.c
+++ b/dlls/gdi32/tests/path.c
@@ -20,6 +20,7 @@
#include <stdarg.h>
#include <stdio.h>
+#include <assert.h>
#include "windef.h"
#include "winbase.h"
#include "wingdi.h"
@@ -93,7 +94,151 @@ static void test_widenpath(void)
return;
}
+/*
+ * Tests for GDI drawing functions in paths
+ */
+
+typedef struct
+{
+ int x, y;
+ BYTE type;
+
+ /* How many extra entries before this one only on wine
+ * but not on native? */
+ int wine_only_entries_preceding;
+
+ /* Does this entry itself not match on wine? */
+ int todo;
+} path_test_t;
+
+/* Helper function to verify that the current path in the given DC matches the expected path.
+ *
+ * We use a "smart" matching algorithm that allows us to detect partial improvements
+ * in conformance. Specifically, two running indices are kept, one through the actual
+ * path and one through the expected path. The actual path index always increases,
+ * whereas, if the wine_entries_preceding field of the appropriate path_test_t element is
+ * non-zero, the expected path index does not increase for that many elements as long as
+ * there is no match. This allows us to todo_wine extra path elements that are present only
+ * in wine but not on native.
+ *
+ * Note that if expected_size is zero and the WINETEST_DEBUG environment variable is
+ * greater than 2, the trace() output is a C path_test_t array structure, useful for making
+ * new tests that use this function.
+ */
+static void ok_path(HDC hdc, const path_test_t *expected, int expected_size, BOOL todo_size)
+{
+ static const char *type_string[8] = { "Unknown (0)", "PT_CLOSEFIGURE", "PT_LINETO",
+ "PT_LINETO | PT_CLOSEFIGURE", "PT_BEZIERTO",
+ "PT_BEZIERTO | PT_CLOSEFIGURE", "PT_MOVETO", "PT_MOVETO | PT_CLOSEFIGURE"};
+ POINT *pnt = NULL;
+ BYTE *types = NULL;
+ int size, numskip,
+ idx, eidx = 0;
+
+ /* Get the path */
+ assert(hdc != 0);
+ size = GetPath(hdc, NULL, NULL, 0);
+ ok(size > 0, "GetPath returned size %d, last error %d\n", size, GetLastError());
+ if (size <= 0)
+ {
+ skip("Cannot perform path comparisons due to failure to retrieve path.\n");
+ return;
+ }
+ pnt = HeapAlloc(GetProcessHeap(), 0, size*sizeof(POINT));
+ assert(pnt != 0);
+ types = HeapAlloc(GetProcessHeap(), 0, size*sizeof(BYTE));
+ assert(types != 0);
+ size = GetPath(hdc, pnt, types, size);
+ assert(size > 0);
+
+ if (todo_size) todo_wine
+ ok(size == expected_size, "Path size %d does not match expected size %d\n", size, expected_size);
+ else
+ ok(size == expected_size, "Path size %d does not match expected size %d\n", size, expected_size);
+
+ if (expected_size) numskip = expected[eidx].wine_only_entries_preceding;
+ for (idx = 0; idx < size && eidx < expected_size; idx++)
+ {
+ /* We allow a few pixels fudge in matching X and Y coordinates to account for imprecision in
+ * floating point to integer conversion */
+ BOOL match = (types[idx] == expected[eidx].type) &&
+ (pnt[idx].x >= expected[eidx].x-1 && pnt[idx].x <= expected[eidx].x+1) &&
+ (pnt[idx].y >= expected[eidx].y-1 && pnt[idx].y <= expected[eidx].y+1);
+
+ if (winetest_debug > 2)
+ trace("{%d, %d, %s, 0, 0}, /* %d */\n", pnt[idx].x, pnt[idx].y, type_string[types[idx]], idx);
+
+ if (expected[eidx].todo || numskip) todo_wine
+ ok(match, "Expected #%d: %s (%d,%d) but got %s (%d,%d)\n", eidx,
+ type_string[expected[eidx].type], expected[eidx].x, expected[eidx].y,
+ type_string[types[idx]], pnt[idx].x, pnt[idx].y);
+ else
+ ok(match, "Expected #%d: %s (%d,%d) but got %s (%d,%d)\n", eidx,
+ type_string[expected[eidx].type], expected[eidx].x, expected[eidx].y,
+ type_string[types[idx]], pnt[idx].x, pnt[idx].y);
+
+ if (match || !numskip--)
+ numskip = expected[++eidx].wine_only_entries_preceding;
+ }
+
+ /* If we are debugging and the actual path is longer than the expected path, make
+ * sure to display the entire path */
+ if (winetest_debug > 2 && idx < size)
+ for (; idx < size; idx++)
+ trace("{%d, %d, %s, 0, 0}, /* %d */\n", pnt[idx].x, pnt[idx].y, type_string[types[idx]], idx);
+
+ HeapFree(GetProcessHeap(), 0, types);
+ HeapFree(GetProcessHeap(), 0, pnt);
+}
+
+static const path_test_t arcto_path[] = {
+ {0, 0, PT_MOVETO, 0, 0}, /* 0 */
+ {229, 215, PT_LINETO, 0, 1}, /* 1 */
+ {248, 205, PT_BEZIERTO, 1, 0}, /* 2 */
+ {273, 200, PT_BEZIERTO, 0, 0}, /* 3 */
+ {300, 200, PT_BEZIERTO, 0, 0}, /* 4 */
+ {355, 200, PT_BEZIERTO, 0, 0}, /* 5 */
+ {399, 222, PT_BEZIERTO, 0, 0}, /* 6 */
+ {399, 250, PT_BEZIERTO, 0, 0}, /* 7 */
+ {399, 263, PT_BEZIERTO, 0, 0}, /* 8 */
+ {389, 275, PT_BEZIERTO, 0, 0}, /* 9 */
+ {370, 285, PT_BEZIERTO, 0, 0}, /* 10 */
+ {363, 277, PT_LINETO, 1, 1}, /* 11 */
+ {380, 270, PT_BEZIERTO, 1, 0}, /* 12 */
+ {389, 260, PT_BEZIERTO, 0, 0}, /* 13 */
+ {389, 250, PT_BEZIERTO, 0, 0}, /* 14 */
+ {389, 228, PT_BEZIERTO, 0, 0}, /* 15 */
+ {349, 210, PT_BEZIERTO, 0, 0}, /* 16 */
+ {300, 210, PT_BEZIERTO, 0, 0}, /* 17 */
+ {276, 210, PT_BEZIERTO, 0, 0}, /* 18 */
+ {253, 214, PT_BEZIERTO, 0, 0}, /* 19 */
+ {236, 222, PT_BEZIERTO | PT_CLOSEFIGURE, 0, 0}}; /* 20 */
+
+static void test_arcto(void)
+{
+ HDC hdc = GetDC(0);
+
+ BeginPath(hdc);
+ SetArcDirection(hdc, AD_CLOCKWISE);
+ if (!ArcTo(hdc, 200, 200, 400, 300, 200, 200, 400, 300) &&
+ GetLastError() == ERROR_CALL_NOT_IMPLEMENTED)
+ {
+ /* ArcTo is only available on Win2k and later */
+ skip("ArcTo is not available\n");
+ goto done;
+ }
+ SetArcDirection(hdc, AD_COUNTERCLOCKWISE);
+ ArcTo(hdc, 210, 210, 390, 290, 390, 290, 210, 210);
+ CloseFigure(hdc);
+ EndPath(hdc);
+
+ ok_path(hdc, arcto_path, sizeof(arcto_path)/sizeof(path_test_t), 1);
+done:
+ ReleaseDC(0, hdc);
+}
+
START_TEST(path)
{
test_widenpath();
+ test_arcto();
}
More information about the wine-cvs
mailing list