[Libreoffice-commits] core.git: framework/CppunitTest_framework_dispatch.mk framework/Module_framework.mk framework/qa

Miklos Vajna vmiklos at collabora.co.uk
Fri Jun 3 07:03:49 UTC 2016


 framework/CppunitTest_framework_dispatch.mk |   40 +++++
 framework/Module_framework.mk               |    4 
 framework/qa/cppunit/dispatchtest.cxx       |  206 ++++++++++++++++++++++++++++
 3 files changed, 250 insertions(+)

New commits:
commit f8f0f834a174f1c63de16dbeb61de91d33de2092
Author: Miklos Vajna <vmiklos at collabora.co.uk>
Date:   Thu Jun 2 20:31:46 2016 +0200

    Add initial CppunitTest_framework_dispatch
    
    Fails with commit 08cf2fd01064306eef7fdbb5b62320947c4d1089 (framework:
    last dispatchInterceptor gets asked first, 2016-05-20) reverted.
    
    Change-Id: I0e025a5d8996ecafa6b2fef753d6134079936b00
    Reviewed-on: https://gerrit.libreoffice.org/25808
    Reviewed-by: Miklos Vajna <vmiklos at collabora.co.uk>
    Tested-by: Jenkins <ci at libreoffice.org>

diff --git a/framework/CppunitTest_framework_dispatch.mk b/framework/CppunitTest_framework_dispatch.mk
new file mode 100644
index 0000000..7e99ed5
--- /dev/null
+++ b/framework/CppunitTest_framework_dispatch.mk
@@ -0,0 +1,40 @@
+# -*- 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,framework_dispatch))
+
+$(eval $(call gb_CppunitTest_add_exception_objects,framework_dispatch, \
+    framework/qa/cppunit/dispatchtest \
+))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,framework_dispatch))
+
+$(eval $(call gb_CppunitTest_use_libraries,framework_dispatch, \
+	comphelper \
+	cppu \
+	cppuhelper \
+	fwk \
+	sal \
+	test \
+	unotest \
+	$(gb_UWINAPI) \
+))
+
+$(eval $(call gb_CppunitTest_use_external,framework_dispatch,boost_headers))
+
+$(eval $(call gb_CppunitTest_use_sdk_api,framework_dispatch))
+
+$(eval $(call gb_CppunitTest_use_ure,framework_dispatch))
+$(eval $(call gb_CppunitTest_use_vcl,framework_dispatch))
+
+$(eval $(call gb_CppunitTest_use_rdb,framework_dispatch,services))
+
+$(eval $(call gb_CppunitTest_use_configuration,framework_dispatch))
+
+# vim: set noet sw=4 ts=4:
diff --git a/framework/Module_framework.mk b/framework/Module_framework.mk
index 2e6d5b7..f048989 100644
--- a/framework/Module_framework.mk
+++ b/framework/Module_framework.mk
@@ -28,6 +28,10 @@ $(eval $(call gb_Module_add_targets,framework,\
     Package_dtd \
 ))
 
+$(eval $(call gb_Module_add_slowcheck_targets,framework,\
+    CppunitTest_framework_dispatch \
+))
+
 $(eval $(call gb_Module_add_l10n_targets,framework,\
     AllLangResTarget_fwe \
     UIConfig_startmodule \
diff --git a/framework/qa/cppunit/dispatchtest.cxx b/framework/qa/cppunit/dispatchtest.cxx
new file mode 100644
index 0000000..b6b9809
--- /dev/null
+++ b/framework/qa/cppunit/dispatchtest.cxx
@@ -0,0 +1,206 @@
+/* -*- 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 <cppuhelper/implbase.hxx>
+#include <test/bootstrapfixture.hxx>
+#include <unotest/macros_test.hxx>
+
+#include <com/sun/star/frame/Desktop.hpp>
+#include <com/sun/star/frame/DispatchHelper.hpp>
+#include <com/sun/star/frame/XDispatchProviderInterceptor.hpp>
+#include <com/sun/star/frame/XInterceptorInfo.hpp>
+
+#include <comphelper/processfactory.hxx>
+
+using namespace ::com::sun::star;
+
+namespace
+{
+
+/// Sample interception implementation that asserts getInterceptedURLs() and queryDispatch() is in sync.
+class MyInterceptor : public cppu::WeakImplHelper
+    <
+    frame::XDispatchProviderInterceptor,
+    frame::XInterceptorInfo
+    >
+{
+    uno::Reference<frame::XDispatchProvider> m_xMaster;
+    uno::Reference<frame::XDispatchProvider> m_xSlave;
+    uno::Sequence<OUString> m_aDisabledCommands;
+    int m_nExpected;
+    int m_nUnexpected;
+
+public:
+    MyInterceptor();
+
+    /// Number of queryDispatch() calls that operate on a command advertised by getInterceptedURLs().
+    int getExpected();
+    /// Number of queryDispatch() calls that operate on a command not advertised by getInterceptedURLs().
+    int getUnexpected();
+
+    // frame::XInterceptorInfo
+    virtual uno::Sequence<OUString> SAL_CALL getInterceptedURLs() throw (uno::RuntimeException, std::exception) override;
+
+    // frame::XDispatchProviderInterceptor
+    virtual void SAL_CALL setMasterDispatchProvider(const uno::Reference<frame::XDispatchProvider>& xNewSupplier) throw (uno::RuntimeException, std::exception) override;
+    virtual uno::Reference<frame::XDispatchProvider> SAL_CALL getMasterDispatchProvider() throw (uno::RuntimeException, std::exception) override;
+    virtual void SAL_CALL setSlaveDispatchProvider(const uno::Reference<frame::XDispatchProvider>& xNewSupplier) throw (uno::RuntimeException, std::exception) override;
+    virtual uno::Reference<frame::XDispatchProvider> SAL_CALL getSlaveDispatchProvider() throw (uno::RuntimeException, std::exception) override;
+
+    // frame::XDispatchProvider
+    virtual uno::Sequence<uno::Reference<frame::XDispatch>> SAL_CALL queryDispatches(const uno::Sequence<frame::DispatchDescriptor>& xRequests) throw (uno::RuntimeException, std::exception) override;
+    virtual uno::Reference<frame::XDispatch> SAL_CALL queryDispatch(const util::URL& rURL, const OUString& rTargetFrameName, sal_Int32 SearchFlags) throw (uno::RuntimeException, std::exception) override;
+};
+
+MyInterceptor::MyInterceptor()
+    : m_nExpected(0),
+      m_nUnexpected(0)
+{
+    // Not part of the initializer list as MSVC 2013 would not handle that.
+    m_aDisabledCommands = {".uno:Bold"};
+}
+
+int MyInterceptor::getExpected()
+{
+    int nRet = m_nExpected;
+    m_nExpected = 0;
+    return nRet;
+}
+
+int MyInterceptor::getUnexpected()
+{
+    int nRet = m_nUnexpected;
+    m_nUnexpected = 0;
+    return nRet;
+}
+
+uno::Sequence<OUString> MyInterceptor::getInterceptedURLs() throw (uno::RuntimeException, std::exception)
+{
+    return m_aDisabledCommands;
+}
+
+void MyInterceptor::setMasterDispatchProvider(const uno::Reference<frame::XDispatchProvider>& xNewSupplier) throw (uno::RuntimeException, std::exception)
+{
+    m_xMaster = xNewSupplier;
+}
+
+uno::Reference<frame::XDispatchProvider> MyInterceptor::getMasterDispatchProvider() throw (uno::RuntimeException, std::exception)
+{
+    return m_xMaster;
+}
+
+void MyInterceptor::setSlaveDispatchProvider(const uno::Reference<frame::XDispatchProvider>& xNewSupplier) throw (uno::RuntimeException, std::exception)
+{
+    m_xSlave = xNewSupplier;
+}
+
+uno::Reference<frame::XDispatchProvider> MyInterceptor::getSlaveDispatchProvider() throw (uno::RuntimeException, std::exception)
+{
+    return m_xSlave;
+}
+
+uno::Sequence<uno::Reference<frame::XDispatch>> MyInterceptor::queryDispatches(const uno::Sequence<frame::DispatchDescriptor>& rRequests) throw (uno::RuntimeException, std::exception)
+{
+    uno::Sequence<uno::Reference<frame::XDispatch>> aResult(rRequests.getLength());
+
+    for (sal_Int32 i = 0; i < rRequests.getLength(); ++i)
+    {
+        aResult[i] = queryDispatch(rRequests[i].FeatureURL, rRequests[i].FrameName, rRequests[i].SearchFlags);
+    }
+
+    return aResult;
+}
+
+uno::Reference<frame::XDispatch> MyInterceptor::queryDispatch(const util::URL& rURL, const OUString& /*rTargetFrameName*/, sal_Int32 /*SearchFlags*/)
+throw (uno::RuntimeException, std::exception)
+{
+    if (std::find(m_aDisabledCommands.begin(), m_aDisabledCommands.end(), rURL.Complete) != m_aDisabledCommands.end())
+        ++m_nExpected;
+    else
+        ++m_nUnexpected;
+
+    return uno::Reference<frame::XDispatch>();
+}
+
+/// Tests how InterceptionHelper invokes a registered interceptor.
+class DispatchTest : public test::BootstrapFixture, public unotest::MacrosTest
+{
+    uno::Reference<uno::XComponentContext> mxComponentContext;
+    uno::Reference<lang::XComponent> mxComponent;
+    void dispatchCommand(const uno::Reference<lang::XComponent>& xComponent, const OUString& rCommand, const uno::Sequence<beans::PropertyValue>& rPropertyValues);
+
+public:
+    virtual void setUp() override;
+    virtual void tearDown() override;
+    void testInterception();
+
+    CPPUNIT_TEST_SUITE(DispatchTest);
+    CPPUNIT_TEST(testInterception);
+    CPPUNIT_TEST_SUITE_END();
+};
+
+void DispatchTest::setUp()
+{
+    test::BootstrapFixture::setUp();
+
+    mxComponentContext.set(comphelper::getComponentContext(getMultiServiceFactory()));
+    mxDesktop.set(frame::Desktop::create(mxComponentContext));
+}
+
+void DispatchTest::tearDown()
+{
+    if (mxComponent.is())
+        mxComponent->dispose();
+
+    test::BootstrapFixture::tearDown();
+}
+
+void DispatchTest::dispatchCommand(const uno::Reference<lang::XComponent>& xComponent, const OUString& rCommand, const uno::Sequence<beans::PropertyValue>& rPropertyValues)
+{
+    uno::Reference<frame::XController> xController = uno::Reference<frame::XModel>(xComponent, uno::UNO_QUERY)->getCurrentController();
+    CPPUNIT_ASSERT(xController.is());
+    uno::Reference<frame::XDispatchProvider> xFrame(xController->getFrame(), uno::UNO_QUERY);
+    CPPUNIT_ASSERT(xFrame.is());
+
+    uno::Reference<uno::XComponentContext> xContext = ::comphelper::getProcessComponentContext();
+    uno::Reference<frame::XDispatchHelper> xDispatchHelper(frame::DispatchHelper::create(xContext));
+    CPPUNIT_ASSERT(xDispatchHelper.is());
+
+    xDispatchHelper->executeDispatch(xFrame, rCommand, OUString(), 0, rPropertyValues);
+}
+
+void DispatchTest::testInterception()
+{
+    mxComponent = loadFromDesktop("private:factory/swriter", "com.sun.star.text.TextDocument");
+    CPPUNIT_ASSERT(mxComponent.is());
+    uno::Reference<frame::XModel> xModel(mxComponent, uno::UNO_QUERY);
+    CPPUNIT_ASSERT(xModel.is());
+
+    uno::Reference<frame::XDispatchProviderInterception> xRegistration(xModel->getCurrentController()->getFrame(), uno::UNO_QUERY);
+    CPPUNIT_ASSERT(xRegistration.is());
+
+    rtl::Reference<MyInterceptor> pInterceptor(new MyInterceptor());
+    xRegistration->registerDispatchProviderInterceptor(uno::Reference<frame::XDispatchProviderInterceptor>(pInterceptor.get()));
+
+    dispatchCommand(mxComponent, ".uno:Bold", {});
+    CPPUNIT_ASSERT_EQUAL(1, pInterceptor->getExpected());
+    CPPUNIT_ASSERT_EQUAL(0, pInterceptor->getUnexpected());
+    dispatchCommand(mxComponent, ".uno:Italic", {});
+    CPPUNIT_ASSERT_EQUAL(0, pInterceptor->getExpected());
+    // This was 1: MyInterceptor::queryDispatch() was called for .uno:Italic.
+    CPPUNIT_ASSERT_EQUAL(0, pInterceptor->getUnexpected());
+}
+
+CPPUNIT_TEST_SUITE_REGISTRATION(DispatchTest);
+
+}
+
+CPPUNIT_PLUGIN_IMPLEMENT();
+
+/* vim:set shiftwidth=4 softtabstop=4 expandtab: */


More information about the Libreoffice-commits mailing list