André Hentschel : gdi32/tests: Simplify testcase.

[Alexandre Julliard]

Module: wine
Branch: master
Commit: 762280eab0eb5426040bf27932648ec055e1949c
URL:    http://source.winehq.org/git/wine.git/?a=commit;h=762280eab0eb5426040bf27932...

Author: André Hentschel 
Date:   Tue Nov 17 17:43:54 2009 +0100

gdi32/tests: Simplify testcase.

---

dlls/gdi32/tests/font.c |   61 +++++++++++++---------------------------------
 1 files changed, 17 insertions(+), 44 deletions(-)

diff --git a/dlls/gdi32/tests/font.c b/dlls/gdi32/tests/font.c
index 4ad418f..08ceb85 100644
--- a/dlls/gdi32/tests/font.c
+++ b/dlls/gdi32/tests/font.c
@@ -2980,61 +2980,34 @@ static void test_GetTextMetrics2(const char *fontname, int font_height)
 void test_CreateFontIndirect(void)
 {
     LOGFONTA lf, getobj_lf;
-    int ret;
+    int ret, i;
     HFONT hfont;
+    char TestName[][16] = {"Arial", "Arial Bold", "Arial Italic"};

memset(&lf, 0, sizeof(lf));
-
     lf.lfCharSet = ANSI_CHARSET;
     lf.lfClipPrecision = CLIP_DEFAULT_PRECIS;
     lf.lfHeight = 16;
     lf.lfWidth = 16;
     lf.lfQuality = DEFAULT_QUALITY;
-
     lf.lfItalic = FALSE;
     lf.lfWeight = FW_DONTCARE;
-    lstrcpyA(lf.lfFaceName, "Arial");
-    hfont = CreateFontIndirectA(&lf);
-    ok(hfont != 0, "CreateFontIndirectA failed\n");
-    ret = GetObject(hfont, sizeof(getobj_lf), &getobj_lf);
-    ok(lf.lfWeight == getobj_lf.lfWeight ||
-       broken((SHORT)lf.lfWeight == getobj_lf.lfWeight), /* win9x */
-       "lfWeight: expect %08x got %08x\n", lf.lfWeight, getobj_lf.lfWeight);
-    ok(lf.lfItalic == getobj_lf.lfItalic, "lfItalic: expect %02x got %02x\n", lf.lfItalic, getobj_lf.lfItalic);
-    ok(!lstrcmpA(lf.lfFaceName, getobj_lf.lfFaceName) ||
-       broken(!memcmp(lf.lfFaceName, getobj_lf.lfFaceName, LF_FACESIZE-1)), /* win9x doesn't ensure '\0' termination */
-       "font names don't match: %s != %s\n", lf.lfFaceName, getobj_lf.lfFaceName);
-    DeleteObject(hfont);

-    lf.lfItalic = FALSE;
-    lf.lfWeight = FW_DONTCARE;
-    lstrcpyA(lf.lfFaceName, "Arial Bold");
-    hfont = CreateFontIndirectA(&lf);
-    ok(hfont != 0, "CreateFontIndirectA failed\n");
-    ret = GetObject(hfont, sizeof(getobj_lf), &getobj_lf);
-    ok(lf.lfItalic == getobj_lf.lfItalic, "lfItalic: expect %02x got %02x\n", lf.lfItalic, getobj_lf.lfItalic);
-    ok(lf.lfWeight == getobj_lf.lfWeight ||
-       broken((SHORT)lf.lfWeight == getobj_lf.lfWeight), /* win9x */
-       "lfWeight: expect %08x got %08x\n", lf.lfWeight, getobj_lf.lfWeight);
-    ok(!lstrcmpA(lf.lfFaceName, getobj_lf.lfFaceName) ||
-       broken(!memcmp(lf.lfFaceName, getobj_lf.lfFaceName, LF_FACESIZE-1)), /* win9x doesn't ensure '\0' termination */
-       "font names don't match: %s != %s\n", lf.lfFaceName, getobj_lf.lfFaceName);
-    DeleteObject(hfont);
-
-    lf.lfItalic = FALSE;
-    lf.lfWeight = FW_DONTCARE;
-    lstrcpyA(lf.lfFaceName, "Arial Italic");
-    hfont = CreateFontIndirectA(&lf);
-    ok(hfont != 0, "CreateFontIndirectA failed\n");
-    ret = GetObject(hfont, sizeof(getobj_lf), &getobj_lf);
-    ok(lf.lfWeight == getobj_lf.lfWeight ||
-       broken((SHORT)lf.lfWeight == getobj_lf.lfWeight), /* win9x */
-       "lfWeight: expect %08x got %08x\n", lf.lfWeight, getobj_lf.lfWeight);
-    ok(lf.lfItalic == getobj_lf.lfItalic, "lfItalic: expect %02x got %02x\n", lf.lfItalic, getobj_lf.lfItalic);
-    ok(!lstrcmpA(lf.lfFaceName, getobj_lf.lfFaceName) ||
-       broken(!memcmp(lf.lfFaceName, getobj_lf.lfFaceName, LF_FACESIZE-1)), /* win9x doesn't ensure '\0' termination */
-       "font names don't match: %s != %s\n", lf.lfFaceName, getobj_lf.lfFaceName);
-    DeleteObject(hfont);
+    for (i = 0; i < sizeof(TestName)/sizeof(TestName[0]); i++)
+    {
+        lstrcpyA(lf.lfFaceName, TestName[i]);
+        hfont = CreateFontIndirectA(&lf);
+        ok(hfont != 0, "CreateFontIndirectA failed\n");
+        ret = GetObject(hfont, sizeof(getobj_lf), &getobj_lf);
+        ok(lf.lfItalic == getobj_lf.lfItalic, "lfItalic: expect %02x got %02x\n", lf.lfItalic, getobj_lf.lfItalic);
+        ok(lf.lfWeight == getobj_lf.lfWeight ||
+           broken((SHORT)lf.lfWeight == getobj_lf.lfWeight), /* win9x */
+           "lfWeight: expect %08x got %08x\n", lf.lfWeight, getobj_lf.lfWeight);
+        ok(!lstrcmpA(lf.lfFaceName, getobj_lf.lfFaceName) ||
+           broken(!memcmp(lf.lfFaceName, getobj_lf.lfFaceName, LF_FACESIZE-1)), /* win9x doesn't ensure '\0' termination */
+           "font names don't match: %s != %s\n", lf.lfFaceName, getobj_lf.lfFaceName);
+        DeleteObject(hfont);
+    }
 }

START_TEST(font)