From c0919c6e5787dfb45f1a3e4b8b5ad6f1ff28f1ae Mon Sep 17 00:00:00 2001 From: Xisco Fauli Date: Wed, 6 Apr 2022 10:57:03 +0200 Subject: [PATCH] tdf#144996: sc: Add unittest Change-Id: I8e6d9f420e9b487089cefaa1e7bf5a6082d6ce52 Reviewed-on: https://gerrit.libreoffice.org/c/core/+/132622 Tested-by: Jenkins Reviewed-by: Xisco Fauli --- sc/qa/uitest/calc_tests9/tdf144996.py | 48 ++++++++++++++++++++++++++ sc/qa/uitest/data/tdf144996.xlsx | Bin 0 -> 9408 bytes 2 files changed, 48 insertions(+) create mode 100644 sc/qa/uitest/calc_tests9/tdf144996.py create mode 100644 sc/qa/uitest/data/tdf144996.xlsx diff --git a/sc/qa/uitest/calc_tests9/tdf144996.py b/sc/qa/uitest/calc_tests9/tdf144996.py new file mode 100644 index 000000000000..970f9a0bcba3 --- /dev/null +++ b/sc/qa/uitest/calc_tests9/tdf144996.py @@ -0,0 +1,48 @@ +# -*- tab-width: 4; indent-tabs-mode: nil; py-indent-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/. +# +from uitest.framework import UITestCase +from uitest.uihelper.common import get_url_for_data_file +from libreoffice.uno.propertyvalue import mkPropertyValues +from org.libreoffice.unotest import systemPathToFileUrl + +class tdf144996(UITestCase): + + def test_tdf144996(self): + + # get_url_for_data_file calls makeCopyFromTDOC + # so using save doesn't affect the original file + xFilePath = get_url_for_data_file("tdf144996.xlsx") + + with self.ui_test.load_file(xFilePath) as document: + + self.assertTrue(document.isReadonly()) + + with self.ui_test.execute_dialog_through_command(".uno:EditDoc") as xDialog: + xPassword = xDialog.getChild("newpassEntry") + xPassword.executeAction("TYPE", mkPropertyValues({"TEXT": "password"})) + + self.assertFalse(document.isReadonly()) + + with self.ui_test.execute_dialog_through_command(".uno:Save", close_button="save"): + # Confirm file format popup + pass + + with self.ui_test.load_file(xFilePath) as document: + + self.assertTrue(document.isReadonly()) + + # Without the fix in place, this test would have failed here because the saved document + # doesn't need a password to edit it + with self.ui_test.execute_dialog_through_command(".uno:EditDoc") as xDialog: + xPassword = xDialog.getChild("newpassEntry") + xPassword.executeAction("TYPE", mkPropertyValues({"TEXT": "password"})) + + self.assertFalse(document.isReadonly()) + +# vim: set shiftwidth=4 softtabstop=4 expandtab: diff --git a/sc/qa/uitest/data/tdf144996.xlsx b/sc/qa/uitest/data/tdf144996.xlsx new file mode 100644 index 0000000000000000000000000000000000000000..aedf252355063fa424a1dfea936fcb4575dd4c92 GIT binary patch literal 9408 zcmeHtgzUbmJ!?N}uYHu{;NXD(L;x}X0H6Yh@c55d!vFyA2mk;c02x+W z%*NW$*xFG~&CS->L6_Cl%91h%9+n{s01JKof5(6E2$ZPxS$D8uw`gn$i?_ZA%MK|j zLMCk_)u!1*dgmIaNmI8*#Bk$9Z=$STiQG=BY&oJ$cPvO)Iz#tT^!bV<>(xs|8hy0) zk)`&B&u*nNCH(VBUPM;MgmYyPV(jlDeyK&0ub+Xinm@jOOpDJ|gUY$legyt5S{y1Y zM%%);UIcqNwf(HG<0Y)z6DL9luWNt`3$g5iMsu?=2j>f#@>B;hb=HLQ$S1%`rG}a- z3Yo3`abT8ir701=7{zn55||HG92DrB!6?|3?7$<+*S?4#>iNe0dJfdi8COsJZ7IUW z^p&zYC%qd1|FM_|^(@7&RW1$YtI=r3$qvUmPkZxA!&v&YjCa82D0i-e#-67(%z4KI zej}Td8SF8KZY_bx4#C)F^MHlMU7QnAhf>&0l*5x3JOl>>*9n+W6TH09X)}PgE(Qd# zW&Ee;@P6n!V)mv}XAz{A#j}|6={X`4efq z4O2aPV@n4R>yPvQ!t=j4CjZv;k~js$PB!%5ed(K!-t({XG1#JVE+W!RRBGP7GILn9 zk-0P^^KJB`*lNT9a8f=k-Z%ZT^Ma9Ey;Nt*PfNmafdbTZE~UY5uk4(V80qYjB<)I; zx^SGoo_;+`mXdX2aBhiaEUy2ME!Ve5E&1h8yb^naRf8A>FP}6R|5>u1R=2{-1;|Ai z%vVv>-O}KSk33mh@gu1|lSu{NF@!_;U^#^Q&Lm8j2J&+=k>>FM8#j9O$%Pt1#^ zbu?0e-CcUI6FHzIYUmqUwf%D zYIl(IZ)Lv$qpMRnb=zx|2X9eSs~xN7U?c%`R5u?}Xj7M}a zTQI4FlLyA*qGc@t9lwx{742}@K3d;)&NVKX0d)B%sgd$4sTWr1Hy080gyW7cP)kia zTr?htA)>F_(c89673C1U#?XF)=ljiL9CRcV95q^vyQBB*4=3rm;l8APa-oXlJ(8CV zSekp(gdBIA#IR;gS}0krHsTKC^~qb)-{X>u7UVSAQ-`E<`oM`rD$9~Ie2-`-ipb@s zqXan*obl>v;LFXZzNns(TWINHHfqcSACOy~9V;p`RA>5pg63CTv?1JgXWF${*LL2Ta~-QWtipO1FIq5cy7bvwPBCiagm#6)I}!Bg z1uXhneb}+$x3Nb?I6I^1$3k3`hgx+#kLV^dLNq$5NJVwtTq~frcC5^ToQU*Zd9o#I zNA4aE2$_m)ht}5m=34ef%kR3X(ChfULFu4Rv$k{gB~6fQ*Q^#D3_FhBabU%Da zoN>um0*QhR_rgcF02y1JUP5aLyG2`|pCk@y%kWV*zP7?d0 z5x3fY$<%2F8zRT|-ie(lVpDI_e8M2_xOcAt({Q#3zbi60O;RguXUNXWTosj061sn# zZWJv>#abozOruVY_tr^T;s_8LkRsrH9?TbmV+3L)8GDwPq?(0fDV4!q!9`t3I&_9q zus)kqDB)~L7xlJFX0!LMcl+6eFe{%Dp1nW4`QRVTVMIOC#Nc_Zm%;Tg%BLtWi|>uZ zday9i7-pWxa*<%5v!Ls0J1kGw3j(FW;)UW8zk?QCKVP_iOwk{+zbJg61yK?1f+NwDmfIlhwWWQsSO)+bK#siJw zf5*9>P{P||XiFnh42c2AFwi*vQxE?h?f=n67-;GWoh|;mj}jFHxh`nxhISXi=923C z2xr<6^ms>o8wnT@iOJf1gV^jDVy3?v#n>?sii{wJ8(?g6 zf{N*)UE&P+sQJc3!EM;0J@*QIjkH*fyol?WXRsq>NUs!y&yzTZwVdQ8_5CGB(d+&v zZL>$r11I|?t9?CR7ond1H)Yy$Cq|8+bXkN400^N^{3Q(zrpCsO4xpbG_8%INmY{1h z!v+lAr{5LUY^9Gv8O6n6WO(x}T;@w-w8csqcCpOP^m=aU*`mu1P^+-%10xK76xN8x z@ZmPIHCR^;Q$Ky&OT;Zs1-4}(amxka_O6svS9Q60lWA;=9Q6u0YJpkUTa&cb%NE0z zUV2kkgk)b~_+31PV+ZZb^JszEy7Ytk8iDl)1m(}dPAnwvzehNUo;p&loBh^)8nWqeF9dY2;rlExiJr-*pYY7b_nDsCfaes~hmZ{iyu^kfQ8WOES)A$ZMRMg$cSKD;f zzpQ+8sVxUFnBfyEDrluziwNWPa~!I9rOmB|*A2Y+q-#PtB(eA+)=cm3e+)9GyMrMiWqmBXGgI z13rzaQR;EFOLKlFm7>2>iy{`PYZjfJrh(Cl20)X+bCfEr6$aUA`gve^+l1mE?zCJ* zYS@Cq>QF<7K^`KJbqlC#${?)aDi`&q$W6^DN%y~H=JUR4TXiG=pqui?0{ka49Zij` zj6pxoKS{asQahZ42j~qt5k^1WyWn0Kz@@)Pv8rl9c}Y(jt)+d&P)tN5vk-~m7bkd{ zuOuy}hw8q$XFOTqy1GwFmhN!!eQ@kD+y$~l zX26Opr{#UA(|2ym0Wpk|*S?Ef${Ia*My1RlnLt~=2C6s!y}G!_JvNn-`h=Gg_gI> z{pDx0jc`*Yyd3+05-OfipJW|6HldPDzRMGRE;H;vku z+htX?Ez?hBd3q}l8E&^m=ZnvajL~d1%gd(cFu(S6SSS?K^0-O=bZp%9c98 z)nBSlEtKArmo9DRJA*!GUvV+y(D3Pqz3tGdZPM9AiR?HA=xxe^+tCuO{dT??#GCil2Nc%A| zuJFnE@-^uG?u>K4T(8yR@SxanE8f*$IrIMH@_5nPC)r)oZtu-&Jz@8|i?M2(){Enw z5X@bxC`|c_m@Z9MkBh0MVTfi{r?=;)MR^Dz6+|Wr*-=34+}H@_96z+tmdvv0fDS3i z4hA%2qg}Tr<;famxXz?(3FAVPOV*u^S)a4cAt;zeuX?`fX}w%V?YNrK6pqXE>Gl$R zGqGOHnc6bhEiy+u=LxvtWWr_#+puTL8V7Re^frPQ&+>7LcGouLvSVKisb*LLnpHgE za+NI+a(10(wa1rjwk<6uEQ)FADg03W2nvXK!y#^p9I#0ZQ(0aiWS^3#A!1JNAU13p z4J9@M(%*bXh_)2h&~MNpIa&_rtnX}X zNh`1F8jd3@8lPb*k6WvZgUf6Ye%gm4o^-dQE-BNv>^R zgIHi<)wOFs_f&NguZSR8LRYBGcr%jnM->7vAARDh{9G}-da10N08cyqwITqSr2RRM zSqnbZ_KwO(M|wG$_2+%3Gs2yLS(`?X2x40$I`*~2wMpm)E|Cm2F?fXFaBGn+R3Fxy zVRaU6mwtsEh13ujRO;e|FqxzNrl8Su({yJC{n{N=EB~tXxpBOUo8t)m!qibR zw(R+u|1w#2lMw8JX*V3<#4f{pyiP6PHIC;Hu{^$Ye{9krZ+s>$(XNy!Vrh1jwLVy) zz@4#~2yc{qyt6U!jpF7vulH`v1Knott94%J4ow8X_IWk~%eu4WXjL6cGhFsxY&i)# z7B!W~!6odZ5*o;xkSIG5D(hj20$-T~zM3x79COqhQ`&l)ttDxtBEcr42^)rw-q$2IN0gxs7fQOva~3AUC_zA+l+li=(~Wj+=1yisEs}XG)De!= zk!<5^{sc?R0z*mHX5Ed+Arl;m4Uh&Bd(N%Ma~una$Mp4nbabktT>{pZwdTkl8L6LI zT;gpKc22q?Hc(G!6XdAYI}^!dSYLcJNFpMr3K37>*NF%BBeKW7^GN7o(PQyR#CSSJ znq$SJ;fpP#xsvjni6X@g+fks`m7|^7EO?$!IRi}Lx1?HTdsp;%k-UUvnzW;ZA~L!K zj(=kkCr?uo#Do+T<{KVhzcs|}ms;^CcJ0z-S-;5~gaA)a?^BNRi`yCL?)?XKVX55-^jQ@Tuu!>Q9BsNjSm~VXx zX0b1`U~Ry}{pu#$ygcC*3D?fXk~zqs@Uj`)`KG#BdA3HC85v!1g8RbCNs{;3VI942 zZ~CB?({tu++4PI6{<-F_CP*Ls1l?_Yc;BmRDs`lDNhzJ=^Ta%{`h?#=T}_PB>gE8} zx|>&-!fZ^e&}QK3@TU5Jm@i6gse!i3R+4mW&@i#Bw%6YNIJcg(VDc2Mz9TDqXZ%%k zx$a!gfOd)lH9kRd>pPHn=jms!hUNPD^WZI$g>&SX;dw3N(GqTfTG8qma-jdK_sXWu z7U`np-%8IBTdb!{G50PPViA#c&3QhYk_2fr%(B4O#&NWbF;4`@=95j#MtYC7zD;DW zI7$%W(DU`E)g`$5ZRvp1tj@9ynjgMK1OPDpRAC24H%nuOA4y?e;=DC9DeQqJg=EZ& z@4>JJzHw@~PgU}*QzpPOr81-yTUbUU_}se&XUrazWb`op{8p+S0lRqDE_WN9bS>6- z&pk|-CmB1()I)Jp^ULbBUSAv=;>fdCMHRh4E!7Oh8eHDK*-f2j4A93S=_Z8@tRhYHo>xvISt93>Fh-e zBD0LpP_{(?X=x%r5Su|X8AHfa#`V#XU}>#Aexmz(QXpR76>u;!;jRevzK^6y*Uh{w z``IJ?Ft)xgDrigIto(p9I&5%Xpz8t8vYc1D zwRxU%Rm@&lKieEjX5b1C_WOIXa&6bUm~WMOkrE1CeMrmC&w@7NJyty^J1Le>@2{R@ zrMn|)?bh4s+={S2FJ4az7=8Qc@_Nc-V--itaFD*(vSV~pC&5=6Rieio4nZ22O$V`^v-*%p~pHT^=SrL&l^IuxU`|o6*y%a=Qq*o>c=CYx7zc!Zijtyc75;~Jz8W;PisVD^)KI2_EzGq-eASv zLn-|41<;)A^j zW^u(VB;@#SDDwPG`T7Db70#+Py?a)q% zLR+c-?A(u1z+bDoKWhWO)_6ZF1A)=)Z#vk366bJtVy+(X--6J^RCylPt8c(Tu4WKt z-^QE+Z#}9>jX$le*^aDnr(O#dl9*t|GEX$Z2CW(rm{uHk%&@eHRPt`k0Z~{4V+_8q zwx+;=!LngJF`AFiDua}1ZV5ZRyQV9nPzqy&)~98{+}Qdq!%0R_&Swr-*(Alcks^?*dmu}kp z5@Y@14e&prNTI?}!wHMQ)F8bSpibz7%}e$r?7GK--+Tf6ncol% zh1~gzFw_sq(Ag9RTAVSmF;ud*v2_3$+SnWawVn9CnhbR4BI1-S0@#2-i%8d4aW0W# zbxa~Gmbr>d=gwWH-K*>bdsfIfMvV zuGEaRBV!&7m~R;>qMo#GNs2^)^~Z|p+Xm_jn%NJ`i7~p>_2Cp%lI+uS0~}HymX|Oz z)v+!Npgm&h)wb0%mPr`1TT5E-0h+a!Vx*XX}(x&)7Y^{e`(fHn&9v`Hk9k&n4)1 z*AU-X$j2_3En#VJZgbytH=%Z|^lkIF`=8q(A%}iK$`YFkuTsd&7~o=zvUC) z3oMZ1R}^lF)!1)(E+&?NbRSaByn@7^s>+dy4f3TrKc#2z{_PClHUVr2fwtg4%m3Ja zXPtqq?f*#kBU1l7(&9vHX4%k#_TjE5(C10`td!CHN+8l2ClwHwxU=Ct$H(GH@L}w6 zw^w7_Dg5)`rdUrb#Y~365X0>^n5uQ{8W15S3BQHFnJX*WS3HW7dJ?fHo*1mIC0!`h zuA>@y6O2KjVa)6P)ShKsQ4FkkgC-#?t3(4CiFA4Rp|drK*=8l}J^Gj|?r}WEZlFWy zO^cH96+Weiu<)omo*K-8jRBifRU`e<*3X+IFw5Fo1Qsg2+1bn2V_B;9jEn`^xqLKo z^tz0ER$pg)J86-&5p^VKg%P)ivh`+AE&H{cYBooeq$TRw+^l$VE)a-3(qtWDm|Y>CJ147B7&K? zSm@OIfp&rp{X9C!idP9O=vMGPkR_el3hzw46*@uKrE7jrhmQo`bDI9Uc@O#D%>SCSmE;hh90UN+p&vRZjRUEE-2ERJ_W70o literal 0 HcmV?d00001