sindresorhus/eslint-plugin-unicorn

consistent-function-scoping should ignore jest mocks

Chiusa

#805 aperta il 4 ago 2020

Vedi su GitHubGuida contributor
 (1 commento) (0 reazioni) (0 assegnatari)JavaScript (468 fork)user submission
enhancementhelp wanted

Metriche repository

Star
 (5022 stelle)
Metriche merge PR
 (Merge medio 4h 30m) (26 PR mergiate in 30 g)

Descrizione

Using jest.mock() does not work properly with consistent-function-scoping as it's not allowed to reference any out of scope variables as mocked function.

ReferenceError: /__tests__/search.spec.tsx: The module factory of `jest.mock()` is not allowed to reference any out-of-scope variables.

jest.mock("next/dynamic", () => () => "dynamic-drawer");

Guida contributor

Direzione di ricerca
Individua l'implementazione della regola consistent function scoping in rules/consistent function scoping.js. Comprendi come controlla le variabili fuori dallo scope. Il problema è che le funzioni factory di jest.mock possono fare riferimento a variabili che sono effettivamente nello scope (a livello di modulo), ma la regola le segnala erroneamente. Cerca dove la regola valuta le espressioni di funzione o i callback e aggiungi un'eccezione per le chiamate jest.mock(). Controlla i test esistenti per questa regola per comprenderne il comportamento atteso.
Tech stack
javascripttypescript
Dominio
toolingtesting
Tipo issue
Bug
DifficoltàQuanto dovrebbe essere impegnativa per un nuovo contributor.
3
Tempo stimatoTempo stimato per completare e verificare uno scope piccolo.
Mezza giornata
Stato attivitàQuanto è probabile che la issue sia ancora attiva e reviewable.
Datata
ChiarezzaQuanto chiaramente la issue descrive problema, scope e risultato atteso.
Abbastanza chiara
Prerequisiti
understanding of ESLint plugin developmentfamiliarity with jest.mock()
Adatta ai principiantiQuanto la issue è adatta a contributor alla prima esperienza.
60