Mobile view tests #1006
Merged
Mobile view tests #1006
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
ScharfViktor
force-pushed
the
mobile-view-tests
branch
7 times, most recently
from
14c6dd7 to
57aff48
Compare
Contributor
There was a problem hiding this comment.
Pull Request Overview
This PR introduces comprehensive mobile E2E testing support by adding device-specific test configurations for mobile and tablet devices including Android (Pixel 5), iOS (iPhone 12), and iPad Pro 11 in both portrait and landscape orientations.
Key changes include:
- Added mobile device browser configurations with proper viewport and user agent settings
- Created mobile-specific test scenarios and conditional logic for mobile UI interactions
- Implemented parallel test execution with separate logging for each device configuration
Reviewed Changes
Copilot reviewed 6 out of 6 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| tests/e2e/support/objects/app-files/resource/actions.ts | Added mobile-specific UI interaction logic for Collabora editor and upload workflows |
| tests/e2e/support/environment/actor/shared.ts | Configured device emulation settings for mobile and tablet browsers |
| tests/e2e/cucumber/features/mobile-view/smoke.feature | Created comprehensive mobile test scenario covering file operations and viewers |
| tests/e2e/cucumber/environment/index.ts | Added browser launch configurations for mobile device testing |
| package.json | Added npm scripts for individual and parallel mobile test execution |
| .woodpecker.star | Implemented CI pipeline for mobile E2E testing with proper service dependencies |
ScharfViktor
force-pushed
the
mobile-view-tests
branch
from
57aff48 to
a0a9ff6
Compare
ScharfViktor
commented
Jul 29, 2025
ScharfViktor
requested review from
AlexAndBear,
JammingBen,
individual-it and
prashant-gurung899
JammingBen
approved these changes
Jul 30, 2025
Contributor
JammingBen
left a comment
JammingBen
left a comment
There was a problem hiding this comment.
LGTM.
We also should add tests for the recently added bottom drawer at some point. But it makes sense to wait until #1017 is done.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
related #891
https://github.com/user-attachments/assets/3025dbee-d618-40ca-910d-d6980a8d352c
Add Mobile E2E Testing Support
This PR introduces comprehensive mobile and tablet testing capabilities for our e2e test suite.
Overview
Mobile view testing is now supported with multiple device configurations running in parallel within a single
e2e-mobile-viewpipeline.Devices
Localy running:
pnpm test:e2e:mobile-chromiumpnpm test:e2e:mobile-webkitpnpm test:e2e:ipad-chromiumpnpm test:e2e:ipad-safarior
pnpm test:e2e:mobile-parallel# Run all mobile tests in paralleltest output
Since we are running tests in parallel, I have adjusted the test results output to separate them by device.
Again issue in the webkit after closing text editor. similar was in the #954