[Libreoffice-commits] core.git: 2 commits - qadevOOo/Jar_OOoRunner.mk qadevOOo/objdsc qadevOOo/tests sc/qa

Libreoffice Gerrit user logerrit at kemper.freedesktop.org
Sat Mar 2 08:29:14 UTC 2019


 qadevOOo/Jar_OOoRunner.mk                                           |    1 
 qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv |    5 
 qadevOOo/tests/java/mod/_sc/ScDataPilotItemsObj.java                |  327 ----------
 sc/qa/extras/scdatapilotitemsobj.cxx                                |   15 
 sc/qa/unoapi/sc_4.sce                                               |    1 
 5 files changed, 15 insertions(+), 334 deletions(-)

New commits:
commit bb6f5dc046966b7295dc199ea07008098757f0ef
Author:     Jens Carl <j.carl43 at gmx.de>
AuthorDate: Sat Mar 2 01:42:20 2019 +0000
Commit:     Jens Carl <j.carl43 at gmx.de>
CommitDate: Sat Mar 2 09:29:03 2019 +0100

    tdf#45904 Move XNameAccess Java tests to C++
    
    Move XNameAccess Java tests to C++ for ScDataPilotItemsObj.
    
    Change-Id: Id8e83bb68408600f471148df7527467a371590fb
    Reviewed-on: https://gerrit.libreoffice.org/68611
    Tested-by: Jenkins
    Reviewed-by: Jens Carl <j.carl43 at gmx.de>

diff --git a/qadevOOo/Jar_OOoRunner.mk b/qadevOOo/Jar_OOoRunner.mk
index b73fd46f1c44..d74a31d909db 100644
--- a/qadevOOo/Jar_OOoRunner.mk
+++ b/qadevOOo/Jar_OOoRunner.mk
@@ -967,7 +967,6 @@ $(eval $(call gb_Jar_add_sourcefiles,OOoRunner,\
     qadevOOo/tests/java/mod/_sc/ScDataPilotFieldGroupObj \
     qadevOOo/tests/java/mod/_sc/ScDataPilotFieldObj \
     qadevOOo/tests/java/mod/_sc/ScDataPilotItemObj \
-    qadevOOo/tests/java/mod/_sc/ScDataPilotItemsObj \
     qadevOOo/tests/java/mod/_sc/ScDocumentConfiguration \
     qadevOOo/tests/java/mod/_sc/ScDrawPageObj \
     qadevOOo/tests/java/mod/_sc/ScHeaderFieldObj \
diff --git a/qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv b/qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv
deleted file mode 100644
index 711e0e47234e..000000000000
--- a/qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv
+++ /dev/null
@@ -1,3 +0,0 @@
-"ScDataPilotItemsObj";"com::sun::star::container::XNameAccess";"getByName()"
-"ScDataPilotItemsObj";"com::sun::star::container::XNameAccess";"getElementNames()"
-"ScDataPilotItemsObj";"com::sun::star::container::XNameAccess";"hasByName()"
diff --git a/qadevOOo/tests/java/mod/_sc/ScDataPilotItemsObj.java b/qadevOOo/tests/java/mod/_sc/ScDataPilotItemsObj.java
deleted file mode 100644
index 3d06faee132f..000000000000
--- a/qadevOOo/tests/java/mod/_sc/ScDataPilotItemsObj.java
+++ /dev/null
@@ -1,327 +0,0 @@
-/*
- * 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 incorporates work covered by the following license notice:
- *
- *   Licensed to the Apache Software Foundation (ASF) under one or more
- *   contributor license agreements. See the NOTICE file distributed
- *   with this work for additional information regarding copyright
- *   ownership. The ASF licenses this file to you under the Apache
- *   License, Version 2.0 (the "License"); you may not use this file
- *   except in compliance with the License. You may obtain a copy of
- *   the License at http://www.apache.org/licenses/LICENSE-2.0 .
- */
-package mod._sc;
-
-import com.sun.star.beans.XPropertySet;
-import com.sun.star.container.XIndexAccess;
-import com.sun.star.container.XNamed;
-import com.sun.star.lang.XComponent;
-import com.sun.star.sheet.DataPilotFieldOrientation;
-import com.sun.star.sheet.XDataPilotDescriptor;
-import com.sun.star.sheet.XDataPilotField;
-import com.sun.star.sheet.XDataPilotTables;
-import com.sun.star.sheet.XDataPilotTablesSupplier;
-import com.sun.star.sheet.XSpreadsheet;
-import com.sun.star.sheet.XSpreadsheetDocument;
-import com.sun.star.sheet.XSpreadsheets;
-import com.sun.star.table.CellAddress;
-import com.sun.star.table.CellRangeAddress;
-import com.sun.star.uno.AnyConverter;
-import com.sun.star.uno.Type;
-import com.sun.star.uno.UnoRuntime;
-import com.sun.star.uno.XInterface;
-
-import java.io.PrintWriter;
-
-import lib.TestCase;
-import lib.TestEnvironment;
-import lib.TestParameters;
-
-import util.SOfficeFactory;
-
-
-/**
-* Test for object which is represented by service
-* <code>com.sun.star.sheet.DataPilotField</code>. <p>
-* Object implements the following interfaces :
-* <ul>
-*  <li> <code>com::sun::star::container::XNamed</code></li>
-*  <li> <code>com::sun::star::sheet::DataPilotField</code></li>
-*  <li> <code>com::sun::star::beans::XPropertySet</code></li>
-* </ul>
-* @see com.sun.star.sheet.DataPilotField
-* @see com.sun.star.container.XNamed
-* @see com.sun.star.sheet.DataPilotField
-* @see com.sun.star.beans.XPropertySet
-* @see ifc.container._XNamed
-* @see ifc.sheet._DataPilotField
-* @see ifc.beans._XPropertySet
-*/
-public class ScDataPilotItemsObj extends TestCase {
-    private XSpreadsheetDocument xSheetDoc = null;
-
-    /**
-     * A field is filled some values. This integer determines the size of the
-     * field in x and y direction.
-     */
-    private static final int mMaxFieldIndex = 6;
-
-    /**
-    * Creates Spreadsheet document.
-    */
-    @Override
-    protected void initialize(TestParameters tParam, PrintWriter log) throws Exception {
-        SOfficeFactory SOF = SOfficeFactory.getFactory(
-                                     tParam.getMSF());
-
-        log.println("creating a Spreadsheet document");
-        xSheetDoc = SOF.createCalcDoc(null);
-    }
-
-    /**
-    * Disposes Spreadsheet document.
-    */
-    @Override
-    protected void cleanup(TestParameters tParam, PrintWriter log) {
-        log.println("    disposing xSheetDoc ");
-
-        XComponent oComp = UnoRuntime.queryInterface(
-                                   XComponent.class, xSheetDoc);
-        util.DesktopTools.closeDoc(oComp);
-    }
-
-    /**
-    * Creating a TestEnvironment for the interfaces to be tested.
-    * Retrieves a collection of spreadsheets from a document
-    * and takes one of them. Fills some table in the spreadsheet.
-    * Obtains the collection of data pilot tables using the interface
-    * <code>XDataPilotTablesSupplier</code>. Creates a data pilot descriptor
-    * for the filled table and inserts new data pilot table with this descriptor
-    * to the collection. Obtains the collection of all the data pilot fields
-    * using the interface <code>XDataPilotDescriptor</code>. Retrieves from
-    * the collection the data pilot field with index 0. This data pilot field
-    * is the instance of the service <code>com.sun.star.sheet.DataPilotField</code>.
-    * @see com.sun.star.sheet.DataPilotField
-    * @see com.sun.star.sheet.XDataPilotTablesSupplier
-    * @see com.sun.star.sheet.XDataPilotDescriptor
-    */
-    @Override
-    protected TestEnvironment createTestEnvironment(TestParameters Param,
-                                                                 PrintWriter log) throws Exception {
-        XInterface oObj = null;
-
-
-        // creation of testobject here
-        // first we write what we are intend to do to log file
-        log.println("Creating a test environment");
-
-        // the cell range
-        CellRangeAddress sCellRangeAdress = new CellRangeAddress();
-        sCellRangeAdress.Sheet = 0;
-        sCellRangeAdress.StartColumn = 1;
-        sCellRangeAdress.StartRow = 0;
-        sCellRangeAdress.EndColumn = mMaxFieldIndex - 1;
-        sCellRangeAdress.EndRow = mMaxFieldIndex - 1;
-
-        // position of the data pilot table
-        CellAddress sCellAdress = new CellAddress();
-        sCellAdress.Sheet = 0;
-        sCellAdress.Column = 7;
-        sCellAdress.Row = 8;
-
-        log.println("Getting a sheet");
-
-        XSpreadsheets xSpreadsheets = xSheetDoc.getSheets();
-        XSpreadsheet oSheet = null;
-        XSpreadsheet oSheet2 = null;
-        XIndexAccess oIndexAccess = UnoRuntime.queryInterface(
-                                            XIndexAccess.class, xSpreadsheets);
-        // Make sure there are at least two sheets
-        xSpreadsheets.insertNewByName("Some Sheet", (short)0);
-
-        oSheet = (XSpreadsheet) AnyConverter.toObject(
-                         new Type(XSpreadsheet.class),
-                         oIndexAccess.getByIndex(0));
-        oSheet2 = (XSpreadsheet) AnyConverter.toObject(
-                          new Type(XSpreadsheet.class),
-                          oIndexAccess.getByIndex(1));
-
-        log.println("Filling a table");
-
-        for (int i = 1; i < mMaxFieldIndex; i++) {
-            oSheet.getCellByPosition(i, 0).setFormula("Col" + i);
-            oSheet.getCellByPosition(0, i).setFormula("Row" + i);
-            oSheet2.getCellByPosition(i, 0).setFormula("Col" + i);
-            oSheet2.getCellByPosition(0, i).setFormula("Row" + i);
-        }
-
-        for (int i = 1; i < mMaxFieldIndex; i++)
-            for (int j = 1; j < mMaxFieldIndex; j++) {
-                oSheet.getCellByPosition(i, j).setValue(i * (j + 1));
-                oSheet2.getCellByPosition(i, j).setValue(i * (j + 2));
-            }
-
-        oSheet.getCellByPosition(1, 5);
-
-        int x = sCellAdress.Column;
-        int y = sCellAdress.Row + 3;
-
-
-        oSheet.getCellByPosition(x, y);
-
-
-        // create the test objects
-        log.println("Getting test objects");
-
-        XDataPilotTablesSupplier DPTS = UnoRuntime.queryInterface(
-                                                XDataPilotTablesSupplier.class,
-                                                oSheet);
-        XDataPilotTables DPT = DPTS.getDataPilotTables();
-        XDataPilotDescriptor DPDsc = DPT.createDataPilotDescriptor();
-        DPDsc.setSourceRange(sCellRangeAdress);
-
-        XPropertySet fieldPropSet = null;
-
-        Object oDataPilotField = DPDsc.getDataPilotFields().getByIndex(0);
-        fieldPropSet = UnoRuntime.queryInterface(
-                               XPropertySet.class, oDataPilotField);
-
-        fieldPropSet.setPropertyValue("Function",
-                                      com.sun.star.sheet.GeneralFunction.SUM);
-        fieldPropSet.setPropertyValue("Orientation",
-                                      com.sun.star.sheet.DataPilotFieldOrientation.DATA);
-
-        log.println("Insert the DataPilotTable");
-
-        if (DPT.hasByName("DataPilotTable")) {
-            DPT.removeByName("DataPilotTable");
-        }
-
-        XIndexAccess IA = DPDsc.getDataPilotFields();
-        getSRange(IA);
-
-        DPT.insertNewByName("DataPilotTable", sCellAdress, DPDsc);
-
-        oObj = (XInterface) AnyConverter.toObject(
-                       new Type(XInterface.class), IA.getByIndex(0));
-
-        log.println("Creating object - " +
-                    ((oObj == null) ? "FAILED" : "OK"));
-
-        XDataPilotField xDataPilotField = UnoRuntime.queryInterface(
-                                                  XDataPilotField.class, oObj);
-
-        oObj = xDataPilotField.getItems();
-
-        TestEnvironment tEnv = new TestEnvironment(oObj);
-
-        log.println("Implementationname: " + util.utils.getImplName(oObj));
-
-        // Other parameters required for interface tests
-        return tEnv;
-    }
-
-    private void getSRange(XIndexAccess IA) {
-        int fieldsAmount = IA.getCount() + 1;
-
-        String[] fieldsNames = new String[fieldsAmount];
-
-        int i = -1;
-        int cnt = 0;
-
-        while ((++i) < fieldsAmount) {
-            Object field;
-
-            try {
-                field = IA.getByIndex(i);
-            } catch (com.sun.star.lang.WrappedTargetException e) {
-                e.printStackTrace(log);
-
-                return;
-            } catch (com.sun.star.lang.IndexOutOfBoundsException e) {
-                e.printStackTrace(log);
-
-                return;
-            }
-
-            XNamed named = UnoRuntime.queryInterface(XNamed.class,
-                                                              field);
-            String name = named.getName();
-
-            log.println("**Field : '" + name + "' ... ");
-
-            if (!name.equals("Data")) {
-                fieldsNames[cnt] = name;
-
-                XPropertySet props = UnoRuntime.queryInterface(
-                                             XPropertySet.class, field);
-
-                try {
-                    switch (cnt % 5) {
-                    case 0:
-                        props.setPropertyValue("Orientation",
-                                               DataPilotFieldOrientation.COLUMN);
-                        log.println("  Column");
-
-                        break;
-
-                    case 1:
-                        props.setPropertyValue("Orientation",
-                                               DataPilotFieldOrientation.ROW);
-                        log.println("  Row");
-
-                        break;
-
-                    case 2:
-                        props.setPropertyValue("Orientation",
-                                               DataPilotFieldOrientation.DATA);
-                        log.println("  Data");
-
-                        break;
-
-                    case 3:
-                        props.setPropertyValue("Orientation",
-                                               DataPilotFieldOrientation.HIDDEN);
-                        log.println("  Hidden");
-
-                        break;
-
-                    case 4:
-                        props.setPropertyValue("Orientation",
-                                               DataPilotFieldOrientation.PAGE);
-                        log.println("  Page");
-
-                        break;
-                    }
-                } catch (com.sun.star.lang.WrappedTargetException e) {
-                    e.printStackTrace(log);
-
-                    return;
-                } catch (com.sun.star.lang.IllegalArgumentException e) {
-                    e.printStackTrace(log);
-
-                    return;
-                } catch (com.sun.star.beans.PropertyVetoException e) {
-                    e.printStackTrace(log);
-
-                    return;
-                } catch (com.sun.star.beans.UnknownPropertyException e) {
-                    e.printStackTrace(log);
-
-                    return;
-                }
-
-                if ((++cnt) > 4) {
-                    break;
-                }
-            } else {
-                return;
-            }
-        }
-    }
-}
diff --git a/sc/qa/extras/scdatapilotitemsobj.cxx b/sc/qa/extras/scdatapilotitemsobj.cxx
index cf0b7057e517..720a2ad9e8ec 100644
--- a/sc/qa/extras/scdatapilotitemsobj.cxx
+++ b/sc/qa/extras/scdatapilotitemsobj.cxx
@@ -11,6 +11,7 @@
 #include <test/container/xelementaccess.hxx>
 #include <test/container/xenumerationaccess.hxx>
 #include <test/container/xindexaccess.hxx>
+#include <test/container/xnameaccess.hxx>
 #include <test/lang/xserviceinfo.hxx>
 
 #include <com/sun/star/beans/XPropertySet.hpp>
@@ -45,6 +46,7 @@ class ScDataPilotItemsObj : public CalcUnoApiTest,
                             public apitest::XElementAccess,
                             public apitest::XEnumerationAccess,
                             public apitest::XIndexAccess,
+                            public apitest::XNameAccess,
                             public apitest::XServiceInfo
 {
 public:
@@ -67,6 +69,11 @@ public:
     CPPUNIT_TEST(testGetByIndex);
     CPPUNIT_TEST(testGetCount);
 
+    // XNameAccess
+    CPPUNIT_TEST(testGetByName);
+    CPPUNIT_TEST(testGetElementNames);
+    CPPUNIT_TEST(testHasByName);
+
     // XServiceInfo
     CPPUNIT_TEST(testGetImplementationName);
     CPPUNIT_TEST(testGetSupportedServiceNames);
@@ -82,6 +89,7 @@ ScDataPilotItemsObj::ScDataPilotItemsObj()
     : CalcUnoApiTest("/sc/qa/extras/testdocuments")
     , XElementAccess(cppu::UnoType<beans::XPropertySet>::get())
     , XIndexAccess(5)
+    , XNameAccess("2")
     , XServiceInfo("ScDataPilotItemsObj", "com.sun.star.sheet.DataPilotItems")
 {
 }
diff --git a/sc/qa/unoapi/sc_4.sce b/sc/qa/unoapi/sc_4.sce
index dcff7d102652..2e467f5433e9 100644
--- a/sc/qa/unoapi/sc_4.sce
+++ b/sc/qa/unoapi/sc_4.sce
@@ -16,7 +16,6 @@
 #   the License at http://www.apache.org/licenses/LICENSE-2.0 .
 #
 -o sc.ScDataPilotItemObj
--o sc.ScDataPilotItemsObj
 -o sc.ScDatabaseRangeObj
 -o sc.ScDocumentConfiguration
 -o sc.ScDrawPageObj
commit 67033fd7d0f357d52e424254576ea716ec0ebccc
Author:     Jens Carl <j.carl43 at gmx.de>
AuthorDate: Sat Mar 2 01:33:28 2019 +0000
Commit:     Jens Carl <j.carl43 at gmx.de>
CommitDate: Sat Mar 2 09:28:52 2019 +0100

    tdf#45904 Move XIndexAccess Java tests to C++
    
    Move XIndexAccess Java tests to C++ for ScDataPilotItemsObj.
    
    Change-Id: I66888c4515e84cf0e8f051378616c0ecbf60c0e8
    Reviewed-on: https://gerrit.libreoffice.org/68610
    Tested-by: Jenkins
    Reviewed-by: Jens Carl <j.carl43 at gmx.de>

diff --git a/qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv b/qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv
index 002f0b5ba1dc..711e0e47234e 100644
--- a/qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv
+++ b/qadevOOo/objdsc/sc/com.sun.star.comp.office.ScDataPilotItemsObj.csv
@@ -1,5 +1,3 @@
 "ScDataPilotItemsObj";"com::sun::star::container::XNameAccess";"getByName()"
 "ScDataPilotItemsObj";"com::sun::star::container::XNameAccess";"getElementNames()"
 "ScDataPilotItemsObj";"com::sun::star::container::XNameAccess";"hasByName()"
-"ScDataPilotItemsObj";"com::sun::star::container::XIndexAccess";"getCount()"
-"ScDataPilotItemsObj";"com::sun::star::container::XIndexAccess";"getByIndex()"
diff --git a/sc/qa/extras/scdatapilotitemsobj.cxx b/sc/qa/extras/scdatapilotitemsobj.cxx
index cb97648e796f..cf0b7057e517 100644
--- a/sc/qa/extras/scdatapilotitemsobj.cxx
+++ b/sc/qa/extras/scdatapilotitemsobj.cxx
@@ -10,6 +10,7 @@
 #include <test/calc_unoapi_test.hxx>
 #include <test/container/xelementaccess.hxx>
 #include <test/container/xenumerationaccess.hxx>
+#include <test/container/xindexaccess.hxx>
 #include <test/lang/xserviceinfo.hxx>
 
 #include <com/sun/star/beans/XPropertySet.hpp>
@@ -43,6 +44,7 @@ const auto nMaxFieldIndex = 6;
 class ScDataPilotItemsObj : public CalcUnoApiTest,
                             public apitest::XElementAccess,
                             public apitest::XEnumerationAccess,
+                            public apitest::XIndexAccess,
                             public apitest::XServiceInfo
 {
 public:
@@ -61,6 +63,10 @@ public:
     // XEnumerationAccess
     CPPUNIT_TEST(testCreateEnumeration);
 
+    // XIndexAccess
+    CPPUNIT_TEST(testGetByIndex);
+    CPPUNIT_TEST(testGetCount);
+
     // XServiceInfo
     CPPUNIT_TEST(testGetImplementationName);
     CPPUNIT_TEST(testGetSupportedServiceNames);
@@ -75,6 +81,7 @@ private:
 ScDataPilotItemsObj::ScDataPilotItemsObj()
     : CalcUnoApiTest("/sc/qa/extras/testdocuments")
     , XElementAccess(cppu::UnoType<beans::XPropertySet>::get())
+    , XIndexAccess(5)
     , XServiceInfo("ScDataPilotItemsObj", "com.sun.star.sheet.DataPilotItems")
 {
 }


More information about the Libreoffice-commits mailing list