[3/3] dwrite: Added basic test for IDWriteFont created from LOGFONTW

Nikolay Sivov nsivov at codeweavers.com
Mon Jul 30 23:22:34 CDT 2012


Added basic test for IDWriteFont created from LOGFONTW
-------------- next part --------------
>From 2647ce5c773abe44e874f2253a69e67d4883ef3e Mon Sep 17 00:00:00 2001
From: Nikolay Sivov <nsivov at codeweavers.com>
Date: Tue, 31 Jul 2012 08:18:02 +0400
Subject: [PATCH 3/3] Added basic test for IDWriteFont created from LOGFONTW

---
 configure.ac                  |    1 +
 dlls/dwrite/Makefile.in       |    5 +-
 dlls/dwrite/gdiinterop.c      |    3 +
 dlls/dwrite/tests/Makefile.in |    7 +++
 dlls/dwrite/tests/font.c      |  101 +++++++++++++++++++++++++++++++++++++++++
 include/dwrite.idl            |    2 +
 6 files changed, 117 insertions(+), 2 deletions(-)
 create mode 100644 dlls/dwrite/tests/Makefile.in
 create mode 100644 dlls/dwrite/tests/font.c

diff --git a/configure.ac b/configure.ac
index 35c0e6c..49c97b8 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2625,6 +2625,7 @@ WINE_CONFIG_TEST(dlls/dssenh/tests)
 WINE_CONFIG_DLL(dswave)
 WINE_CONFIG_DLL(dwmapi,,[implib])
 WINE_CONFIG_DLL(dwrite)
+WINE_CONFIG_TEST(dlls/dwrite/tests)
 WINE_CONFIG_DLL(dxdiagn,,[po])
 WINE_CONFIG_TEST(dlls/dxdiagn/tests)
 WINE_CONFIG_LIB(dxerr8)
diff --git a/dlls/dwrite/Makefile.in b/dlls/dwrite/Makefile.in
index a4510ea..db99fbe 100644
--- a/dlls/dwrite/Makefile.in
+++ b/dlls/dwrite/Makefile.in
@@ -1,5 +1,6 @@
-MODULE  = dwrite.dll
-IMPORTS = uuid
+MODULE    = dwrite.dll
+IMPORTLIB = dwrite
+IMPORTS   = uuid
 
 C_SRCS = \
 	gdiinterop.c \
diff --git a/dlls/dwrite/gdiinterop.c b/dlls/dwrite/gdiinterop.c
index 7b3c589..8215185 100644
--- a/dlls/dwrite/gdiinterop.c
+++ b/dlls/dwrite/gdiinterop.c
@@ -57,6 +57,9 @@ static HRESULT WINAPI gdiinterop_CreateFontFromLOGFONT(IDWriteGdiInterop *iface,
     LOGFONTW const *logfont, IDWriteFont **font)
 {
     FIXME("(%p %p): stub\n", logfont, font);
+
+    if (!logfont) return E_INVALIDARG;
+
     return E_NOTIMPL;
 }
 
diff --git a/dlls/dwrite/tests/Makefile.in b/dlls/dwrite/tests/Makefile.in
new file mode 100644
index 0000000..f00d0e7
--- /dev/null
+++ b/dlls/dwrite/tests/Makefile.in
@@ -0,0 +1,7 @@
+TESTDLL = dwrite.dll
+IMPORTS = dwrite
+
+C_SRCS = \
+	font.c
+
+ at MAKE_TEST_RULES@
diff --git a/dlls/dwrite/tests/font.c b/dlls/dwrite/tests/font.c
new file mode 100644
index 0000000..512fa4f
--- /dev/null
+++ b/dlls/dwrite/tests/font.c
@@ -0,0 +1,101 @@
+/*
+ *    Font related tests
+ *
+ * Copyright 2012 Nikolay Sivov for CodeWeavers
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Lesser General Public
+ * License as published by the Free Software Foundation; either
+ * version 2.1 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Lesser General Public License for more details.
+ *
+ * You should have received a copy of the GNU Lesser General Public
+ * License along with this library; if not, write to the Free Software
+ * Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301, USA
+ */
+
+#define COBJMACROS
+
+#include "windows.h"
+//#include "ole2.h"
+
+#include "initguid.h"
+#include "dwrite.h"
+
+#include "wine/test.h"
+
+#define EXPECT_HR(hr,hr_exp) \
+    ok(hr == hr_exp, "got 0x%08x, expected 0x%08x\n", hr, hr_exp)
+
+IDWriteFactory *factory;
+
+static void test_CreateFontFromLOGFONT(void)
+{
+    static const WCHAR arialW[] = {'A','r','i','a','l',0};
+    IDWriteGdiInterop *interop;
+    DWRITE_FONT_WEIGHT weight;
+    DWRITE_FONT_STYLE style;
+    IDWriteFont *font;
+    LOGFONTW logfont;
+    HRESULT hr;
+    BOOL ret;
+
+    hr = IDWriteFactory_GetGdiInterop(factory, &interop);
+    EXPECT_HR(hr, S_OK);
+
+if (0)
+    /* null out parameter crashes this call */
+    hr = IDWriteGdiInterop_CreateFontFromLOGFONT(interop, NULL, NULL);
+
+    hr = IDWriteGdiInterop_CreateFontFromLOGFONT(interop, NULL, &font);
+    EXPECT_HR(hr, E_INVALIDARG);
+
+    memset(&logfont, 0, sizeof(logfont));
+    logfont.lfHeight = 12;
+    logfont.lfWidth  = 12;
+    logfont.lfWeight = FW_NORMAL;
+    logfont.lfItalic = 1;
+    lstrcpyW(logfont.lfFaceName, arialW);
+
+    hr = IDWriteGdiInterop_CreateFontFromLOGFONT(interop, &logfont, &font);
+todo_wine
+    EXPECT_HR(hr, S_OK);
+
+if (hr == S_OK)
+{
+    /* now check properties */
+    weight = IDWriteFont_GetWeight(font);
+    ok(weight == DWRITE_FONT_WEIGHT_NORMAL, "got %d\n", weight);
+
+    style = IDWriteFont_GetStyle(font);
+    ok(style == DWRITE_FONT_STYLE_ITALIC, "got %d\n", style);
+
+    ret = IDWriteFont_IsSymbolFont(font);
+    ok(!ret, "got %d\n", ret);
+
+    IDWriteFont_Release(font);
+}
+
+    IDWriteGdiInterop_Release(interop);
+}
+
+START_TEST(font)
+{
+    HRESULT hr;
+
+    hr = DWriteCreateFactory(DWRITE_FACTORY_TYPE_ISOLATED, &IID_IDWriteFactory, (IUnknown**)&factory);
+    ok(hr == S_OK, "got 0x%08x\n", hr);
+    if (hr != S_OK)
+    {
+        win_skip("failed to create factory\n");
+        return;
+    }
+
+    test_CreateFontFromLOGFONT();
+
+    IDWriteFactory_Release(factory);
+}
diff --git a/include/dwrite.idl b/include/dwrite.idl
index dc0e135..aff529a 100644
--- a/include/dwrite.idl
+++ b/include/dwrite.idl
@@ -1392,3 +1392,5 @@ interface IDWriteFactory : IUnknown
         FLOAT baseline_y,
         IDWriteGlyphRunAnalysis **analysis);
 }
+
+cpp_quote("HRESULT WINAPI DWriteCreateFactory(DWRITE_FACTORY_TYPE,REFIID,IUnknown**);")
-- 
1.5.6.5




More information about the wine-patches mailing list