Integration tests on Apple Silicon attempt to run under Rosetta and fail
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 4/5
- Geschätzter Aufwand
- 3-5 Tage
- Anfängerfreundlichkeit
- 48/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- electron, node.js, typescript
- Bereich
- desktop, testing-qa, tooling
Rechercherichtung
Beginne in extensions/ql-vscode mit run-integration-tests.ts und dem npm run integration-Einstiegspunkt; untersuche anschließend das Verhalten von download.ts im referenzierten vscode-test. Reproduziere das Problem auf einem M1 Mac und vergleiche die heruntergeladene Electron-Binärdatei mit dem ARM-native Pfad, der von der Run and Debug-Startkonfiguration verwendet wird. Als erledigt gilt die Aufgabe, wenn der Befehl ein ARM-natives VS Code ohne Rosetta startet und die Integrationstests bestehen.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the bug
Running npm run integration from the command line on an M1 Mac leads to the following error:
> vscode-codeql@1.5.9 integration
> node ./out/vscode-tests/run-integration-tests.js no-workspace,minimal-workspace
Found .vscode-test/vscode-1.63.2. Skipping download.
Running integration tests in these directories: no-workspace,minimal-workspace
Next integration test dir: no-workspace
Launch args: --disable-extensions,--disable-gpu
rosetta error: /var/db/oah/279281325309952_279281325309952/79a831ad051c57eb341ad61bf19eea2efece2baf38aaf9eb68316bb0af278da6/Electron.aot: attachment of code signature supplement failed: 1
Exit code: null
Done
Unexpected exception while running tests: SIGTRAP
Explanation:
- The
run-integration-tests.tstest harness downloads a stable copy of VS Code to execute the tests in. - On Apple Silicon, this downloads an x86 VS Code / Electron binary, even if there is an ARM-native or universal VS Code binary installed.
% file .vscode-test/vscode-1.63.2/Visual Studio Code.app/Contents/MacOS/Electron
.vscode-test/vscode-1.63.2/Visual Studio Code.app/Contents/MacOS/Electron:: Mach-O 64-bit executable x86_64
- I suspect this behaviour originates from
vscode-test, specifically https://github.com/microsoft/vscode-test/blob/fd0e599cd8e096f58f226509c6b76c68f502e926/lib/download.ts#L50, because it only acceptsdarwinas the download platform, without allowing a choice between x86 or ARM. - This means that the integration tests attempt to run under Rosetta.
- Rosetta fails to launch VS Code due to a code signature mismatch.
- This error doesn't happen when running the integration tests from the
Run and DebugUI. This is because the UI test launcher uses the tasks defined inlaunch.json, which start VS Code directly without going throughrun-integration-tests.ts, and use the same VS Code binary as the parent instance (which will be ARM native if you installed it on M1).
Version
VS Code 1.63.2
CodeQL for VS Code 1.5.9
To reproduce
- Clone repo on an M1 Mac.
cd extensions/ql-vscode && npm install && npm run integration- Observe error.
- Choose
Launch Integration Testsfrom theRun and Debugview in the UI. - Observe no error.
Expected behavior
npm run integration should download and launch an ARM-native VS Code binary, without using Rosetta, and the tests should pass.
Additional context
Observed by @angelapwen and diagnosed in internal discussion.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 539
- Forks
- 240
- Ø Merge
- 2 T. 18 Std.
- Gemergte PRs (30 T.)
- 37
Beitragsleitfaden
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus github/vscode-codeql
-
bug VSCode
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
github/vscode-codeql#4471 · 1 Kommentar ·
-
bug enhancement VSCode
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
github/vscode-codeql#592 · 2 Kommentare ·
-
enhancement VSCode
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 55/100
github/vscode-codeql#4432 · 1 Kommentar ·
-
bug VSCode
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 52/100
github/vscode-codeql#4356 ·
-
bug VSCode
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 64/100
github/vscode-codeql#4339 · 1 Kommentar ·
Alle Issues in github/vscode-codeql
Ähnliche Issues
-
blocklist removal
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 65/100
MetaMask/eth-phishing-detect#296544 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
pastelsky/bundlephobia#1122 ·
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100
-
category/development priority/P2 scope/file-operations scope/testing type/enhancement
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
-
Enatega Customer and Rider app: Add-ons price is not visible to customer after order is placed. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100