Skip to content

Conversation

@landomen
Copy link
Contributor

@landomen landomen commented Dec 11, 2025

Task/Issue URL: https://app.asana.com/1/137249556945/project/1210594645151737/task/1212389758584564?focus=true

Description

Fixes the pre-onboarding maestro test to account for an additional step

Steps to test this PR

Run maestro test .maestro/onboarding/onboarding_dismiss_all_dialogs.yaml

UI changes

No UI changes


Note

Updates the pre-onboarding Maestro test to handle an additional AI chat step in the onboarding flow.

  • E2E Tests:
    • Update /.maestro/shared/pre_onboarding.yaml:
      • Add tapOn: "next" and assertVisible for ".*want easy access to private AI chat?.*" to cover new onboarding step.
      • Ensure flow continues to final ".*Try a search!.*" screen.

Written by Cursor Bugbot for commit f153631. This will update automatically on new commits. Configure here.

Copy link
Contributor

@karlenDimla karlenDimla left a comment

Choose a reason for hiding this comment

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

LGTM.

@karlenDimla karlenDimla self-assigned this Dec 11, 2025
@landomen landomen merged commit 0107a5a into develop Dec 11, 2025
13 checks passed
@landomen landomen deleted the fix/domen/pre-onboarding-maestro branch December 11, 2025 09:07
cursor bot pushed a commit that referenced this pull request Dec 19, 2025
Task/Issue URL:
https://app.asana.com/1/137249556945/project/1210594645151737/task/1212389758584564?focus=true

### Description
Fixes the pre-onboarding maestro test to account for an additional step

### Steps to test this PR
Run `maestro test
.maestro/onboarding/onboarding_dismiss_all_dialogs.yaml`

### UI changes
No UI changes

<!-- CURSOR_SUMMARY -->
---

> [!NOTE]
> Updates the pre-onboarding Maestro test to handle an additional AI
chat step in the onboarding flow.
> 
> - **E2E Tests**:
>   - Update `/.maestro/shared/pre_onboarding.yaml`:
> - Add `tapOn: "next"` and `assertVisible` for `".*want easy access to
private AI chat?.*"` to cover new onboarding step.
>     - Ensure flow continues to final `".*Try a search!.*"` screen.
> 
> <sup>Written by [Cursor
Bugbot](https://cursor.com/dashboard?tab=bugbot) for commit
f153631. This will update automatically
on new commits. Configure
[here](https://cursor.com/dashboard?tab=bugbot).</sup>
<!-- /CURSOR_SUMMARY -->
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants