Test environment detection doesn't consider Vitest workspaces
Les mainteneurs répondent en général sous 1 jour
Évaluation
- Difficulté
- 2/5
- Temps estimé
- 1-3 heures
- Accessibilité débutants
- 78/100
- Type d'issue
- Bug
- Clarté
- Clairement spécifiée
- Activité
- Active
- Stack technique
- typescript, vite
- Domaine
- testing-qa
Piste de recherche
Commencez aux lignes 213-224 de src/index.ts et comparez la détection de l’environnement de test du plugin avec la configuration test.workspace dans la reproduction liée. Vérifiez que les environnements basés sur un workspace sont respectés et que le workaround dans vite.config.ts n’est plus nécessaire, notamment en évitant la demande répétée d’installation de jsdom.
Rédigé par le modèle d'indexation à partir du texte de l'issue.
Description
Hello! I am integrating the new Vitest Browser mode into a project using Solid. Their guide suggests that if you want to use it alongside other test environments, you should configure it using workspaces.
The Solid Vite plugin does not appear to read the user test.workspace configuration and defaults to jsdom.
// ...
const userTest = (userConfig as any).test ?? {};
const test = {} as any;
if (userConfig.mode === 'test') {
// ...
if (!userTest.environment && !options.ssr) {
test.environment = 'jsdom';
}
// ...
Which results on being prompted to install jsdom each time
Reproduction
I made a quick reproduction here.
Workaround
My current workaround is to do this on my vite.config.ts:
import { defineConfig } from "vite"
import solid from "vite-plugin-solid"
import tsConfigPaths from "vite-tsconfig-paths"
export default defineConfig({
plugins: [tsConfigPaths(), solid()],
test: {
// Workaround for Solid Vite plugin. Get's superseeded by `workspace` config
environment: "node",
workspace: [
{
extends: true,
test: {
include: ["**/*.test.{ts,tsx}"],
name: "Node.js",
environment: "node",
},
},
{
extends: true,
test: {
include: ["**/*.test.dom.{ts,tsx}"],
browser: {
enabled: true,
headless: true,
ui: false,
provider: "playwright",
instances: [
{
browser: "chromium",
name: "DOM",
},
],
},
},
},
],
},
})
Could the Solid Vite plugin also check the user test.workspace configuration? 🙂 I can draft a PR if the request makes sense
- Langage dominant
- TypeScript
- Étoiles
- 522
- Forks
- 72
- Merge moyen
- 1 j 3 h
- PR mergées (30 j)
- 38
Préparer son environnement
- Aucun Dockerfile ni fichier Docker Compose
- Aucun modèle de pull request
- Lire le guide de contribution
Par où commencer
- Lisez l'issue en entier, puis le guide de contribution du projet.
- Signalez en commentaire que vous la prenez — cela évite que deux personnes fassent le même travail.
- Forkez le dépôt et travaillez sur une branche.
- Ouvrez une pull request qui référence le numéro de l'issue.
Autres issues de solidjs/solid-vite-plugin
-
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
solidjs/solid-vite-plugin#369 · 3 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 5/5 Plus d'une semaine Accessibilité débutants 35/100
solidjs/solid-vite-plugin#328 ·
Les mainteneurs répondent en général sous 1 jour
-
Retain $ServerOnly templates while HMR is activePeut-être pris @brenelz l’a pris il y a 24 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 68/100
solidjs/solid-vite-plugin#308 ·
Les mainteneurs répondent en général sous 1 jour
-
jsx: 'preserve' in optimizeDeps breaks dependency scan when .tsx files use `import.meta.glob`Peut-être pris @brenelz l’a pris il y a 24 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 72/100
solidjs/solid-vite-plugin#262 · 2 commentaires ·
Les mainteneurs répondent en général sous 1 jour
-
Speed up vite-plugin-solid with vite hook filtersPeut-être pris @ChristophP l’a pris il y a 41 jours. Ouverte
Difficulté 3/5 1-2 jours Accessibilité débutants 65/100
solidjs/solid-vite-plugin#259 · 2 commentaires · 1 réaction ·
Les mainteneurs répondent en général sous 1 jour
Toutes les issues de solidjs/solid-vite-plugin
Issues similaires
-
./collections should export setUmbracoVersion and setAllowFilePathUploads for in-process consumersOuverte
Difficulté 2/5 1-3 heures Accessibilité débutants 78/100
umbraco/Umbraco-CMS-MCP-Dev#563 ·
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 65/100
Les mainteneurs répondent en général sous 1 jour
-
Difficulté 2/5 1-3 heures Accessibilité débutants 76/100
-
afk-ok area:data-quality importer size:S
Difficulté 2/5 1-3 heures Accessibilité débutants 77/100
enorm-labs/event-junkie#3166 ·
Les mainteneurs répondent en général sous 1 jour
-
enhancement
Difficulté 1/5 Moins d'une heure Accessibilité débutants 80/100
alunduil/projects-v2-sync#21 ·