Skip to content

Add manage folders E2E tests for mobile and desktop (BrowserStack) - #102

Open
rwood-moz wants to merge 4 commits into
mainfrom
e2e/folders
Open

Add manage folders E2E tests for mobile and desktop (BrowserStack)#102
rwood-moz wants to merge 4 commits into
mainfrom
e2e/folders

Conversation

@rwood-moz

@rwood-moz rwood-moz commented Aug 11, 2026

Copy link
Copy Markdown
Contributor

What changed?

Add E2E tests for managing folders in WebMail, and update the UI smoke test also.

Why?

Expand the current E2E test suite that runs in BrowserStack on desktop and mobile browsers.

Limitations and Notes

More specifically:

  • Updated the smoke test to run regardless on if the inbox is empty or not at the test start (bug fix)
  • Updated the smoke test to verify the settings dialog appears
  • Updated the light/dark theme test (in the smoke test) to first turn off the 'follow system theme' option so that the switch theme/mode button appears
  • Updated the smoke test to exercise the Manage Folders dialog controls (basic check only)
  • Updated the smoke test for other recent app/locator changes
  • Consolidated the UI smoke test into one test (no need to have separate tests for desktop and mobile)
  • Added a new Manage Folders E2E test to actually add/modify/delete folders via the Manage Folders dialog
  • Moving messages in/out of folders will be done in the separate Message Processing test in Add message processing E2E tests to run in BrowserStack #68
  • Updated to the newest browserstack-node-sdk 1.67.0 I tested this out with these E2E tests in BrowserStack and it worked fine

I worked with Codex AI to make this patch. I reviewed all code changes and tested thoroughly.

Applicable Issues

Fixes #70.

QA Log

I ran the updated E2E tests (UI smoke test and the new folder management test):

All tests pass.

@rwood-moz
rwood-moz force-pushed the e2e/folders branch 6 times, most recently from 5033e40 to f24af0c Compare August 19, 2026 14:16
@rwood-moz
rwood-moz force-pushed the e2e/folders branch 4 times, most recently from 5ad5cf0 to a97ca84 Compare August 26, 2026 20:46
@rwood-moz rwood-moz changed the title WIP: Add manage folders E2E tests for mobile and desktop (BrowserStack) Add manage folders E2E tests for mobile and desktop (BrowserStack) Aug 27, 2026
@rwood-moz
rwood-moz marked this pull request as ready for review August 27, 2026 14:52
@rwood-moz
rwood-moz requested a review from Sancus August 27, 2026 14:52
@rwood-moz
rwood-moz force-pushed the e2e/folders branch 2 times, most recently from 3e950e4 to 9e36820 Compare September 10, 2026 20:32
@rwood-moz

Copy link
Copy Markdown
Contributor Author

After the most recent Stormbox updates this new test no longer works. Moving back into WIP.

@rwood-moz rwood-moz changed the title Add manage folders E2E tests for mobile and desktop (BrowserStack) WIP: Add manage folders E2E tests for mobile and desktop (BrowserStack) Sep 10, 2026
@rwood-moz
rwood-moz removed the request for review from Sancus September 10, 2026 20:33
@rwood-moz
rwood-moz marked this pull request as draft September 10, 2026 20:33
@rwood-moz rwood-moz changed the title WIP: Add manage folders E2E tests for mobile and desktop (BrowserStack) Add manage folders E2E tests for mobile and desktop (BrowserStack) Sep 11, 2026
@rwood-moz
rwood-moz marked this pull request as ready for review September 11, 2026 19:14
@rwood-moz

Copy link
Copy Markdown
Contributor Author

After the most recent Stormbox updates this new test no longer works. Moving back into WIP.

Ok I've updated the new folder management test and the UI smoke test to work with the latest Webmail changes, and this patch is now ready for review.

@rwood-moz
rwood-moz requested a review from Sancus September 11, 2026 19:15
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.

Add message folder E2E tests to run in BrowserStack

1 participant