Skip to content

Fix testUserFlow integration tests#5352

Open
manuroe wants to merge 1 commit intodevelopfrom
fix/user-flow-tests
Open

Fix testUserFlow integration tests#5352
manuroe wants to merge 1 commit intodevelopfrom
fix/user-flow-tests

Conversation

@manuroe
Copy link
Copy Markdown
Member

@manuroe manuroe commented Apr 3, 2026

Fixes two issues causing testUserFlow integration tests to fail:

  • Use Close instead of Cancel button label when dismissing the location picker (button label changed)
  • Scroll up before tapping the People button in room details to ensure it's visible on screen

At least, it now passes on my machine:
image

... but not luck on the CI (yet):
https://github.com/element-hq/element-x-ios/actions/runs/23952355031/job/69862641061
It failed to find the room list search button at UserFlowTests.checkRoomFlows():42:

        // Search for the special test room
        let searchField = app.searchFields.firstMatch
        searchField.clearAndTypeText(Self.integrationTestsRoomName, app: app)

- Use 'Close' instead of 'Cancel' button label when dismissing location picker
- Scroll up to make the People button visible before tapping it

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@manuroe manuroe requested a review from a team as a code owner April 3, 2026 15:49
@manuroe manuroe requested review from pixlwave and removed request for a team April 3, 2026 15:49
@manuroe manuroe added the pr-build for changes related to build, tools, CI/CD label Apr 3, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Apr 3, 2026

❌ 1 Tests Failed:

Tests completed Failed Passed Skipped
914 1 913 0
View the full list of 1 ❄️ flaky test(s)
UserFlowTests::testUserFlow()

Flake rate in main: 64.63% (Passed 162 times, Failed 296 times)

Stack Traces | 664s run time
Failed to get list of active applications: Accessibility error kAXErrorIPCTimeout from AXUIElementCopyMultipleAttributeValues for 1102 (XCUIElement.swift:32)

To view more test analytics, go to the Test Analytics Dashboard
📋 Got 3 mins? Take this short survey to help us improve Test Analytics.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

pr-build for changes related to build, tools, CI/CD

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant