Skip to content

Conversation

@dragosb01
Copy link
Collaborator

📜 Tickets

https://mozilla-hub.atlassian.net/browse/MTE-4988

💡 Description

Refactored bookmark smoke tests with Page Object Model

@dragosb01 dragosb01 requested review from a team as code owners November 7, 2025 14:00
@mobiletest-ci-bot
Copy link

mobiletest-ci-bot commented Nov 7, 2025

💪 Quality guardian

11 tests files modified. You're a champion of test coverage! 🚀

🥇 Perfect PR size

Smaller PRs are easier to review. Thanks for making life easy for reviewers! ✨

✅ Per-file coverage

All changed files meet the threshold of 35.0%.

Generated by 🚫 Danger Swift against 45360ef

}

// https://mozilla.testrail.io/index.php?/cases/view/2306907
// Smoketest
Copy link
Contributor

Choose a reason for hiding this comment

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

Add Smoketest TAE in the comment to align with the other features

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated

}

// https://mozilla.testrail.io/index.php?/cases/view/2306909
// Smoketest
Copy link
Contributor

Choose a reason for hiding this comment

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

Same here

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated

}

// https://mozilla.testrail.io/index.php?/cases/view/2784448
// Smoketest
Copy link
Contributor

Choose a reason for hiding this comment

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

Same

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

updated

Copy link
Contributor

@mdotb-moz mdotb-moz left a comment

Choose a reason for hiding this comment

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

I add some minor comments, but the PR looks go to me

@isabelrios isabelrios merged commit 6b44b56 into main Nov 11, 2025
9 checks passed
@isabelrios isabelrios deleted the dbigu/MTE-4988 branch November 11, 2025 14:19
@github-actions
Copy link
Contributor

🚀 PR merged to main, targeting version: 145.2

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.

6 participants