[Feature] Update the VSIX path used by smoke tests

Geschlossen Anfängerfreundlich
  • Stimmt etwas nicht?
#3,002 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Maintainer antworten meist innerhalb von 1 Tag

@EmmaYuan1015 arbeitet bereits daran.

Seit 14.10.2026.

  • #3001 von @EmmaYuan1015 — offen

Bewertung

Schwierigkeit
2/5
Geschätzter Aufwand
1-3 Stunden
Anfängerfreundlichkeit
76/100
Issue-Typ
Bug
Klarheit
Klar beschrieben
Aktivitätsstatus
Aktiv
Tech-Stack
typescript
Bereich
testing

Rechercherichtung

Der fehlerhafte Pfad ist ein Aufruf von path.resolve(__dirname, "..", "..", "resources", "extension") im VSIX-Smoke-Testcode unter test/smoke. Das Issue nennt die Datei nicht, daher suche nach diesem Aufruf, um sie zu finden. Gib das aufgelöste Verzeichnis aus dem kompilierten Test in test/smoke/out aus, um zu bestätigen, dass es auf test/resources/extension zeigt. Fertig ist es, wenn der Pfad zu test/smoke/resources/extension aufgelöst wird, die VSIX-Tests ihre Dateien finden statt zu überspringen und eine Regressionsprüfung fehlschlägt, falls sich der Pfad erneut verschiebt.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Beschreibung

enhancement

Description

The VSIX smoke tests resolve the extension resource directory using two parent-directory segments:

path.resolve(__dirname, "..", "..", "resources", "extension");

At runtime, the compiled test is located under:

test/smoke/out

As a result, the current path resolves to:

test/resources/extension

Instead of the intended directory:

test/smoke/resources/extension

This is not detected because the existing tests are skipped when the resolved directory or VSIX does not exist.

Expected behavior

The smoke tests should resolve and validate VSIX files from:

test/smoke/resources/extension

Proposed fix

  • Remove the extra parent-directory segment:
path.resolve(__dirname, "..", "resources", "extension");
  • Ensure the smoke tests use the intended VSIX resource directory.
  • Add a regression check to prevent the path from resolving to the wrong directory.
Vorherrschende Sprache
TypeScript
Sterne
2.7k
Forks
301
Ø Merge
1 T. 3 Std.
Gemergte PRs (30 T.)
30

Entwicklungsumgebung

Erste Schritte

  1. Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
  3. Forken Sie das Repository und arbeiten Sie in einem Branch.
  4. Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.

Mehr aus microsoft/vscode-react-native

Alle Issues in microsoft/vscode-react-native

Ähnliche Issues

Weitere Issues zu TypeScript

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.