From f2471280c7c625e39369f76bd0efa1d2402f62bf Mon Sep 17 00:00:00 2001
From: Kevin Hunter <hunteke@earlham.edu>
Date: Sun, 16 Feb 2014 22:12:27 -0500
Subject: [PATCH] Test for fdo#74824.

The bug in question crashed LibO when inserting a group of cells.
This bug was quashed, per se, by commit
07e2c31831ad265b018e5fdf59bdde048fbb4d35, but it occurs to me that at
least the particular functionality of inserting a group of cells could
use more testing.

Change-Id: Icdbfff86fb0265eef325bcc94d9fc9f3e9e38413
---
 pyuno/Module_pyuno.mk                         |   1 +
 pyuno/PythonTest_pytests.mk                   |   1 +
 ...honTest_pyuno_pytests_insertremovecells.mk |  20 +++++
 pyuno/qa/pytests/insertremovecells.py         |  85 ++++++++++++++++++
 pyuno/qa/pytests/testdocuments/fdo74824.ods   | Bin 0 -> 4811 bytes
 5 files changed, 107 insertions(+)
 create mode 100644 pyuno/PythonTest_pyuno_pytests_insertremovecells.mk
 create mode 100644 pyuno/qa/pytests/insertremovecells.py
 create mode 100644 pyuno/qa/pytests/testdocuments/fdo74824.ods

diff --git a/pyuno/Module_pyuno.mk b/pyuno/Module_pyuno.mk
index 31c831a2e1d4..b79c21d458b9 100644
--- a/pyuno/Module_pyuno.mk
+++ b/pyuno/Module_pyuno.mk
@@ -63,6 +63,7 @@ endif # SYSTEM_PYTHON
 ifneq (,$(filter PythonTest_pytests,$(MAKECMDGOALS)))
 $(eval $(call gb_Module_add_targets,pyuno, \
     PythonTest_pytests \
+    PythonTest_pyuno_pytests_insertremovecells \
 ))
 endif
 
diff --git a/pyuno/PythonTest_pytests.mk b/pyuno/PythonTest_pytests.mk
index afb4ba52291a..63ffbb67731c 100644
--- a/pyuno/PythonTest_pytests.mk
+++ b/pyuno/PythonTest_pytests.mk
@@ -24,5 +24,6 @@
 $(eval $(call gb_PythonTest_PythonTest,pytests))
 
 $(call gb_PythonTest_get_target,pytests) : \
+    $(call gb_PythonTest_get_target,pyuno_pytests_insertremovecells) \
 
 # vim: set noet sw=4 ts=4:
diff --git a/pyuno/PythonTest_pyuno_pytests_insertremovecells.mk b/pyuno/PythonTest_pyuno_pytests_insertremovecells.mk
new file mode 100644
index 000000000000..3b3e3c0988ee
--- /dev/null
+++ b/pyuno/PythonTest_pyuno_pytests_insertremovecells.mk
@@ -0,0 +1,20 @@
+# -*- 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_PythonTest_PythonTest,pyuno_pytests_insertremovecells))
+
+$(eval $(call gb_PythonTest_set_defs,pyuno_pytests_insertremovecells,\
+    TDOC="$(SRCDIR)/pyuno/qa/pytests/testdocuments" \
+))
+
+$(eval $(call gb_PythonTest_add_modules,pyuno_pytests_insertremovecells,$(SRCDIR)/pyuno/qa/pytests,\
+    insertremovecells \
+))
+
+# vim: set noet sw=4 ts=4:
diff --git a/pyuno/qa/pytests/insertremovecells.py b/pyuno/qa/pytests/insertremovecells.py
new file mode 100644
index 000000000000..4ce2707602bc
--- /dev/null
+++ b/pyuno/qa/pytests/insertremovecells.py
@@ -0,0 +1,85 @@
+import unittest
+
+from os import getenv, path
+
+try:
+    from urllib.parse import quote
+except ImportError:
+    from urllib import quote
+
+from org.libreoffice.unotest import pyuno, mkPropertyValue
+
+class InsertRemoveCells(unittest.TestCase):
+
+    @classmethod
+    def setUpClass(cls):
+        cls.xContext = pyuno.getComponentContext()
+        pyuno.experimentalExtraMagic()
+
+
+    # no need for a tearDown(cls) method.
+
+
+    def test_fdo74824_load(self):
+        ctxt = self.xContext
+        assert(ctxt)
+
+        smgr = ctxt.ServiceManager
+        desktop = smgr.createInstanceWithContext('com.sun.star.frame.Desktop', ctxt)
+        loadProps = tuple(mkPropertyValue(k, v) for (k, v) in (
+          ('Hidden', True),
+          ('ReadOnly', False)
+        ))
+        tdoc_dir = getenv('TDOC')
+        url = 'file://' + quote(path.join(tdoc_dir, 'fdo74824.ods'))
+        doc = desktop.loadComponentFromURL(url, "_blank", 0, loadProps)
+
+        sheet = doc.Sheets.Sheet1
+        area = sheet.getCellRangeByName( 'A2:B4' )
+        addr = area.getRangeAddress()
+
+        # 2 = intended to shift cells right, but I don't know where to find
+        # the ENUM to put in it's place.  Corrections welcome.
+        sheet.insertCells( addr, 2 )
+
+        # basically, the insertCells call is the test: it should not crash
+        # LibreOffice.  However, for completeness, we should test the cell
+        # contents as well.
+
+        empty_cells = (
+          (0, 0), (0, 1), (0, 2), (0, 3), (1, 0), (1, 1), (1, 2), (1, 3),
+          (3, 1), (4, 0), (4, 2), (5, 0), (5, 2), (5, 3),
+        )
+        formula_cells = (
+          (2, 0, '=(1+GDR)^-D1', '1.000', 1.0),
+          (4, 1, '=(1+GDR)^-F2', '0.125', 0.125),
+          (4, 3, '=SUM(C1:C2)', '1.000', 1.0),
+        )
+        value_cells = (
+          (2, 2, '2010', 2010.0),
+          (2, 3, '7', 7.0),
+          (3, 0, '0', 0),
+          (3, 2, '2012', 2012.0),
+          (3, 3, '6', 6.0),
+          (5, 1, '1', 1.0),
+        )
+        for pos in empty_cells:
+            cell = sheet.getCellByPosition(*pos)
+            self.assertEqual( 'EMPTY', cell.Type.value )
+        for x, y, f, s, val in formula_cells:
+            cell = sheet.getCellByPosition(x, y)
+            self.assertEqual( 'FORMULA', cell.Type.value )
+            self.assertEqual( f, cell.getFormula() )
+            self.assertEqual( s, cell.String )
+            self.assertEqual( val, cell.Value )
+        for x, y, s, val in value_cells:
+            cell = sheet.getCellByPosition(x, y)
+            self.assertEqual( s, cell.String )
+            self.assertEqual( val, cell.Value )
+
+        doc.close( True )
+
+
+if __name__ == '__main__':
+    unittest.main()
+
diff --git a/pyuno/qa/pytests/testdocuments/fdo74824.ods b/pyuno/qa/pytests/testdocuments/fdo74824.ods
new file mode 100644
index 0000000000000000000000000000000000000000..122bae22d9d45e0f282b4c734ae4f294976ee0da
GIT binary patch
literal 4811
zcmaKwbyQUA-^CA|0ulq#A`USlA>AzvBMj0a7Y0EZhLVP%L<XcJMQZ3)KvLi;h#;Xf
zQc}_gNQm$b`Y!eU^xj$PoORAWdp&1A-}&sl9+)N`J}rO?5Chk%V9HFwkKJtW0YIJ(
z0LTDlz{=SPiEu*lc{@57z)1kSB7YqH7ArvC!<!7i!C%G$fN!^TBU9%IQQ8wocya^?
zPg~Pz{92?rIZ5q98%k{<a&F0>WbRZ36;e)w#A!g2(N#K4R^PGYh-UHHH(qZJz02X{
z)9Vn)3CoKlw@bJm@#kEyxzI-fF&yaV%xarhet(okPw>hmZKt$IK%f1C*2h+%vL5S|
zypqE6&;>Q>)T?mI)MDs@h)w2=m0bPRZkY!w4+u+4D=W&jZiewPt;y3nw##5&Gs)hC
z^D>p9iz;qz!4{hH9?wXw7{t)}wEv|`^2MFbF{xOec%nA5?zIH{<@*)o<&Tp-qiR?U
z#X?Nrj})}3T)~#ov6c~Km5SNN<iiDjr`-uvaj}h4<T5TJNM_85wO_f~6IUTrSYdv_
z%)HT!qh|~hH6vLwV_5{<+0rv!Y;g*E=gHCPJevs`k8RxVY@jwM<PT**@zUf`$YPYf
z9A)f0D!%U0>+}&E{Q|Na;Jf@8+C-;pIonoKvqe5o8Y^D$G6dr)_>MTp3p4)(d^I?Z
zhLi+3#|qt1f7?3U+}c-Luc#_;=qtdk#rrq<!Fe!A0#`E%cf_nvRzT%oYiLa4u9K<3
zz{dg0JeDW4J~~_!)u_Aom%)TmArowbHG6(CW(s7VmOdJXdE=bJIICxFeAuE#v}Tb`
zRZhm>(33Y+RIy(}6&h#~fsRQr#}F-6&$Qc?0?Q;A^Zk8QMv)GQ!aP1g^W7#Lnf2Kz
zsv+5)3?Aeodfh9sc1K#>`HZjEJ1P6Sf{fGe3NlOk)P-@bY!W(w)JEmxW9IH6ACQmE
z@{No*bxri8esNN($RS_7homHhpFgU<=U^U0u0O7RhqI@*;{?#z1ko2}^R7VIlH|*D
zqr*6iSl{)Y2XgYP<v(3ZS=CVZoay>rhhcAo@rojRj@UE&sWY4u?(E-4uzx>IDTRT}
zqjU{*En9|pRL+4Blh!p_rE99yUG}+oRyfwLo(6b75vd*BJDlSk2-q_AlkGnnZ9T$_
zvQgE3^*^lGcH54<yvROzL8&3EkruApR~X3SXhAje`V<f9kZP!wh{Cx?wr;C1TH^>3
zmuRO5IoHP)o1i-_b)Bz{p=9r=uX<&c(EfXtnW@MRwg3=P*Fgjh%}i@nvd26rO2OyT
zHL7^<_f{tC6WQOMwZ0C70ZMxwH!Dtuq7OPPoaCC0@%+-JUuNYR*Wr0ZiZT>-j?)q^
zsHuF=vW{VJR~DwsHBJX)5${}k>dQGY@M7)jRCQR6sLeT+x;$D&$kr$PSJg5a{Mt7;
zH<qXc@7qzw3_R3We%R>RkO>t|S)Gn!uMKdjhjT@cMQ!av;^vL%)tKSz+d|xN*N+wz
znUJ`6<lLyuZYT*NIRo)o#PS^C5F9y|&L3@7xHF>g#v;&noKH1<_hfeCMZuwV#vAAO
zKK|2CaG2Gzy)0z$BC|w4KNweYXNo-nH`qwREa~n6GEYp6wOoX|FXn`|!6h9Tu>Dp=
zt_9l8nD(6I6}mFm%9oi!R3_tqX!(dAhD8-AaQ5SmgeiY}`2QOT7FF6RaD^-CI@kEW
zfl5LINB`{lA5fi1e5c(W6--%i%o@#(0{|*m;?eyo;b`e(XM=G6$-Oh?{egRBa80KU
zcB=PFw4jOZej9^qabx%j$sis~+B5E!d=J&-I(~Gbu0ekB(AwgcDJw!FM~XHv->!KM
z&p8#@Xu|%NQ?sK{vFpBHctuxV@G>Crs8(wR-pS@gPejFES5-m_rvZ6mD=EMiTTQ_F
zYxB*l6uzmLpnN0#G|2Rf^pmG@Y!yRF+b!DRUYW3c`SMwyS?CHI4YECNK2yZ(^*dHy
zA8M$xhla)lEnO)9bO)WdUXIcgev>d)!$X7i%B#P14W=IeBExZif9{oJpHvq)(GC5(
zpvsu8w})&oOhLUJ3Q@7Fta735T8jY(9p{P-bZaE0I4GQ;hxiJbh*AQ|p%~+__?=Cu
z1GW)13j(?f&aSz<W|fpK!<A0M$`jn5c8BD{l*fgnichdRb_p9B<XF3Zxa?+a^KH|t
z?EaW8z42{Mp#oIOn_o$--*^Y#B*k^(?tXTs^|=VRid6Iy-{oU}mJr^=&v6<OyY<7i
zQ$1ZTHr{!)6}T0a@ff#Tfw(U@=JsY=UwS$~Ghq;BKYsjlis+_;z`o%tw6YR~fHqAQ
z>oX4`wQ6Xnc$jcZ6G!`4oVW_1lBZxHokjuCr@R@zXfG+fS8<Cv7x#xt(2ELJa0~j;
zI>pXLR+rr%Fe_^!1gKb{(y=5z1Xa)vp}v(a>St=U-=I%kj9Y#nhieelCN4+ncOE^Q
zcCt`0Kl_2Ix@0XqppJ7)=Tg~4#PZaHw1vZIhxkNzoyxP<q)XgSldLw!THP(bej4ha
z|LLIs*2B>!Lyk$f06;+k0HjzC-H|>H2=^cP{Mz=94rbn%K#vPheWgp3C88PB(8OOe
z&1&$stz(<<0Jjx8SF^OA_&Y9z)DT#&lj%t>^(=V5>^M1WA6p?Ghw)GtS4~+%sFa7{
zPa@X(R`-v2d}TKfZ(+CwSJXQdMkhzT2yGVX+u@<xaULdcYNk89!4fIm@fy5TG)4CY
z3?hXy@NXCG0_ZUipJ>ur6&YWEW-Rk_!N%cw-OIkE3Bnk$t4V%>P-kl$PQ%ebN@Fty
zx<JaYS0)A*-|c^G;2+VeD;!<-<ThGDy}GZTX@g@xgHP@!5PHue7z`zf*5BC~6~8>N
zE!O{P@K*F`qd|U~cqC+`4w06dQnOs)o<3>5^cmeVkMeNKBG{@n=$358UW;xHx?UUm
zB+cSh2R?_c>yTc3E1xw*-kc&0K6m*}Kc0n*VlMIy*1`8qq80;NEZJWowr=W^Nrogu
zxP)9trF(dsY)fIt9?><sW*VSqZ?S9MHyuZK`DrGN-5{G(!5dH3p@fCClft{J_>Lat
zHBtnLT`^@$X~$aGr`6uCSyx+?aA;dCC`n!r`t(TS>P)R#l}z}s2v^hW33!GwxLki+
zBxVV+9cova6x0f6U#M5&pp3pQf_YN+#P@4h)0KVu=Q7CLhqi}~H9kWkgJsVMX}av$
zdzkk-Xhhg5!kx1+CK8-8E6r=&`Z`fFiOs6v5aa8blhRpLnAxlSuCW_0nEN6erSf4)
zp;Dq7Gp}|S_$Qrpf~p)sAf>J)Rgw!DjZDMIAqoitkn@?HK70^+>lc-_Qj-lzRI@?#
zQq|*zhf;9YOU)kwMhQM>7|l!L9~dX5ui-g}1W3?vcR~$&?Ea!KPSpnKsOW@+4Ih@;
zX_nQ1E=DnJm*eUx=1iR4MxO#d2Rj~%bF}1ikr0-}Vc6j&K7#`SY58NUcd;b@1Ab%T
z&D}!TX~XK~`fn2)t6vITEQ6@g%Wr|RRl0~@LR$+5Fha7s4?4WWmvE1_Y(0<>Z+Pr^
zm1#ma1O`RB&3JcxjwAP%VeY=N5R%fWYUj3tdIJ8O8DVs<Dcy=xN2yD<;3X569!(qP
zTOcyu{INtK2xVE1S6KPfG{egd0Zygl(Txy0<7Dq~^Zsk)lVzK`$|BUE;eu_JX|SeB
ztB%uT#<3BN;c*e$zn2E}%{0`|OM8kI4VVE_JbclP?CdR$76`)>Y?t|Tp+*16)P_;+
zqW*=k;H}`u&<iVx{nB7Nt!OKa3u9FoU_?uwngUt(c6g{*yrc27@gs2KKx&f8%k7Zf
zqSYNqwl{7!U)c^ei`zx!ug6d(4L~*e?+$e7*A(!_ofy6l=?o0-Fl8$`!TD+W9M<$m
zc6ZGPERBhozZ2OVfkfIl-Tujw2+=<pzn(hbhUJNTVA!^l;6t#Tw0w9!CR;n59Bh{$
zX376Js&r57YPg?2;=T}S1{Iy8Q{aya{kT?bvaaHkp@VlLEhvuG=WQ`7yp-C7(!X0(
zdleIS^Tal@mQkEpkCK_KnHwnYrcN9bvR1EH98`};zrk4_s;u|Xg*phCxO9tH+H?`x
z=ebxt$fbYnQ0Xlrgif-SrE^=sOio^~x`d01>$9J5`pQMN!2wLp;MnUNs%>>xHM@Zg
zU+Ik^zsk4MH|(<K;}Lg;6IiNSr|&@V^Vkr@>}n4BCek`^dlQp9p=>HS<6e`xjl(ok
z-j6{FrocxlTSVv`(27EL(onJR+QYcA=MM$oHeGT>6)#u&BO->5rgq65=Wd(ymUJe4
z(B57MLLs{SQ$kvKN$2)_g)MTMHG4#_Gt8B@=H$Z_rGkbuCbJuoFeFHoKG^|6AI*U!
z%}K$aVyBtFOf@yy5$>YFjnfi&F&~{4NSt4Ah%AR8>e<EI6{(}jJ12toWSE&zIgfab
z?tM-Tz9;?o^%2udos0^XK*2sS+d@h~EtB-yx|zpGmcBk05xP!QZBn_9vh^5fiCa|3
zkJ9uCRk`|2+c}JwiXn~TLHF6C$|?q9JyDHT_gje@7;A|l$%*SsG?XhJIkY~yvNqCw
zgi*C;-EzOG!8o>0A@fN7C86@X&DQvKApOL|3u>I(4e>tpptwX_pAAt?t@e$-HWX&e
z0-rU@Ik%q%chlajQ#t@iWP8SC8z~5Bp<z!w;T1y^v81DhcsHK0>y@w)G(~I^P+5q9
zuc3PIqR_&G%jQ0GJnwsgrN{`o^2oYTR1Xs>V`Vt#i1D}}1YQd{mjuDtM0%TEPKW02
zC8#UrHnnEn+s?ctNqV+IkKYQRno8nnk;xy-bajTty;b#Yy>n{-U7p3R;21C+CmV^c
zF6x~g9ihn?pc_Fy?_k4m<pIeg#*5dmsRNcKE@ysd=X!1)9o046+N|t{*ESJP9l+wM
zt75G!WPp0?$xuBOgTEcct>^b-%@*&x?iW9zP*{+bC}xXy9lRSA;+{<=PppV(r9V|2
zf6z{0dDEKkRDvNWu-xAI(9kB)OXFU6$Vub7bB-^OI0flEdvruB?J~DskO^jt^(H$%
zOq9!0l{o!Wa#A3)DR&){zG2maw76z3)}c~1>*<xHBU5+diA7Uc_0H6XrvZ3B|IBA?
zHq1fd9u!-z_^{VEg4$sj>f?g=R;~V}z!J0tlK||*_|Ib;zylE|J$3t&4RQBW7t;gp
za$2(+v?L>OjkDTGM5Q9ZCooMMTvoh)J1b!LR>iRZ2l9f?9{+K2K=A$K;I}e<CiriS
z{<|OmPyy=-CD?-gOTWj;{juBsT;K2Q{@3+w{3`dOg?>BZz)JriXny_l-<#~O(&mJ~
z|MXg{;#sf#<F|h|^Y>o&t70}0@Q;><)jMl>|DpGLoc^kJ7i;*>aK);fh3kK){hkxQ
zs%?^CSND(9z$%`l#(yaOKCHhgHc?{N_hWRiYJc1h0<6zxV@?c2V+#W>)!Ezs0SQ&)
A>i_@%

literal 0
HcmV?d00001

-- 
2.18.1