Skip to content

fix(e2e): repair remaining failing cases against the redesigned console - #7209

Merged
lalaliat merged 6 commits into
agentscope-ai:mainfrom
yutai78786:fix/e2e-remaining-failures
Aug 25, 2026
Merged

fix(e2e): repair remaining failing cases against the redesigned console#7209
lalaliat merged 6 commits into
agentscope-ai:mainfrom
yutai78786:fix/e2e-remaining-failures

Conversation

@yutai78786

@yutai78786 yutai78786 commented Aug 21, 2026

Copy link
Copy Markdown
Collaborator

Submitted by: Li Shizhen·E2E@QPQAT

Problem

The recent console redesign (workspace unification #6504, marketplace unification #6880, session user-groups, virtualized sidebar, textarea sender, redesigned memory card) left a set of e2e cases pointing at removed routes, CSS classes and controls. On fork CI (run 32445567327/878/14) 18 cases failed across p0/p1/p2 tiers; every failure is a test-side staleness issue, no product defect found.

Fix

Re-anchor the affected cases to the current UI surface:

  • Routes: /workspace -> /files; /plugin-manager -> /market?tab=plugins; /coding removed (the IDE now renders on /files behind Coding Mode)
  • Files: the whole-workspace zip upload/download was removed; cases now verify the single-file upload entry (aria Upload files + hidden input) and the per-file download button in the editor toolbar
  • Chat sidebar: two-tier layout (collapsible user groups containing non-collapsible date buckets); date headers now carry data-date-group; collapse assertions moved to the user-group header; session rows scoped to the sessionItem root class so virtual-list rows are not over-matched
  • Chat: sender input is a textarea now; session-menu interactions scroll the virtual list into view before resolving rows
  • Plugins: market defaults to card view and the install action reveals on hover; selectors cover both card and list views
  • Memory: the redesigned ReMeLight card renders switches inline (no collapse panel); cases reset coding-mode/project binding before asserting the workspace tree
  • MCP: enable/disable toggle disambiguated from the OAuth Authorize button that shares its class

Verification

All 19 affected cases pass locally against a fresh build of upstream main (19 passed in 139.65s). Fork CI p0/p1/p2 dispatched on this branch for confirmation.

Companion of #7173 (agents action-cell + project-directory rename, merged).

李时珍·E2E added 3 commits August 21, 2026 17:09
李时珍·E2E
fix(e2e): follow agentscope-ai#6504 workspace redesign — /workspace→/…
a2803aa 
…files route, per-file download, coding IDE on files page

Upstream agentscope-ai#6504 removed the /workspace and /coding routes and the
whole-workspace zip download/upload feature. Re-anchor the affected
cases to the new surface:

- route constants and gotos: /workspace -> /files
- files cases: verify single-file upload entry (aria Upload files +
  hidden input) and per-file download button in the editor toolbar
  (lucide-download), instead of the removed zip buttons
- coding cases: drive Coding Mode via the API toggle and verify the
  IDE activity rail on /files (navigator + source-control buttons),
  dropping the removed /coding navigation and header toggles

署名：李时珍·E2E@QPQAT
李时珍·E2E
Merge remote-tracking branch 'upstream/main' into fix/e2e-remaining-f…
0eaefe2 
…ailures
李时珍·E2E
fix(e2e): repair 18 failing cases against the redesigned console
42559cb 
The console redesign (agentscope-ai#6504 workspace unification, agentscope-ai#6880 marketplace
unification, session user-groups, virtualized sidebar, textarea sender,
redesigned memory card) left many e2e cases pointing at removed routes,
classes and controls. Re-anchor them to the current surface:

- routes: /workspace -> /files, /plugin-manager -> /market?tab=plugins,
  /coding removed (IDE now lives on /files behind Coding Mode)
- files: zip whole-workspace upload/download removed; verify the
  single-file upload entry and per-file download button instead
- chat sidebar: two-tier user-group/date-bucket layout; date headers
  carry data-date-group, collapse moved to the user-group header;
  session rows scoped to the sessionItem root class (virtual list)
- chat: sender input is a textarea now; session menu interactions
  scroll the virtual list into view before resolving rows
- plugins: market defaults to card view; install action reveals on
  hover; selectors cover both card and list views
- memory: redesigned ReMeLight card renders switches inline (no
  collapse panel); reset coding-mode/project binding before asserting
  the workspace tree
- mcp: enable/disable toggle disambiguated from the OAuth Authorize
  button that shares its class

All 19 affected cases pass locally against a fresh build of upstream
main.

署名：李时珍·E2E@QPQAT
@github-project-automation github-project-automation Bot moved this to Todo in QwenPaw Aug 21, 2026
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

Welcome to QwenPaw! 🐾

Hi @yutai78786, this is your 66th Pull Request.

📋 About PR Template

To help maintainers review your PR faster, please make sure to include:

  • ✅ Description - What this PR does and why
  • ✅ Type of Change - Bug fix / Feature / Breaking change / Documentation / Refactoring
  • ✅ Component(s) Affected - Core / Console / Channels / Skills / CLI / Documentation / Tests / CI/CD / Scripts
  • ✅ Checklist:
    • Run and pass pre-commit run --all-files
    • Run and pass relevant tests (pytest or as applicable)
    • Update documentation if needed
  • ✅ Testing - How to test these changes
  • ✅ Local Verification Evidence: 
    pre-commit run --all-files
# paste summary result

pytest
# paste summary result

Complete PR information helps speed up the review process. You can edit the PR description to add these details.

🙌 Join Developer Community

Thanks so much for your contribution! We'd love to invite you to join the official QwenPaw developer group! You can find the Discord and DingTalk group links under the "Developer Community" section on our docs page:
https://qwenpaw.agentscope.io/docs/community

We truly appreciate your enthusiasm—and look forward to your future contributions! 😊

We'll review your PR soon.

李时珍·E2E
fix(e2e): harden three CI-only failures — seed files for empty worksp…
1717caf 
…ace, drop stale /coding wait, anchor agent-stats on settled page

- FILE-004 / FILE-P2-002: CI workspaces start empty, so the file tree
  has no row to open; seed a file via the files API first
- CODE-003: remove the stale wait_for_url(**/coding) left from the
  removed /coding route; assert the IDE rail on /files instead
- agent stats: anchor the post-load assertion on the date-picker
  filter row (only rendered once loading settles) and match the
  CSS-module summary card classes

署名：李时珍·E2E@QPQAT
李时珍·E2E
fix(e2e): reset leftover project binding in files cases, anchor backu…
46c8b3f 
…ps on settled header

CI p0 showed two environment-ordering failures:
- files cases: a preceding coding case leaves a project directory bound,
  so /files renders the (empty) project tree and the seeded workspace
  files never appear; reset coding-mode and the project binding first
- backups: on slow runners the page header (Import button) renders
  after the fixed sleep; anchor on the header action buttons instead

署名：李时珍·E2E@QPQAT
@yutai78786

yutai78786 commented Aug 24, 2026

Copy link
Copy Markdown
Collaborator Author

Verification update: fork CI now green on all three tiers for the latest head (46c8b3f) — p0 (run 32688717575), p1 (run 32684891145), p2 (run 32684889714). All 19 affected cases also pass locally against a fresh build of upstream main (19 passed in 139.65s). Ready for review.

李时珍·E2E
fix(e2e): split p2 add-env cases out of the p0 TestAddEnvironment class
9a0baef 
Two methods carried both the class-level p0 marker and a method-level
p2 marker, so the nightly tier shards (integration and pX) would run
them twice. Move them into a dedicated TestAddEnvironmentP2 class so
tier markers stay mutually exclusive. Verified: p0 71 / p1 89 / p2 42,
union 202 with zero overlap and zero omission.

署名：李时珍·E2E@QPQAT
lalaliat
lalaliat approved these changes Aug 25, 2026
View reviewed changes

@lalaliat lalaliat left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

LGTM

@lalaliat
lalaliat merged commit 9593d16 into agentscope-ai:main Aug 25, 2026
20 of 21 checks passed
@github-project-automation github-project-automation Bot moved this from Todo to Done in QwenPaw Aug 25, 2026
@yutai78786 yutai78786 mentioned this pull request Aug 26, 2026
Merged
7 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@lalaliat lalaliat lalaliat approved these changes

Assignees

No one assigned

Labels

None yet

Projects

QwenPaw
Status: Done

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@yutai78786 @lalaliat