Enable unit test related to Spin button controls changes
Change-Id: Ib2f0098d5d0b9f8b6eae28a4cce8680c6378e623
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/94124
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Henry Castro <hcastro@collabora.com>
So when unit test fails, we can inspect the response
from server some UNO command states
Change-Id: Ic652918a8e57cadfd1cd58a89d7a1c9905abcfc9
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/94081
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Henry Castro <hcastro@collabora.com>
It's not an actual regression, but using clickFormulaBar()
method to move the cursor to a specific position is not
stable it seems.
Change-Id: I9abd470fd1f0f753181f3550df3b41384fa166dc
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/93258
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
With the recent mobile detection related changes having
mobile and dekstop tests running in parallel make the build
fail.
A related commit:
255d24ba8f
Change-Id: I69929b5dbd37298e27409d247a651c79f6dbe33f
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/93222
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
So the build will update the plugin package after every change.
Change-Id: I687def1d29fe3acdd364b61c69ac1a7d3cf7cb84
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/93115
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Since cypress retries only the last command, it's better to use
the compact cy.contains(selector, content) version, instead of
cy.get(selector).contains(content) call.
Change-Id: Ie1f4c17bbf736058ecf6bd996b46384fdff19446
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/93081
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
The helper functions were improved in the meantime.
Change-Id: Ie8ae8caf5b33567681010a7a0cfc45d2da061f8b
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92973
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
New tests to exercise the Formula-Bar in
both desktop and mobile modes.
Change-Id: Ibc174232f7b614132913c1d66af0051b51151a29
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/91537
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
We can use should method everywhere with a callback function
parameter. When there is no a specific item, which this should
is connected to we can use 'body' or any other existing item.
Change-Id: I3e1c5ca930b28304e886971ee50201d93a086b45
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92931
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
It's better to use should, when we need an indicator.
Change-Id: I0f08c8862411e3c7627302bbda7b632b0bc0e15c
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92930
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
It's better to use should, when we need an indicator.
Change-Id: I5d1fe825fdd6b20f98ce179c7cb6260ec0966928
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92918
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
We don't actually need cy.waitUntil() extension here,
becasue we can do that same check with should()'s special
usage.
Change-Id: I2b14afe84d2fa70f493aae873e3f1534540e96ea
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92917
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
This parserOptions was added for cypress test so
move that into the tests' own config:
b2ec95dcfd
Change-Id: I3c565d212409cc01621eca97cd59ae7869d852cc
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92857
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
So we can use cypress calls to wait on the tested state
using cypress' retry functionality. It's also useful to
have the acceptInput state in the DOM, so we can check
it in the interactive test runner.
Change-Id: I17cd8f88b270f7a329d35e77dec4eedd7aa1c2a3
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92682
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Plus rename one test file: focus.odp -> slide_operations.odp.
Change-Id: I278fc22ec55bcfbd47d2ecda95b759c16b615403
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92681
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
It was added initially to have some dekstop test too.
Now it fails, so no need to keep it.
Change-Id: I94aae2b5e86c46dfd5afa5decabb3333cefd1302
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92680
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
We can use cypress methods on #copy-paste-container and so
we don't need to use explicit wait() method here. The added
cypress call will do the waiting more effectively.
Change-Id: Ib15ffb1d72e422ef303f9e0e17b29d740cd4e7f6
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92678
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
So we don't need to trigger a fake copying to get the
content. It will make things easier.
Change-Id: I0964fe697602d77078f10792355fbda23879c7c9
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92645
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Since we are in an asynchronous environment, we can't just use
variables locally that easy. Cypress has alias feature to
introduce global variables and access them. Since these are
global we should init them anytime before use.
So a stable workflow seems to be this one:
* Set alias to an initial value (e.g. 0)
* Wait until it's actually set (e.g. cy.get().should())
* Set the alias to the value you need in the test
* Wait until it's actually set
* Move on with your test and assert on something using this alias
So we need indicators also for this variable to be set, before we
use it. This will make it work a sequential way, how we need it
to work.
Change-Id: I047373aa884b1ac7bac419cfb1811534c0e95862
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92417
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
So enable the related checks.
Change-Id: Id8af54e8a47de96a9147f074570e0c2b35205e3d
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92361
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
We can store the original position in a 'variable' and
we can use that variable to assert on.
Change-Id: I026c1f3adc445b94d7d7720608ca36a373678352
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92350
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
These checks were not actually unreliable, but the checkbox
handling code was changed.
Change-Id: I4b3f7dc161d9a774466cdfb6bca49de8d1311714
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92339
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Checkboxes have wierd flickering on mobile wizard.
Change-Id: Ie9756f5dc7213a113ea6d05c7c02594a5d1eb1a3
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92308
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
It's a known issue, that these checkboxes have wrong state.
Change-Id: Idca61bc261d03e19e5aa9266bbf8727da94de642
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92214
Tested-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
This helper method was introduced as a reliable
alternative of cy.get(<selector>).contains(<content>).
However we can have the same result using contains()
differently: cy.contains(<selector>, <content>), which
works reliable.
Change-Id: I11553f7d280a0f3a94a8f353410308113dd1c825
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/92194
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
