Commit 578059f
authored
fix(e2e): stabilize Linux UI tests on small Xvfb display (#1003)
* fix(e2e): stabilize Linux UI tests on small Xvfb display
The Linux-UI workflow was failing 6 steps across the two e2e plans on
the 1024x768 Xvfb display used in CI. Three independent root causes:
1. Sticky pane-header click interception. The JAVA PROJECTS view is
rendered inside the Explorer sidebar. With OUTLINE/TIMELINE/MAVEN
sections also visible, the Java Projects pane has very little
vertical space, and its sticky pane-header sits right on top of the
first tree row. Playwright finds the my-app treeitem but the click
is consumed by the section header (the Playwright call log shows the
pane-header subtree intercepts pointer events). Fix: bump Xvfb to
1920x1080 and explicitly close the auxiliary (Chat) bar plus
collapse OUTLINE, TIMELINE and the workspace root before
interacting with the Java Projects tree.
2. Unsupported action syntax in java-dep-project-explorer.yaml. The
strings "expand my-app tree item", "expand src/main/java tree item"
and "expand com.mycompany.app tree item" do not match any pattern in
autotest's ActionResolver (only "expandTreeItem <name>" is
recognized). They silently fell back to the command palette and
no-op'd, so the tree never actually expanded and the subsequent
verifyTreeItem checks for "com.mycompany.app" and "App" timed out.
Fix: use the "expandTreeItem <name>" form.
3. Hidden command-palette commands. java.view.package.linkWith
FolderExplorer, unlinkWithFolderExplorer and revealInProjectExplorer
all have "when": false on their commandPalette menu contribution and
cannot be invoked via the command palette. They were no-ops in the
previous plan (the reveal-in-project-explorer step was even opening
the wrong "Create Java Project" picker). Fix: invoke them by command
id with "executeVSCodeCommand".
Also close the editor opened by the create-class step before the
create-package step, so link-with-editor doesn't auto-expand the tree
and push my-app under the sticky header again.
* test(plans): replace executeVSCodeCommand with real UI for link/unlink/reveal
Once autotest >=0.6.6 ships clickViewTitleAction and contextMenuOnEditorTab,
exercise the actual VS Code UI affordances instead of dispatching the
extension commands by id:
- unlinkWithFolderExplorer / linkWithFolderExplorer → clickViewTitleAction
on the JAVA PROJECTS pane title bar (overflow menu when sync is on/off).
- revealInProjectExplorer → right-click the editor tab and pick
'Reveal in Java Project Explorer' from editor/title/context.
These were palette-hidden ('when: false') and therefore previously fell
back to executeVSCodeCommand. Verified locally end-to-end: 21/21 steps
pass on the project-explorer plan and 39/39 on file-operations with the
locally-built autotest.
closeAuxiliaryBar still uses executeVSCodeCommand intentionally — it is
a built-in VS Code helper (not extension UI under test) and now dispatches
through a real keybinding rather than the broken driver.executeCommand
path.1 parent 952342e commit 578059f
3 files changed
Lines changed: 69 additions & 9 deletions
File tree
- .github/workflows
- test/e2e-plans
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
18 | 18 | | |
19 | 19 | | |
20 | 20 | | |
21 | | - | |
| 21 | + | |
| 22 | + | |
| 23 | + | |
| 24 | + | |
22 | 25 | | |
23 | 26 | | |
24 | 27 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
25 | 25 | | |
26 | 26 | | |
27 | 27 | | |
| 28 | + | |
28 | 29 | | |
29 | 30 | | |
30 | | - | |
| 31 | + | |
31 | 32 | | |
32 | 33 | | |
33 | 34 | | |
34 | 35 | | |
35 | 36 | | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
| 47 | + | |
| 48 | + | |
| 49 | + | |
| 50 | + | |
| 51 | + | |
| 52 | + | |
| 53 | + | |
36 | 54 | | |
37 | 55 | | |
38 | 56 | | |
| |||
68 | 86 | | |
69 | 87 | | |
70 | 88 | | |
| 89 | + | |
| 90 | + | |
| 91 | + | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
| 96 | + | |
| 97 | + | |
71 | 98 | | |
72 | 99 | | |
| 100 | + | |
73 | 101 | | |
74 | 102 | | |
75 | 103 | | |
| 104 | + | |
76 | 105 | | |
77 | 106 | | |
78 | 107 | | |
| |||
99 | 128 | | |
100 | 129 | | |
101 | 130 | | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
102 | 134 | | |
103 | 135 | | |
104 | 136 | | |
| |||
146 | 178 | | |
147 | 179 | | |
148 | 180 | | |
| 181 | + | |
| 182 | + | |
| 183 | + | |
149 | 184 | | |
150 | 185 | | |
151 | 186 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
21 | 21 | | |
22 | 22 | | |
23 | 23 | | |
| 24 | + | |
24 | 25 | | |
25 | 26 | | |
26 | 27 | | |
| |||
29 | 30 | | |
30 | 31 | | |
31 | 32 | | |
| 33 | + | |
| 34 | + | |
| 35 | + | |
| 36 | + | |
| 37 | + | |
| 38 | + | |
| 39 | + | |
| 40 | + | |
| 41 | + | |
| 42 | + | |
| 43 | + | |
| 44 | + | |
| 45 | + | |
| 46 | + | |
32 | 47 | | |
33 | 48 | | |
34 | 49 | | |
| |||
45 | 60 | | |
46 | 61 | | |
47 | 62 | | |
| 63 | + | |
| 64 | + | |
48 | 65 | | |
49 | | - | |
| 66 | + | |
50 | 67 | | |
51 | 68 | | |
52 | | - | |
| 69 | + | |
53 | 70 | | |
54 | 71 | | |
55 | 72 | | |
| |||
60 | 77 | | |
61 | 78 | | |
62 | 79 | | |
63 | | - | |
| 80 | + | |
64 | 81 | | |
65 | 82 | | |
66 | 83 | | |
| |||
72 | 89 | | |
73 | 90 | | |
74 | 91 | | |
| 92 | + | |
| 93 | + | |
| 94 | + | |
| 95 | + | |
75 | 96 | | |
76 | | - | |
| 97 | + | |
77 | 98 | | |
78 | 99 | | |
79 | 100 | | |
| |||
84 | 105 | | |
85 | 106 | | |
86 | 107 | | |
87 | | - | |
| 108 | + | |
88 | 109 | | |
89 | 110 | | |
90 | 111 | | |
91 | | - | |
| 112 | + | |
| 113 | + | |
92 | 114 | | |
93 | 115 | | |
94 | 116 | | |
| |||
98 | 120 | | |
99 | 121 | | |
100 | 122 | | |
101 | | - | |
| 123 | + | |
102 | 124 | | |
103 | 125 | | |
104 | 126 | | |
| |||
0 commit comments