[Libreoffice-commits] core.git: sc/qa

[Xisco Fauli (via logerrit)]

 sc/qa/unit/uicalc/data/tdf126904.ods |binary
 sc/qa/unit/uicalc/uicalc.cxx         |   50 ++++++++++++++++++++++++++++++++++-
 2 files changed, 49 insertions(+), 1 deletion(-)

New commits:
commit 976cabe84f3b6e5591ccf2b043d72cbca3e31ba0
Author:     Xisco Fauli <xiscofauli at libreoffice.org>
AuthorDate: Mon Mar 9 21:20:19 2020 +0100
Commit:     Xisco Faulí <xiscofauli at libreoffice.org>
CommitDate: Tue Mar 10 11:50:42 2020 +0100

    tdf#126904: Add unittest
    
    Change-Id: Ie5ec86642bcd4a8822d885d18516ee3ad6fecbfe
    Reviewed-on: https://gerrit.libreoffice.org/c/core/+/90250
    Tested-by: Jenkins
    Reviewed-by: Xisco Faulí <xiscofauli at libreoffice.org>

diff --git a/sc/qa/unit/uicalc/data/tdf126904.ods b/sc/qa/unit/uicalc/data/tdf126904.ods
new file mode 100644
index 000000000000..0f2cf05132c4
Binary files /dev/null and b/sc/qa/unit/uicalc/data/tdf126904.ods differ
diff --git a/sc/qa/unit/uicalc/uicalc.cxx b/sc/qa/unit/uicalc/uicalc.cxx
index 3dfcade9de98..193f50e7978e 100644
--- a/sc/qa/unit/uicalc/uicalc.cxx
+++ b/sc/qa/unit/uicalc/uicalc.cxx
@@ -29,7 +29,7 @@ public:
 
     ScModelObj* createDoc(const char* pName);
 
-private:
+protected:
     uno::Reference<lang::XComponent> mxComponent;
 };
 
@@ -87,6 +87,54 @@ CPPUNIT_TEST_FIXTURE(ScUiCalcTest, testTdf122232)
     CPPUNIT_ASSERT_EQUAL(sal_Int32(6), ScDocShell::GetViewData()->GetCurY());
 }
 
+CPPUNIT_TEST_FIXTURE(ScUiCalcTest, testTdf126904)
+{
+    ScModelObj* pModelObj = createDoc("tdf126904.ods");
+    ScDocument* pDoc = pModelObj->GetDocument();
+    CPPUNIT_ASSERT(pDoc);
+
+    CPPUNIT_ASSERT_EQUAL(sal_Int16(0), ScDocShell::GetViewData()->GetCurX());
+    CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+
+    dispatchCommand(mxComponent, ".uno:GoRight", {});
+
+    CPPUNIT_ASSERT_EQUAL(sal_Int16(1), ScDocShell::GetViewData()->GetCurX());
+    CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+
+    dispatchCommand(mxComponent, ".uno:GoRight", {});
+
+    CPPUNIT_ASSERT_EQUAL(sal_Int16(4), ScDocShell::GetViewData()->GetCurX());
+    CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+
+    dispatchCommand(mxComponent, ".uno:GoRight", {});
+
+    CPPUNIT_ASSERT_EQUAL(sal_Int16(5), ScDocShell::GetViewData()->GetCurX());
+    CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+
+    dispatchCommand(mxComponent, ".uno:GoRight", {});
+
+    CPPUNIT_ASSERT_EQUAL(sal_Int16(8), ScDocShell::GetViewData()->GetCurX());
+    CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+
+    dispatchCommand(mxComponent, ".uno:GoRight", {});
+
+    CPPUNIT_ASSERT_EQUAL(sal_Int16(9), ScDocShell::GetViewData()->GetCurX());
+    CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+
+    dispatchCommand(mxComponent, ".uno:GoRight", {});
+
+    CPPUNIT_ASSERT_EQUAL(sal_Int16(12), ScDocShell::GetViewData()->GetCurX());
+    CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+
+    //Cursor can't move forward to the right
+    for (size_t i = 0; i < 5; ++i)
+    {
+        dispatchCommand(mxComponent, ".uno:GoRight", {});
+
+        CPPUNIT_ASSERT_EQUAL(sal_Int16(13), ScDocShell::GetViewData()->GetCurX());
+        CPPUNIT_ASSERT_EQUAL(sal_Int32(4), ScDocShell::GetViewData()->GetCurY());
+    }
+}
 CPPUNIT_PLUGIN_IMPLEMENT();
 
 /* vim:set shiftwidth=4 softtabstop=4 expandtab: */