Module: wine Branch: master Commit: 2c9c761b5607b19544a22caa1e5449ba0382a48d URL: http://source.winehq.org/git/wine.git/?a=commit;h=2c9c761b5607b19544a22caa1e...
Author: Misha Koshelev mk144210@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(); }