jestjs/jest

Jest should resolve its dependencies from its own location rather than rootDir

Open

#5 913 ouverte le 3 avr. 2018

Voir sur GitHub
 (21 commentaires) (30 réactions) (0 assignés)TypeScript (45 361 stars) (6 653 forks)batch import
ConfirmedHelp WantedPinned

Description

Create React App users bumped into this issue recently. A third party package started depending on jest-validate, and that caused npm to hoist jest-environment-node to the top of the app tree. Unfortunately, Jest resolves environment package from the project root instead of from its own Node module location so as a result, it loads the wrong (hoisted) jest-environment-node.

I think the jest package should resolve any own dependencies from its own location (just like Node resolution works) rather than from the project root folder.

Guide contributeur

Stack technique
javascripttypescriptnodejs
Domaine
tooling
Type d'issue
bug
DifficultéDifficulté estimée pour un nouveau contributeur, de 1 pour un très petit changement à 5 pour un travail expert.
3
Temps estiméFourchette de temps approximative pour investiguer, implémenter, tester et préparer une pull request.
1-3 hours
Statut d'activitéDisponibilité apparente de l'issue : fraîche, active, ancienne, bloquée ou en attente d'un mainteneur.
stale
ClartéClarté avec laquelle l'issue explique le changement attendu, les critères d'acceptation et la prochaine étape.
clear
Prérequis
Node.js module resolutionJest internals
Accessibilité débutantScore de 1 à 100 estimant l'accessibilité de cette issue pour un premier contributeur.
55
Direction de recherche
Examine the resolution logic in packages/jest config/src/utils.js around lines 109-111. Understand how the environment package is resolved from the project root. Investigate using require.resolve from Jest's own directory (e.g., dirname) to fix hoisting issues. Review the linked CRA issue for additional context. Ensure no breaking changes to other dependency resolutions.