Skip to content

TF-3605 E2E Mailbox create, rename, move and delete sub folder #3607

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 4 commits into
base: patrol-integration-test
Choose a base branch
from

Conversation

tddang-linagora
Copy link
Collaborator

@tddang-linagora tddang-linagora commented Apr 2, 2025

Issue

Test result

✅ Should create personal mailbox successfully (integration_test/tests/mailbox/create_personal_folder_test.dart) (17s)
✅ Should create, rename, move and delete mailbox successfully (integration_test/tests/mailbox/create_rename_move_and_delete_mailbox_test.dart) (31s)

Test summary:
📝 Total: 2
✅ Successful: 2
❌ Failed: 0
⏩ Skipped: 0
📊 Report: .../tmail-flutter/build/app/reports/androidTests/connected/index.html
⏱️  Duration: 73s

Test video

personal-and-sub-folder.webm

@tddang-linagora tddang-linagora self-assigned this Apr 2, 2025
@tddang-linagora tddang-linagora changed the base branch from master to patrol-integration-test April 2, 2025 02:14
@tddang-linagora tddang-linagora linked an issue Apr 2, 2025 that may be closed by this pull request
2 tasks
})
.$(TMailButtonWidget)
.which<TMailButtonWidget>((widget) {
return widget.icon == ImagePaths().icArrowRight;
Copy link
Member

Choose a reason for hiding this comment

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

We should use key to avoid duplicated action

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Done

@@ -290,7 +290,7 @@ dev_dependencies:

patrol_log: 0.3.0

patrol_finders: 2.7.0
patrol_finders: 2.7.2
Copy link
Member

Choose a reason for hiding this comment

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

why do we need change it?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Comment on lines 22 to 29
await threadRobot.openMailbox();
await mailboxMenuRobot.longPressMailboxWithName(
appLocalizations.inboxMailboxDisplayName,
);
await mailboxMenuRobot.tapCreateNewSubFolder();
await mailboxMenuRobot.enterNewSubFolderName(subFolderName);
await mailboxMenuRobot.confirmCreateNewSubFolder();
await _expectMailboxWithNameVisible(subFolderName);
Copy link
Member

Choose a reason for hiding this comment

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

duplicate with PR #3599

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

The sub folder created in #3599 is hidden, I need to create a new one for this PR for other purposes.

@dab246
Copy link
Member

dab246 commented Apr 9, 2025

  • Handle case create folder

@tddang-linagora tddang-linagora requested a review from dab246 April 9, 2025 07:24
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.

[E2E] [Mailbox management] Move, rename, create, delete mailbox
2 participants