Commit c429a05
ci(e2e-autotest): add Linux and macOS to the runner matrix (#1621)
* ci(e2e-autotest): add Linux to the runner matrix
The e2e-test job was hard-coded to `runs-on: windows-latest` even though
the autotest framework and the platformMap in this workflow were already
written to be OS-aware. This change wires the matrix axis through so we
exercise every plan on both Windows and Linux on every PR / schedule run.
Changes:
* matrix.os = [windows-latest, ubuntu-latest]; runs-on uses it
* JDK 25 step is now cross-platform pwsh (junction on Windows,
symlink to /opt/jdk-25 on Linux, plus an in-place rewrite of the
Windows-style java.jdt.ls.java.home in the java25 plan)
* New `Setup virtual display (Linux)` step installs xvfb and
exports DISPLAY=:99 before the autotest CLI launches VS Code
(the autotest framework does not spawn Xvfb itself)
* Artifact name suffix: results-<plan>-<os> so Windows and Linux
results don't collide
* analyze.organize-results: preserve the <plan>-<os> suffix so the
aggregate report keeps Windows and Linux runs distinct
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
* ci(e2e-autotest): also include macOS in the runner matrix
Adds `macos-latest` to the matrix.os list. The cross-platform pwsh
branch added in the previous commit already covers macOS:
* `\True` is false on macOS so the symlink path runs unchanged
* macOS supports `sudo ln -sfn` to /opt/jdk-25 (added `mkdir -p /opt`
so the path exists before the link is created)
* macOS GitHub-hosted runners have a real display, so no xvfb step is
needed (the xvfb step stays gated on runner.os == 'Linux')
* PowerShell Core is preinstalled on macos-latest runners
* @vscode/test-electron downloads the darwin VSIX automatically
Run count grows from 17 plans × 2 OSes to 17 × 3.
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
---------
Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>1 parent 5ec3f2b commit c429a05
1 file changed
Lines changed: 36 additions & 6 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
88 | 88 | | |
89 | 89 | | |
90 | 90 | | |
91 | | - | |
| 91 | + | |
92 | 92 | | |
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
| 96 | + | |
96 | 97 | | |
97 | 98 | | |
98 | 99 | | |
| |||
117 | 118 | | |
118 | 119 | | |
119 | 120 | | |
120 | | - | |
| 121 | + | |
121 | 122 | | |
122 | 123 | | |
123 | 124 | | |
124 | | - | |
| 125 | + | |
| 126 | + | |
| 127 | + | |
| 128 | + | |
| 129 | + | |
| 130 | + | |
| 131 | + | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
125 | 141 | | |
126 | 142 | | |
127 | 143 | | |
| |||
132 | 148 | | |
133 | 149 | | |
134 | 150 | | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
| 158 | + | |
| 159 | + | |
| 160 | + | |
| 161 | + | |
135 | 162 | | |
136 | 163 | | |
137 | 164 | | |
| |||
217 | 244 | | |
218 | 245 | | |
219 | 246 | | |
220 | | - | |
| 247 | + | |
221 | 248 | | |
222 | 249 | | |
223 | 250 | | |
| |||
246 | 273 | | |
247 | 274 | | |
248 | 275 | | |
| 276 | + | |
| 277 | + | |
249 | 278 | | |
| 279 | + | |
| 280 | + | |
250 | 281 | | |
251 | | - | |
252 | | - | |
| 282 | + | |
253 | 283 | | |
254 | 284 | | |
255 | 285 | | |
| |||
0 commit comments