To avoid issues coming from the delayed autosave.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: Ib98b35396efb4fdfdf84846dd23619c666accbdb
So we have separate canvasShouldBeFullWhite() and
canvasShouldNotBeFullWhite() methods, which might make
the code more readable, then having it controlled by
a bool parameter.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I0932a7615c84c35293c2cc8dade0db6468de8224
So we have separate imageShouldBeFullWhite() and
imageShouldNotBeFullWhite() methods, which might make
the code more readable, then having it controlled by
a bool parameter.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I2ada6b9e6f6d93b8a48e9ddd3c851ef3e6fcd632
By now, we don't run the same online branch against more
core branches, so this code is obsolete.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I90768d16628e735129880cc4cf823bf0b7d9ad1e
We use this method in various contexts, where it's not
necessary that we have the cursor of the focus. Focus can
be tested in focus related tests.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: Ic27de45917657d4ac1734565d81d5bbf190ad333
So we can specify which cursor we need here. The default is
the document cursor, but this methods can be used for
formulabar cursor or dialog cursor too. More cursors might
exists at the same time, that's why simple '.blinking-cursor'
selector does not work.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I1fd04e188ab53e70d0e8303fd7502f183c18bef3
cy.click() will grab the focus to the textarea.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I20f0df2fd8a31d3ff4b85ba38fdc503c10987dbb
View also jumps with cursor movement, so we can't be
sure that the vertical/horizontal pozition actually
increase / decrease. Better to test only that the position
was changed.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I2358348c3f6d4f2278eff4e2db4a8409bc86b25a
Now we handle support file better for parallel test run too:
a13b1a93bc
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I4a88f815ddb1c4d79042134cc07e0c418d4ea10f
moveCursor follows the event-indicator rule,
which makes it more stable.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: Ief6e6c4d3a058c95652113229fd0ecbb65fb3e48
Add a modifier parameter, so we can easily extend this
code with 'shift' modifier later.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I820c2c89838ba2a367d561e362d3c04f8487022e
Also use a bit longer waiting time for mobile wizard to
become idle. With the interfering user the timing is
a bit different.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I4f24c700cd718bd129b713f2fd12819f92772b64
Added `make check-interfer-mobile` command for running
interference tests. Now it works with writer tests.
I'll generalize it later.
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I2181a8653d61cc6ed873e2887a3e42767f46f177
Create an alias for the iframes, so we don't need
to find them every time, but we can use the
reference to them. It make the log more clear
and also makes things faster.
Change-Id: I6ca21f951568e46cec3d6a574ce1798b522067ea
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
For some reason, assertions do not work properly
in afterEach hook after a test failure. So in case
of test failure we don't check the admin page,
whether the test file was actually closed, but we
just wait a constant time.
Change-Id: If2eb927b0383816639689ec67eef7e3cd1142366
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
To reuse as much code as we can. We can use the same
method on mobile and on desktop.
Change-Id: I15b1038313145db6cb6ea07489c0d30a0fc2ce64
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Simplified getTextPosForFirstCell() method and
renamed to better match with the function.
Also modified getCursorPos() method. Sometimes
we have an additional hidden text cursor for
formulabar.
Change-Id: Ia8adc645ef3f20426dd509803fcfc4f35662a779
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
After a test failure, it's common, that the afterAll
method also fails which covers the right error
message and also makes the following tests to be ignored.
Change-Id: I5c1bbc5dd8b1b48dd2a8f1fc20a8ca3b8ecd8462
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
For some reason if we use proxy.php continuously, then
it will eventually fail on the test. With visiting the
localhost in the afterEach method we can avoid that.
Change-Id: If56c8e40e9e2b9002c1c5b0b91851e4a19a2c9bd
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
We got some new exceptions and also we have a start
screen now.
Change-Id: I22288ede30540437e8b0a5d3ce4c607d42a465bc
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Let's check the canvas to exist and wait until the
first tile arrives and so the canvas is not fully
white.
Change-Id: I7102298a406d61246fc40a0842bafb2cd124d81e
I reported the issue. Let's have a workaround until,
the bug fix is released.
Change-Id: I7d7719d311e0c0584b7d6e710286f28962d00a96
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/103038
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Newly close button does not actually close the document,
but steps back to read only mode. So we need to push it
twice.
Change-Id: Ic38f1e658396d9afe49496cfb75aa68df1c4623f
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/102997
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
