[Libreoffice-commits] core.git: Branch 'feature/dialog-screenshots' - fpicker/CppunitTest_fpicker_dialogs_test.mk fpicker/Module_fpicker.mk fpicker/qa

Armin Le Grand Armin.Le.Grand at cib.de
Fri Jul 22 14:30:33 UTC 2016


 fpicker/CppunitTest_fpicker_dialogs_test.mk   |   61 ++++++++++
 fpicker/Module_fpicker.mk                     |    4 
 fpicker/qa/unit/data/fpicker-dialogs-test.txt |   40 ++++++
 fpicker/qa/unit/fpicker-dialogs-test.cxx      |  150 ++++++++++++++++++++++++++
 4 files changed, 255 insertions(+)

New commits:
commit 28699738ec25a4a0c96e2e84b1790991713a6fd4
Author: Armin Le Grand <Armin.Le.Grand at cib.de>
Date:   Fri Jul 22 16:29:17 2016 +0200

    Added dialog test case for screenshot creation for fpicker
    
    Change-Id: I6c94ff393e823552ea9c2332bd05abf5b2fafd54

diff --git a/fpicker/CppunitTest_fpicker_dialogs_test.mk b/fpicker/CppunitTest_fpicker_dialogs_test.mk
new file mode 100644
index 0000000..0caf7e7
--- /dev/null
+++ b/fpicker/CppunitTest_fpicker_dialogs_test.mk
@@ -0,0 +1,61 @@
+# -*- Mode: makefile-gmake; tab-width: 4; indent-tabs-mode: t -*-
+#*************************************************************************
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+#*************************************************************************
+
+$(eval $(call gb_CppunitTest_CppunitTest,fpicker_dialogs_test))
+
+$(eval $(call gb_CppunitTest_add_exception_objects,fpicker_dialogs_test, \
+    fpicker/qa/unit/fpicker-dialogs-test \
+))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,fpicker_dialogs_test))
+
+$(eval $(call gb_CppunitTest_use_libraries,fpicker_dialogs_test, \
+    basegfx \
+    comphelper \
+    cppu \
+    cppuhelper \
+    drawinglayer \
+    editeng \
+    i18nlangtag \
+    i18nutil \
+    msfilter \
+    oox \
+    sal \
+    salhelper \
+    sax \
+    sfx \
+    sot \
+    svl \
+    svt \
+    test \
+    tl \
+    tk \
+    ucbhelper \
+    unotest \
+    utl \
+    vcl \
+    xo \
+    $(gb_UWINAPI) \
+))
+
+$(eval $(call gb_CppunitTest_use_external,fpicker_dialogs_test,boost_headers))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,fpicker_dialogs_test))
+
+$(eval $(call gb_CppunitTest_use_ure,fpicker_dialogs_test))
+$(eval $(call gb_CppunitTest_use_vcl_non_headless_with_windows,fpicker_dialogs_test))
+
+$(eval $(call gb_CppunitTest_use_rdb,fpicker_dialogs_test,services))
+
+# $(eval $(call gb_CppunitTest_use_configuration,fpicker_dialogs_test))
+$(eval $(call gb_CppunitTest_use_instdir_configuration,fpicker_dialogs_test))
+
+# vim: set noet sw=4 ts=4:
diff --git a/fpicker/Module_fpicker.mk b/fpicker/Module_fpicker.mk
index c3c4a6b..2805ca7 100644
--- a/fpicker/Module_fpicker.mk
+++ b/fpicker/Module_fpicker.mk
@@ -32,4 +32,8 @@ $(eval $(call gb_Module_add_targets,fpicker,\
 ))
 endif
 
+$(eval $(call gb_Module_add_slowcheck_targets,fpicker,\
+    CppunitTest_fpicker_dialogs_test \
+))
+
 # vim: set noet sw=4 ts=4:
diff --git a/fpicker/qa/unit/data/fpicker-dialogs-test.txt b/fpicker/qa/unit/data/fpicker-dialogs-test.txt
new file mode 100644
index 0000000..f856a0f
--- /dev/null
+++ b/fpicker/qa/unit/data/fpicker-dialogs-test.txt
@@ -0,0 +1,40 @@
+# -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*-
+#
+# This file is part of the LibreOffice project.
+#
+# This Source Code Form is subject to the terms of the Mozilla Public
+# License, v. 2.0. If a copy of the MPL was not distributed with this
+# file, You can obtain one at http://mozilla.org/MPL/2.0/.
+#
+
+# This file contains all dialogs that the unit tests in the module
+# will work on if it is in script mode. It will read one-by-one,
+# try to open it and create a screenshot that will be saved in
+# workdir/screenshots using the pattern of the ui-file name.
+#
+# Syntax:
+# - emty lines are allowed
+# - lines starting with '#' are treated as comment
+# - all other lines should contain a *.ui filename in the same
+#   notation as in the dialog constructors (see code)
+
+#
+# The 'known' dialogs which have a hard-coded representation
+# in registerKnownDialogsByID/createDialogByID
+#
+
+# No known dialogs in fpicker for now
+
+#
+# Dialogs without a hard-coded representation. These will
+# be visualized using a fallback based on VclBuilder
+#
+
+# currently deactivated, leads to problems and the test to not work
+# This is typically a hint that these should be hard-coded in the
+# test case since they need some document and model data to work
+
+fps/ui/foldernamedialog.ui
+fps/ui/explorerfiledialog.ui
+fps/ui/explorerfiledialog.ui
+fps/ui/remotefilesdialog.ui
diff --git a/fpicker/qa/unit/fpicker-dialogs-test.cxx b/fpicker/qa/unit/fpicker-dialogs-test.cxx
new file mode 100644
index 0000000..82269a1
--- /dev/null
+++ b/fpicker/qa/unit/fpicker-dialogs-test.cxx
@@ -0,0 +1,150 @@
+/* -*- Mode: C++; tab-width: 4; indent-tabs-mode: nil; c-basic-offset: 4 -*- */
+/*
+ * This file is part of the LibreOffice project.
+ *
+ * This Source Code Form is subject to the terms of the Mozilla Public
+ * License, v. 2.0. If a copy of the MPL was not distributed with this
+ * file, You can obtain one at http://mozilla.org/MPL/2.0/.
+ */
+
+#include <sal/config.h>
+#include <test/screenshot_test.hxx>
+#include <rtl/strbuf.hxx>
+#include <osl/file.hxx>
+#include <sfx2/app.hxx>
+#include <vcl/abstdlg.hxx>
+
+using namespace ::com::sun::star;
+
+/// Test opening a dialog in fpicker
+class FpickerDialogsTest : public ScreenshotTest
+{
+private:
+    /// helper method to populate KnownDialogs, called in setUp(). Needs to be
+    /// written and has to add entries to KnownDialogs
+    virtual void registerKnownDialogsByID(mapType& rKnownDialogs) override;
+
+    /// dialog creation for known dialogs by ID. Has to be implemented for
+    /// each registered known dialog
+    virtual VclAbstractDialog* createDialogByID(sal_uInt32 nID) override;
+
+public:
+    FpickerDialogsTest();
+    virtual ~FpickerDialogsTest();
+
+    virtual void setUp() override;
+
+    // try to open a dialog
+    void openAnyDialog();
+
+    CPPUNIT_TEST_SUITE(FpickerDialogsTest);
+    CPPUNIT_TEST(openAnyDialog);
+    CPPUNIT_TEST_SUITE_END();
+};
+
+FpickerDialogsTest::FpickerDialogsTest()
+{
+}
+
+FpickerDialogsTest::~FpickerDialogsTest()
+{
+}
+
+void FpickerDialogsTest::setUp()
+{
+    ScreenshotTest::setUp();
+}
+
+void FpickerDialogsTest::registerKnownDialogsByID(mapType& /*rKnownDialogs*/)
+{
+    // fill map of known dilogs
+}
+
+VclAbstractDialog* FpickerDialogsTest::createDialogByID(sal_uInt32 /*nID*/)
+{
+    return nullptr;
+}
+
+void FpickerDialogsTest::openAnyDialog()
+{
+    /// example how to process an input file containing the UXMLDescriptions of the dialogs
+    /// to dump
+    if (true)
+    {
+        test::Directories aDirectories;
+        OUString aURL = aDirectories.getURLFromSrc("fpicker/qa/unit/data/fpicker-dialogs-test.txt");
+        SvFileStream aStream(aURL, StreamMode::READ);
+        OString aNextUIFile;
+        const OString aComment("#");
+
+        while (aStream.ReadLine(aNextUIFile))
+        {
+            if (!aNextUIFile.isEmpty() && !aNextUIFile.startsWith(aComment))
+            {
+                // first check if it's a known dialog
+                std::unique_ptr<VclAbstractDialog> pDlg(createDialogByName(aNextUIFile));
+
+                if (pDlg)
+                {
+                    // known dialog, dump screenshot to path
+                    dumpDialogToPath(*pDlg);
+                }
+                else
+                {
+                    // unknown dialog, try fallback to generic created
+                    // VclBuilder-generated instance. Keep in mind that Dialogs
+                    // using this mechanism will probably not be layouted well
+                    // since the setup/initialization part is missing. Thus,
+                    // only use for fallback when only the UI file is available.
+                    dumpDialogToPath(aNextUIFile);
+                }
+            }
+        }
+    }
+
+    /// example how to dump all known dialogs
+    if (false)
+    {
+        // example for SfxTabDialog: 5 -> "modules/sdraw/ui/drawpagedialog.ui"
+        // example for TabDialog: 22 -> "modules/simpress/ui/headerfooterdialog.ui"
+        // example for self-adapted wizard: 0 -> "modules/simpress/ui/publishingdialog.ui"
+        for (mapType::const_iterator i = getKnownDialogs().begin(); i != getKnownDialogs().end(); i++)
+        {
+            std::unique_ptr<VclAbstractDialog> pDlg(createDialogByID((*i).second));
+
+            if (pDlg)
+            {
+                // known dialog, dump screenshot to path
+                dumpDialogToPath(*pDlg);
+            }
+            else
+            {
+                // unknown dialog, should not happen in this basic loop.
+                // You have probably forgotten to add a case and
+                // implementastion to createDialogByID, please do this
+            }
+        }
+    }
+
+    /// example how to dump a dialog using fallback functionality
+    if (false)
+    {
+        // unknown dialog, try fallback to generic created
+        // VclBuilder-generated instance. Keep in mind that Dialogs
+        // using this mechanism will probably not be layouted well
+        // since the setup/initialization part is missing. Thus,
+        // only use for fallback when only the UI file is available.
+        //
+        // Take any example here, it's only for demonstration - using
+        // even a known one to demonstrate the fallback possibility
+        const OString aUIXMLDescription("fps/ui/foldernamedialog.ui");
+
+        dumpDialogToPath(aUIXMLDescription);
+    }
+}
+
+CPPUNIT_TEST_SUITE_REGISTRATION(FpickerDialogsTest);
+
+CPPUNIT_PLUGIN_IMPLEMENT();
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */


More information about the Libreoffice-commits mailing list