Skip to content

Design and implement E2E test suite and reorganize e2e directory#812

Open
azayrahmad wants to merge 4 commits into
mainfrom
e2e-suite-reorg-3828893124759560142
Open

Design and implement E2E test suite and reorganize e2e directory#812
azayrahmad wants to merge 4 commits into
mainfrom
e2e-suite-reorg-3828893124759560142

Conversation

@azayrahmad

Copy link
Copy Markdown
Owner

This PR introduces a comprehensive E2E test suite and reorganizes the existing test structure for better maintainability. It also includes a fix for a bug discovered in the Explorer application during test development.

Key changes:

  1. Directory Reorganization: Moved existing and new E2E tests into logical subdirectories (e2e/system, e2e/apps, e2e/explorer).
  2. New Tests:
    • smoke.spec.js: Verifies the system boots correctly and monitors for console errors.
    • notepad.spec.js: Tests opening Notepad, typing, saving to ZenFS, and reopening the file.
    • solitaire.spec.js: Verifies basic interaction with the Solitaire game.
    • basic_ops.spec.js: Tests core Explorer operations like creating files/folders, view modes, and deletion.
  3. Bug Fix: In src/shell/explorer/explorer-app.js, fixed an issue where double-clicking virtual items like "My Computer" didn't trigger navigation.
  4. Maintenance: Updated existing tests to reflect the rename of zenexplorer to explorer and the updated UI labels for "Windows Explorer".
  5. Verification: All 26 E2E tests pass, and visual verification was performed via Playwright screenshots.

PR created automatically by Jules for task 3828893124759560142 started by @azayrahmad

- Reorganized e2e/ directory into system/, apps/, and explorer/
- Added e2e/system/smoke.spec.js for system boot verification
- Added e2e/apps/notepad.spec.js for full lifecycle testing
- Added e2e/apps/solitaire.spec.js for basic game interaction
- Added e2e/explorer/basic_ops.spec.js for file system operations
- Fixed existing tests broken by application ID and label changes
- Fixed a bug in Explorer where virtual items didn't open on double-click
- Verified all tests pass and frontend renders correctly

Co-authored-by: azayrahmad <10110227+azayrahmad@users.noreply.github.com>
@google-labs-jules

Copy link
Copy Markdown
Contributor

👋 Jules, reporting for duty! I'm here to lend a hand with this pull request.

When you start a review, I'll add a 👀 emoji to each comment to let you know I've read it. I'll focus on feedback directed at me and will do my best to stay out of conversations between you and other bots or reviewers to keep the noise down.

I'll push a commit with your requested changes shortly after. Please note there might be a delay between these steps, but rest assured I'm on the job!

For more direct control, you can switch me to Reactive Mode. When this mode is on, I will only act on comments where you specifically mention me with @jules. You can find this option in the Pull Request section of your global Jules UI settings. You can always switch back!

New to Jules? Learn more at jules.google/docs.


For security, I will only act on instructions from the user who triggered this task.

azayrahmad and others added 3 commits February 6, 2026 15:08
- Fixed double initialization of azOS in src/system/os-init.js
- Updated all E2E tests to handle the "Press any key to continue" boot prompt
- Added explicit waits for window.System.launchApp to avoid race conditions
- Switched to context menu for file deletion in basic_ops.spec.js for reliability
- Verified all 26 tests pass successfully

Co-authored-by: azayrahmad <10110227+azayrahmad@users.noreply.github.com>
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.

1 participant