Vitest config is dropped when test.server.deps.external already lists solid-js
Maintainer antworten meist innerhalb von 1 Tag
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 78/100
- Issue-Typ
- Bug
- Klarheit
- Klar beschrieben
- Aktivitätsstatus
- Aktiv
- Tech-Stack
- typescript, vite
- Bereich
- testing
Rechercherichtung
Beginne in src/index.ts und lies den Config-Merge rund um ...(test.server ? { test } : {}); vergleiche ihn mit dem im Issue beschriebenen Verhalten des next-Branches. Prüfe die vorhandenen Vitest-Config-Tests oder ergänze eine Reproduktion mit test.server.deps.external, wobei solid-js bereits aufgeführt ist. Fertig ist es, wenn die aufgelöste Config in beiden Fällen weiterhin environment: 'jsdom' und die jest-dom-Setup-Datei enthält.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
On the 2.x line (main), vite-plugin-solid returns its test config only when it also sets test.server. If the user config already has solid-js in test.server.deps.external, the plugin does not set test.server, so it also drops test.environment = 'jsdom' and the @testing-library/jest-dom setup file.
Reproduction
// vitest.config.ts
import { defineConfig } from 'vitest/config';
import solid from 'vite-plugin-solid';
export default defineConfig({
plugins: [solid()],
test: { server: { deps: { external: [/solid-js/] } } },
});
The resolved Vitest config has no environment and no jest-dom setup file, so tests run in the default node environment, where DOM APIs such as document are not defined. Without the server.deps.external entry, the plugin sets both.
Expected behavior
The plugin sets the jsdom environment and the jest-dom setup file whether or not solid-js is already external. The next branch already does this (Object.keys(test).length ? { test } : {}).
Cause
Introduced in #183: ...(test.server ? { test } : {}) in src/index.ts.
- Vorherrschende Sprache
- TypeScript
- Sterne
- 523
- Forks
- 74
- Ø Merge
- 1 T. 19 Std.
- Gemergte PRs (30 T.)
- 54
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
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 solidjs/solid-vite-plugin
-
Test environment detection doesn't consider Vitest workspacesEvtl. vergeben @carloitaben hat das vor 61 Tagen übernommen. Offen1.x
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 78/100
solidjs/solid-vite-plugin#205 · 1 Kommentar · 2 Reaktionen ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 56/100
solidjs/solid-vite-plugin#456 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 5/5 Über eine Woche Anfängerfreundlichkeit 35/100
solidjs/solid-vite-plugin#328 ·
Maintainer antworten meist innerhalb von 1 Tag
-
jsx: 'preserve' in optimizeDeps breaks dependency scan when .tsx files use `import.meta.glob`Evtl. vergeben @brenelz hat das vor 34 Tagen übernommen. Offen1.x
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 72/100
solidjs/solid-vite-plugin#262 · 2 Kommentare ·
Maintainer antworten meist innerhalb von 1 Tag
-
Speed up vite-plugin-solid with vite hook filtersEvtl. vergeben @ChristophP hat das vor 51 Tagen übernommen. Offen1.x
Schwierigkeit 3/5 1-2 Tage Anfängerfreundlichkeit 65/100
solidjs/solid-vite-plugin#259 · 2 Kommentare · 1 Reaktion ·
Maintainer antworten meist innerhalb von 1 Tag
Alle Issues in solidjs/solid-vite-plugin
Ähnliche Issues
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 68/100
cursor/community-plugins#465 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 66/100
vetro-protocol/vetro-monorepo#978 ·
Maintainer antworten meist innerhalb von 1 Tag
-
ai-generated bug category:security effort:s priority:p3
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 67/100
LanternOps/breeze#9697 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 68/100
maniator/verticopolis#1303 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 73/100
components-web-app/docs#297 ·