Skip to content

no-bug: added tests for unloading all other workspaces#13100

Merged
mr-cheffy merged 8 commits into
zen-browser:devfrom
AfeefurR:unload-spaces-test
Apr 8, 2026
Merged

no-bug: added tests for unloading all other workspaces#13100
mr-cheffy merged 8 commits into
zen-browser:devfrom
AfeefurR:unload-spaces-test

Conversation

@AfeefurR
Copy link
Copy Markdown
Contributor

@AfeefurR AfeefurR commented Apr 5, 2026

Added tests for gZenWorkspaces.unloadAllOtherWorkspaces()
Verifies

  • Nothing is unloaded when only one workspace exists
  • Only tabs in inactiveWorkspaces are unloaded
  • Essential tabs are not unloaded

Reference #12751

@AfeefurR AfeefurR requested a review from mr-cheffy as a code owner April 5, 2026 17:24
@dosubot dosubot Bot added the size:L This PR changes 100-499 lines, ignoring generated files. label Apr 5, 2026
Comment thread src/zen/tests/spaces/browser_unload_all_other_spaces.js Outdated
ok(tab.linkedPanel, "Tab should have linked panel before unload");
}

await gZenWorkspaces.unloadAllOtherWorkspaces();
Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Shouldn't you switch to the original space first? Or maybe switch, create the tabs, switch back?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Not sure I'm getting what you mean. Are you saying I should create the tabs in the new Test Workspace, then switch back to the default one and call unload?

@mr-cheffy mr-cheffy self-requested a review April 7, 2026 17:50
@mr-cheffy mr-cheffy changed the title tests: added tests for unloading all other workspaces no-bug: added tests for unloading all other workspaces Apr 7, 2026
@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 7, 2026
Comment thread src/zen/tests/spaces/browser_unload_all_other_spaces.js Outdated
Comment thread src/zen/tests/spaces/browser_unload_all_other_spaces.js Outdated
@dosubot dosubot Bot removed the lgtm This PR has been approved by a maintainer label Apr 7, 2026
Comment thread src/zen/tests/spaces/browser_unload_all_other_spaces.js
Comment thread src/zen/tests/spaces/browser_unload_all_other_spaces.js Outdated
@AfeefurR AfeefurR force-pushed the unload-spaces-test branch from b0da590 to f7df42b Compare April 8, 2026 20:01
@AfeefurR
Copy link
Copy Markdown
Contributor Author

AfeefurR commented Apr 8, 2026

Not directly related, but the browser_issue_10455.js test always times out for me. All the spaces tests pass when i remove that file from browser.toml.

@dosubot dosubot Bot added the lgtm This PR has been approved by a maintainer label Apr 8, 2026
@mr-cheffy mr-cheffy merged commit 8b9f449 into zen-browser:dev Apr 8, 2026
1 check passed
@mr-cheffy
Copy link
Copy Markdown
Member

Thanks!

@AfeefurR AfeefurR deleted the unload-spaces-test branch April 9, 2026 17:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

lgtm This PR has been approved by a maintainer size:L This PR changes 100-499 lines, ignoring generated files.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants