Commit Graph

31 Commits (190a3acf78d775d6141a9bba00c1c4e6e5dea145)

Author SHA1 Message Date
Neil Guertin 190a3acf78 Cypress: Fix mobile/impress/apply_font_text_spec.js
Failed because mobile wizard opened too soon.
Fix: Wait for selection before opening mobile wizard.

Signed-off-by: Neil Guertin <neil.guertin@collabora.com>
Change-Id: Ie29e3f5be17bafa2dd09b3f8f5b6415104456cb7
2024-04-27 01:37:10 -04:00
Neil Guertin dfa13bfe62 Replace beforeAll with setupAndLoadDocument
Replace beforeAll in most tests with new helper function
Tests that need to close or reopen test files cannot use
the new function and will be updated later

Signed-off-by: Neil Guertin <neil.guertin@collabora.com>
Change-Id: I372280e55993123fc26f83cb2e6e6e78665a6846
2024-04-25 07:37:56 +02:00
Neil Guertin 7cc0c1996e Cypress: Remove afterEach/afterAll
Cypress recommends NOT cleaning up after tests
  https://docs.cypress.io/guides/references/best-practices#Using-after-Or-afterEach-Hooks
Open documents should not affect other tests
Files will be closed automatically by the server anyways
Tests with more strict requirements can explicitly close
  documents before, during, or after those tests
This is necessary to support beforeAll cleanup
Half of tests were verifying the wrong filename was closed anyways

Signed-off-by: Neil Guertin <neil.guertin@collabora.com>
Change-Id: Iab458963a3fbfe87f732dc97140704c2d0ef53d4
2024-04-25 07:37:56 +02:00
Szymon Kłos f8d457cb62 remove-w2ui: convert mobile bottom bar to JSDialog
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I1e86d91282d15c6ed055f9a74808cd3158ff08de
2024-04-04 13:43:35 +02:00
Neil Guertin 42a36b9afb Fix mobile/impress/apply_font_text_spec.js
Exclude placeholder text
Fix removeShapeSelection detection

Signed-off-by: Neil Guertin <neil.guertin@collabora.com>
Change-Id: Ie8cfd791ae4468fa6994e3a49efe05577e7aae37
2024-03-20 08:46:49 -04:00
Neil Guertin dc72922f1e Fix failing impress text positioning tests
Exact positions have changed between core 23.05 and core 24.04.
Allowed for a range because failures report different positions
on different machines.

Signed-off-by: Neil Guertin <neil.guertin@collabora.com>
Change-Id: Iec8024a4e2c8d310748c3b9f52dcb9b325d1529b
2024-02-27 13:39:06 +01:00
Neil Guertin 6cd22f1f40 Update impress tests
Move selection from beforeAll to each testpoint
Remove clickOnIdle

Signed-off-by: Neil Guertin <neil.guertin@collabora.com>
Change-Id: I1ef87e7df34319c4a1307695ffaa0011b6dbb4cd
2024-02-27 13:39:06 +01:00
Neil Guertin 5162b7b050 Simplify text element selection in impress tests
Signed-off-by: Neil Guertin <neil.guertin@collabora.com>
Change-Id: Ie24dc99f9cfba7b2b563dd168154087c1c7f8b6c
2024-02-27 13:39:06 +01:00
Szymon Kłos b85eb1b03f cypress: fix svg text-decoration detection
this was changed in the core:
commit 0a549b2c9595cd076e3b503b67df93bff380885a
svg: export text decoration for text run not paragraph

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I8906ee07c7ff88c9dcdbebe73d2c620e797a8b3a
2023-08-12 10:34:15 +02:00
Gökay Şatır 8e8e644d55 Add nextcloud, notebookbar and screenshot tags to relevant files.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I1b3c1f586d4e030ed02310a57788a7ed40765dde
2023-05-05 07:07:14 +03:00
Gökay Şatır 7b52bc086d Add mobile tags for relevant tests.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: If0bb80548bb5d390d986bb53fedb2d86bf92e4e8
2023-05-05 07:07:14 +03:00
Gökay Şatır 5deb474685 Update impress tests.
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
Change-Id: I0b47edf6e2788b8e3faaef0d0f6aad2ca033782b
Signed-off-by: Gökay Şatır <gokaysatir@gmail.com>
2023-05-05 07:07:14 +03:00
Szymon Kłos 8585e0e933 cypress: use selectFromColorPicker which doesn't rely on generated id's
prefer picker id, not palette id with number which is generated

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I52d30c3991b516378e82455fdc104b5d94e94f91
2022-09-13 11:29:21 +03:00
Ashod Nakashian de74eae245 cypress: give each test a unique test document
We copy each test document into the work-dir
and now we also give them a random prefix.
This is critical to ensure that parallel runs
do not re-use the same document.

Also, when closing a document, we wait for the
document-name to disappear from the admin
console. Sharing the same document, or even the
name, means we are likely to see random results.

Signed-off-by: Ashod Nakashian <ashod.nakashian@collabora.co.uk>
(cherry picked from commit d998f2c1c8)

Change-Id: I2f41566952c49c9c63ee206bcba6a4dd49cdd6d0
2022-01-11 09:28:03 +01:00
Szymon Kłos 8152a3f9b5 mobile-wizard: don't use uno status updates
we receive that information by welding updates

Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I9ff7f5d26740b1f13d1f1cccd88793ca35a4ee76
2021-07-14 18:29:23 +02:00
Szymon Kłos 76733b7e79 cypress: fix mobile color picker tests
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
Change-Id: I5c8f705e3ca69780ed0c441e7a82bf257da33bda
2021-07-13 20:39:57 +02:00
Rash419 623f34e2ef cypress: enable some tests to check
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: I9243769df3dadd4a8dfa07eaceb7e9a8d8f42950
2021-07-05 12:52:41 +05:30
Rash419 2bda36868f cypress: disable tests temporarily
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: Idfe418f20b57c55dceb1b49996ab7134088d47de
2021-07-01 21:11:27 +05:30
Andreas-Kainz 48ca25a1b1 Mobile Sidebar Character remove Grow and Shrink #2488
Mobile Sidebar Remove Grow Shrink commands

Signed-off-by: Andreas-Kainz <kainz.a@gmail.com>
Change-Id: Ia6de337d7f06efdca40e18359adb88b6b21fd58d
2021-06-10 08:22:29 +02:00
Rash419 fb9e41af1f cypress:skip failing tests temporarily
Signed-off-by: Rash419 <rashesh.padia@collabora.com>
Change-Id: Icfdb53967eef14ad93ebfc6b1978304e9298db70
2021-05-19 14:41:21 +05:30
Szymon Kłos 4a218d656e mobile-wizard: listbox prepared for partial updates
Change-Id: I778f7d46de1fc3c9c2eb61daaee690e8438d57d6
Signed-off-by: Szymon Kłos <szymon.klos@collabora.com>
2021-04-06 09:30:15 +02:00
Tamás Zolnai 35fe0b9541 cypress: remove these useless lines of code.
It does not actually do anything. Font size is not
stored as a 'font-color' attribute.

Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
Change-Id: I8e39a7c1064bce386c85ff21811fc6f95253adc0
2020-12-08 13:46:40 +01:00
Tamás Zolnai 8f9f044865 cypress: extract selectListBoxItem() method.
Change-Id: I4dcc1d77f20955ee9bd425d447a3fb77f1a423f5
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-11-18 14:05:12 +01:00
Tamás Zolnai 2aa37cbaf4 cypress: do not run afterAll method in case of failure.
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>
2020-11-13 11:57:23 +01:00
Tamás Zolnai 2d02cf5cc6 cypress: move all Impress related helper methods into a bigger scope.
Change-Id: I6ab06a9c737258afc11e4fcecc91bc678940cd0f
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-11-06 14:51:21 +01:00
Tamás Zolnai 8922df60be cypress: move some Impress related helper methods to a bigger scope.
We will need these methods in desktop tests too.

Change-Id: I302cb0b57fee8b15fbe0804dec14646cb11f0f2f
Signed-off-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-11-06 13:13:02 +01:00
Tamás Zolnai b4bfe69de0 cypress: reenable some impress tests.
removeShapeSelection() method was updated in the
meantime to work with the new canvas layer.

Change-Id: I107f9dd45c66464c10025b938677e6fcee38b360
2020-10-07 16:12:01 +02:00
Jan Holesovsky 26fc803a0e Use the CanvasTileLayer for all rendering.
Fixes problems particularly with the new Chrome where the original
approach to tiles can be causing blurry text.

Change-Id: Icf598df0e997d493b111fad1c59e7dcb5f376801
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/103681
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Jan Holesovsky <kendy@collabora.com>
2020-09-30 20:16:45 +02:00
Tamás Zolnai 1d39969a9a cypress: extract openTextPropertiesPanel() method.
Change-Id: I14f424d3141d596f982cda349df786dd8940b6c1
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/99977
Tested-by: Jenkins
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-08-03 13:58:24 +02:00
Tamás Zolnai 210118a40a cypress: simplify beforeAll() methods.
We don't need an explicit parameter indicating the platform
type (e.g. mobile, desktop). We can check the userAgent to
get that information.

Change-Id: Ibbc041c4a389cff92203d245f55268ec4e93b0a0
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/99889
Tested-by: Jenkins
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-07-31 19:06:31 +02:00
Tamás Zolnai baafb28002 cypress: split some bigger test suites.
Change-Id: I91a129f2f2c65c5190c22a47ff467d26a8047654
Reviewed-on: https://gerrit.libreoffice.org/c/online/+/98907
Tested-by: Jenkins
Tested-by: Jenkins CollaboraOffice <jenkinscollaboraoffice@gmail.com>
Reviewed-by: Tamás Zolnai <tamas.zolnai@collabora.com>
2020-07-16 16:49:13 +02:00