Vitest config is dropped when test.server.deps.external already lists solid-js
Mantenedores costumam responder em até 1 dia
Avaliação
- Dificuldade
- 2/5
- Tempo estimado
- 1-3 horas
- Facilidade para iniciantes
- 78/100
- Tipo de issue
- Bug
- Clareza
- Claramente especificada
- Status de atividade
- Ativa
- Stack de tecnologia
- typescript, vite
- Domínio
- testing
Direção de pesquisa
Comece em src/index.ts e leia a mesclagem da configuração em torno de ...(test.server ? { test } : {}); compare-a com o comportamento do branch next descrito na issue. Verifique os testes existentes da configuração do Vitest ou adicione uma reprodução usando test.server.deps.external, com solid-js já listado. O trabalho estará concluído quando a configuração resolvida ainda incluir environment: 'jsdom' e o arquivo de configuração do jest-dom em ambos os casos.
Escrita pelo modelo de indexação a partir do texto da issue.
Descrição
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.
- Linguagem predominante
- TypeScript
- Estrelas
- 522
- Forks
- 73
- Merge médio
- 1d 8h
- PRs com merge (30d)
- 31
Preparar o ambiente
- Sem Dockerfile nem arquivo Docker Compose
- Sem modelo de pull request
- Ler o guia de contribuição
Primeiros passos
- Leia a issue inteira e depois o guia de contribuição do projeto.
- Comente na issue dizendo que vai assumir — evita que duas pessoas façam o mesmo trabalho.
- Faça um fork do repositório e trabalhe em uma branch.
- Abra um pull request que referencie o número da issue.
Mais de solidjs/solid-vite-plugin
-
Test environment detection doesn't consider Vitest workspacesTalvez já em andamento @carloitaben assumiu há 60 dias. Aberta1.x
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 78/100
solidjs/solid-vite-plugin#205 · 1 comentário · 2 reações ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 4/5 3-5 dias Facilidade para iniciantes 56/100
solidjs/solid-vite-plugin#456 · 3 comentários ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 58/100
solidjs/solid-vite-plugin#428 · 1 comentário ·
Mantenedores costumam responder em até 1 dia
-
Dificuldade 3/5 1-2 dias Facilidade para iniciantes 58/100
solidjs/solid-vite-plugin#427 ·
Mantenedores costumam responder em até 1 dia
-
Dev: a stylesheet shared by the entry and a lazy component moves after the lazy component's stylesAberta
Dificuldade 3/5 Meio dia Facilidade para iniciantes 62/100
solidjs/solid-vite-plugin#425 ·
Mantenedores costumam responder em até 1 dia
Todas as issues de solidjs/solid-vite-plugin
Issues semelhantes
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
phivk/rubiks-lab#33 ·
Mantenedores costumam responder em até 1 dia
-
BUG:Edit Album strips spaces from the password so albums with a leading/trailing space in the password can't be editedTalvez já em andamento @ManyaJain111 assumiu hoje. Abertabug enhancement possible-duplicate
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 77/100
AOSSIE-Org/PictoPy#1637 · 4 comentários ·
Mantenedores costumam responder em até 2 dias
-
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 68/100
chaitin/MonkeyCode#1437 ·
Mantenedores costumam responder em até 1 dia
-
rtdb-info-connected-order test: unhandled 'connection is closed' rejection fails the CLI suiteAberta
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 72/100
Mantenedores costumam responder em até 1 dia
-
area:cards enhancement migration ready-for-agent
Dificuldade 2/5 1-3 horas Facilidade para iniciantes 70/100
fil-donadoni/tolaria#6309 ·
Mantenedores costumam responder em até 1 dia