Commit graph

151 commits

Author SHA1 Message Date
Tamás Zolnai
79073f6ce1 cypress: mobile: add first insertion related tests for calc.
Change-Id: I57561ef5687119cd53d99ea46e9ca3a269548422
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90237
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-10 11:18:04 +01:00
Tamás Zolnai
90c5717466 cypress: mobile: avoid hard coded coordinates for calc focus tests.
Change-Id: I9da74fcd090371cbea5b3a8c8836bf9236f709b3
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90224
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-09 15:39:40 +01:00
Tamás Zolnai
9396cedd28 cypress: mobile: move writer related helper functions to a separate file.
Change-Id: I02dee0270d3b572ae26b3f871fa4c41c0c397e54
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90222
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-09 14:56:56 +01:00
Tamás Zolnai
b1b566c79f cypress: mobile: extend test of comment insertion.
Change-Id: Idb8539a353e884206b7e7b4885b60fa706f81d10
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90111
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-06 15:44:20 +01:00
Tamás Zolnai
8164c2cbc1 cypress: mobile: extend test of hyperlink insertion.
Change-Id: I433b819d5e76c56018fa15da539c48b1ceaeb022
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90110
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-06 15:31:03 +01:00
Tamás Zolnai
68924966c0 cypress: mobile: merge styles tests to font tests.
There is not much more we can test about styles,
since now we have only an apply style button.

Change-Id: I92a1a6dc60701743af44928966475cf4471b317b
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90108
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-06 15:19:34 +01:00
Tamás Zolnai
a0ce834641 cypress: mobile: fix random failure related to long press.
Change-Id: I61698639c0461c0acc6b2c97641d732b036a9d1b
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/90061
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-05 21:15:23 +01:00
Tamás Zolnai
6114b8c8ce cypress: mobile: enable some mobile wizard's state related checks.
Change-Id: I0bf19450f14e8e8d99e1a103558499d53f21f760
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89995
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-04 21:10:13 +01:00
Tamás Zolnai
883b28fbd5 cypress: mobile: re-enable some skipped tests.
Change-Id: I4163fdba2bc0fb7bcdc4d2b33f5deff6b0857013
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89960
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-04 13:58:33 +01:00
Tamás Zolnai
d9a23d9668 parallel-cypress: improve console output of parallel test run.
Change-Id: I29440bafdba101c3f4e86de7d587330b972289de
2020-03-03 16:00:30 +01:00
Tamás Zolnai
033e353e9d parallel-cypress: restore original sequential build.
Running all tests in one command will display a better output
and also might be faster.

Change-Id: Ib061fad28eeb1246c774bfb7fd958c3f5b9c317b
2020-03-03 12:41:54 +01:00
Yunusemre Şentürk
e18b7ca425 Wait until Xvfb says it is ready
Change-Id: I3af4428802d6e0ee380536e8dbad1de8aa6d9337
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89727
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-03-03 12:41:54 +01:00
Tamás Zolnai
2cfd990e75 parallel-cypress: Start own Xvfb in CI environment.
It's not necessary when we build with one job, but when
there are more jobs they try to start their own Xvfb
and they might use the same DISPLAY number for that
which leads to a failure or a freeze.
See also:
https://docs.cypress.io/guides/guides/continuous-integration.html#Xvfb

Change-Id: I2fb301b1d8e889b2898d6792fb38cece303129a6
2020-03-03 12:41:54 +01:00
Tamás Zolnai
eb3b3e54d4 parallel-cypress: during parallel build don't write out execution line-by-line
Because in this case, the output of the different threads will be
mixed in the command line.
For sequential build keep the original behavior.

Change-Id: Ibf9bf7c6111f8b5b0cf188fab66b97dedd613c65
2020-03-03 12:41:54 +01:00
Tamás Zolnai
be9277d700 parallel-cypress: initial parallelization
- Run all spec files separately so we can run them
in parallel.

Change-Id: I521feeac6e1ac4c3d2c87e989c11d4231945e568
2020-03-03 12:41:46 +01:00
Tamás Zolnai
bd83d6f0a6 cypress: mobile: this is not needed anymore.
After this commit:
b77cd954e0

Change-Id: I90e9a6eebb774b7c9ad1780fbfab0528772142a3
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89599
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-26 22:32:16 +01:00
Tamás Zolnai
f2a6c287d4 cypress: mobile: enable writing direction related test.
Change-Id: Ia48d716d2ac2eaba0a3fbf1f2a8e4f9c084e8505
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89600
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-26 22:32:02 +01:00
Tamás Zolnai
892e29377a cypress: mobile: enable spellcheck related tests.
Change-Id: I198c4abb43426cb6a6d4e8e9b43f1c7ed83f1d81
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89595
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-26 22:01:03 +01:00
Pedro Pinto Silva
f401f1f714 Hamburger menu: adding ids (so it uses the respective icon) and adjusting cypress file
Change-Id: I33ad1671cc859ea1af1350f3e3d3a83b33c355c5
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89532
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-26 12:41:57 +01:00
Pedro Pinto Silva
f3b8455aaa Mobile: Insert: Add missing icons, add missing ids, create missing icons, also add sub level icns for footerandheader and formattingmark (fixing cypress)
Change-Id: Id3ca33095b5eae7d13129f4b9629e2c0faf302fb
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89371
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-25 12:06:31 +01:00
Tamás Zolnai
907a23bf0c cypress: mobile: improve shape geometry tests.
Change-Id: I46b2384da6c406c66e3f5d19838df1e87804b2a9
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89390
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-25 00:37:42 +01:00
Tamás Zolnai
b77cd954e0 cypress: use different test files for different test suits.
Change-Id: I29ad09ce99128fe6700070e23cf66dfc0c553b70
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89389
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-25 00:33:32 +01:00
Tamás Zolnai
ca2fe5fa91 cypress: mobile: disable an unstable focus related tests.
It's unstable now, how focus behaves with shape.

Change-Id: If429420852599315bff5d71f33f263f1d79a356c
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89380
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 22:14:25 +01:00
Tamás Zolnai
4f864162e4 cypress: mobile: update table related tests.
Change-Id: Idd65edc8d3c007afec9e93e0a103379c579a178c
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89379
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 22:10:38 +01:00
Tamás Zolnai
5ff34b20c5 cypress: let's make the pid file under workdir.
Change-Id: Ib927b7d4c29c623de8cd4bf083843a7a6a7c9c2f
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89374
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 19:43:23 +01:00
Tamás Zolnai
a7da409624 cypress: make sure that npm packages are not installed more times.
When we build with more jobs.

Change-Id: I1051825ec8db4ccc7e10bd4a599480a97334538d
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89373
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 19:40:24 +01:00
Tamás Zolnai
131f4c340c cypress: mobile: this test has an unexpected side-effect on other tests.
No idea why is that. Maybe cypress has an issue in it somewhere
or loolwsd is doing something. Adding a small wait avoid the
failure.

Change-Id: Ic1b29c34f9990fc4937105925872655a98793f79
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89372
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 19:35:55 +01:00
Tamás Zolnai
55b4fb9ec7 cypress: mobile: simplify this test.
Change-Id: Iaf076f35eeabd7609da024a4f3236a52e69bb543
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89362
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 17:16:26 +01:00
Tamás Zolnai
48bca31c9e cypress: mobile: make selectAllMobile() more stable.
Change-Id: Id8b919e3e0a18b28746541ee2a65e52f85458fa9
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89353
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 16:05:13 +01:00
Tamás Zolnai
2a6dd836c3 cypress: mobile: update shape related tests.
After:
11b7270acd

Default zoom changed -> shape is insert to different
position -> geometry changed.

Change-Id: Ibeed3a57b3b64962951f11054f6aa3410a0e673d
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89345
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 15:00:56 +01:00
Tamás Zolnai
a1eaa97954 Revert "cypress: mobile: split this test case."
This reverts commit c692487acc.

Pushed accidentally.
2020-02-24 11:54:47 +01:00
Tamás Zolnai
c692487acc cypress: mobile: split this test case.
So it might be more stable.

Change-Id: Ie8b8a13ebfe9b681a8e9a456f24f6322ce9cc8a6
2020-02-24 11:36:43 +01:00
Tamás Zolnai
9484ec73d1 cypress: disable an unstable check here.
Focus is not consistent around double taping.
Enable this check again when something is improved here.

Change-Id: I448d57a9fd053d1680352b6d6175f38f3dda33d9
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89331
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-24 11:16:27 +01:00
Tamás Zolnai
75451d9fb6 cypress: use parentheses consistently in the make file.
Change-Id: I3d679fe99749222a6ab56c952d5d3fa09a6dd3d7
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89238
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-21 20:25:43 +01:00
Tamás Zolnai
3f0afa9cfa cypress: better handling of npm package dependencies
For incremental builds, it's not enough to check the node_modules
folder, we also need to check whether all used binaries are in
place.

Change-Id: I9c5f380f3845195bfa2dbfb03ab269ce4659c4ae
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89233
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-21 19:47:22 +01:00
Tamás Zolnai
fdcdcedbfa cypress: don't wait on loolwsd server forever.
Use one minutes timeout.

Change-Id: Ia4e351088d7bd152b4c7f47c66dce2a88f14970c
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89232
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-21 19:43:27 +01:00
Tamás Zolnai
f6a5b90612 cypress: remove obsolete comment.
Change-Id: I9d8293ed27cb756325ffde48f3bcf37f390a1651
2020-02-20 17:10:24 +01:00
Tamás Zolnai
389720d948 cypress: mobile: make mobile wizard navigation more stable.
When we open a new panel in the mobile wizard the new
content appears after a short animation.
I suspect that this animation makes tests to fail sometimes
with 'detached from DOM' error. Waiting the end of the animation
seems help on this issue.

Change-Id: Ie9cc322ff9bd9a6287b4d57126e6afab91a98713
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89107
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 17:08:05 +01:00
Tamás Zolnai
7810825e6b cypress: mobile: make paragraph panel related tests more stable.
Change-Id: I6d6d9e8f2f6d14406327ae074f1e82e411f508cc
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89091
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 13:41:29 +01:00
Tamás Zolnai
c211ce95e2 cypress: mobile: enable some shape related test which are stable now.
First issue was rendering the mobile wizard twice by
opening. I added a workaround for this issue:
20b5825ce5

The second issue was with the animation when moving
one level down in the mobile wizard. Checking the next
level content makes these tests more stable.

Change-Id: I3f3c90e5975747f833e9560140cabb6c310dfe57
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89087
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 13:14:38 +01:00
Tamás Zolnai
be9333bf5b cypress: try harder to kill loolwsd server.
pkill uses SIGTERM signal by default, which is good because
using that makes loolwsd to clean up things before shut down.
For example it removes the pid file.
However, when loolwsd stuck for any reason, then SIGTERM might
not be effective. In this case use SIGKILL.

Change-Id: I3ed4bd24d6e73169b4d04dd7c9b4ac03bee8fdd1
2020-02-20 13:13:52 +01:00
Tamás Zolnai
bde5059f19 cypress: mobile: update test title.
Change-Id: I01751b3f3f25391fd6d846e133dc4fc3e5ff84df
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89076
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 12:00:18 +01:00
Tamás Zolnai
9eaecc1b06 cypress: mobile: Fix-up hidden style items test.
Without waiting for the mobile wizard to be opened
the not exist check will allways pass no matter
whether the items are actually hidden or not.
First we need an indicator that the mobile wizard
is opened and then we can check whether the removed
styles items are not there.

Change-Id: I00c1aaf6e11351cf8714be3ed6d5b9d2ff000324
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89074
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 10:19:38 +01:00
Tamás Zolnai
eb4f5abf52 cypress: kill only our own loolwsd process.
Change-Id: I9c6f71571183c7583bd3c4c122c3c3e2c04696ea
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89070
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 05:45:25 +01:00
Tamás Zolnai
39150fca06 cypress: better way to generate allowed port numbers.
Change-Id: I1900df07f1fe76d39a35ab8cab0d62c0c53a35f9
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89069
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 05:44:57 +01:00
Michael Meeks
c07f45e4e5 wizard: hide styleupdate as well as stylenew.
Also hide calc style panel for now: incomplete.

Change-Id: I46fee97f5d029860a12e5f7215d830091c5f9bf0
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89068
Tested-by: Michael Meeks <michael.meeks@collabora.com>
Reviewed-by: Michael Meeks <michael.meeks@collabora.com>
2020-02-20 05:21:51 +01:00
Tamás Zolnai
16578f0353 cypress: mobile: Disable spellchecking tests for now.
There is no an obvious reason for this issue and also
I can't reproduce this issue locally.

Change-Id: I6860fa878ebafd9deab92b2662c352a99ab85b52
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89071
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 05:02:22 +01:00
Tamás Zolnai
eb53dd5a8c cypress: Always search for an available port number.
Change-Id: Ibfb80d43cd7f7d4c8cab8e3621bebfc2201d4b2d
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/89066
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-20 04:49:38 +01:00
Tamás Zolnai
0a7e3bf30f cypress: mobile: Test whether the line width spin button is readonly.
Change-Id: I628594902b87dbc904628b8815c520fdb01478c6
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/88976
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-19 11:05:05 +01:00
Tamás Zolnai
2e9f59280c cypress: mobile: Test that the arrow style item are hidden.
Change-Id: I958357acb912176070e45ac331b38d42ba24cd15
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/88975
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-02-19 11:03:22 +01:00