Skip to content

Conversation

@ScharfViktor
Copy link
Contributor

@ScharfViktor ScharfViktor commented Sep 15, 2025

remove flaky step create .odt file from mobile suite

@ScharfViktor ScharfViktor force-pushed the disableCreatingOdtInMobileTests branch from d058b7f to 5ac7fbd Compare September 15, 2025 06:55
@ScharfViktor ScharfViktor marked this pull request as ready for review September 15, 2025 07:07
Copilot AI review requested due to automatic review settings September 15, 2025 07:07
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR removes flaky mobile testing functionality from the end-to-end test suite. The main purpose is to eliminate unreliable test steps that were causing instability in the mobile test suite, specifically around creating ODT (OpenDocument Text) files.

  • Removes the creation of .odt files from the mobile smoke test feature
  • Integrates mobile-view testing into the main e2e test pipeline instead of using a separate pipeline
  • Removes the dedicated e2eMobileTests function and associated infrastructure

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 1 comment.

File Description
tests/e2e/cucumber/features/mobile-view/smoke.feature Removes the flaky ODT file creation step from mobile smoke tests
.woodpecker.star Integrates mobile-view tests into main e2e pipeline and removes dedicated mobile test infrastructure

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@ScharfViktor ScharfViktor merged commit bdee513 into main Sep 15, 2025
28 checks passed
@ScharfViktor ScharfViktor deleted the disableCreatingOdtInMobileTests branch September 15, 2025 07:20
@openclouders openclouders mentioned this pull request Sep 15, 2025
1 task
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants