Filter by description on findByLabel for password input tests
Nessuno ha ancora preso questa issue.
Valutazione
- Difficoltà
- 4/5
- Tempo stimato
- 3-5 giorni
- Idoneità per principianti
- 45/100
- Tipo di issue
- Funzionalità
- Chiarezza
- Abbastanza chiara
- Stato di attività
- Ferma
- Stack tecnologico
- javascript, react
- Ambito
- frontend, testing-qa
Direzione di ricerca
Inizia con i punti di ingresso delle query getByLabelText e findByLabelText, quindi confronta il loro comportamento con il filtraggio per description descritto per getByRole. Verifica come l’esempio password-input dovrebbe essere associato tramite il suo label e il contenuto di aria-describedby. Il lavoro è completato quando il filtro description richiesto funziona per i password inputs con label senza compromettere le query label esistenti.
Scritto dal modello di indicizzazione a partire dal testo della issue.
Descrizione
Is your feature request related to a problem? Please describe.
I am trying to test that the inputs in my form have correct inline validation errors set as aria descriptions. This works great when the input is simply a textbox:
<Label>
Email
<input type="text' aria-describedby="emailError" />
</Label>
<div id="emailError">Please enter a valid email address</div>
...
expect(findbyRole('textbox', {
name: 'Email',
description: 'Please enter a valid email'
})).toBeInTheDocument();
However, when the input is a password since it has no role I cannot use the description field on getByRole to find the element.
Describe the solution you'd like
This issue recommends using getByLabelText for password inputs, so it would be a lot easier if that query could also search by description.
<Label>
Password
<input type="password' aria-describedby="passwordError" />
</Label>
<div id="passwordError">Please enter your password</div>
...
expect(findByLabelText('Password', {
description: 'Please enter your password'
})).toBeInTheDocument();
Describe alternatives you've considered
Alternatively, if react-testing library could do some magic that means findByRole('password') just works, then that would be a very simple solution and most users probably wouldn't event have to think about it.
For now my tests merely assert that the error exists in the page and ignore the describedby connection.
expect(await screen.findByText('Please enter your new password')).toBeInTheDocument();
Additional context Add any other context or screenshots about the feature
request here.
- Lingua principale
- JavaScript
- Stelle
- 19.7k
- Fork
- 1.2k
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Nessun Dockerfile né file Docker Compose
- Ha un modello di pull request
- Leggi la guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di testing-library/react-testing-library
-
fireEvent.select does not wrap its automatic native focus in actForse già presa @sergioperezcheco l’ha presa 3 giorni fa. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 88/100
-
bug: calling configure() without reactStrictMode resets it to undefined, silently disabling strict modeForse di nuovo libera @suhailopensource l’ha presa 74 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 2/5 1-3 ore Idoneità per principianti 35/100
testing-library/react-testing-library#1466 · 1 commento ·
-
Difficoltà 4/5 3-5 giorni Idoneità per principianti 30/100
testing-library/react-testing-library#1459 · 2 commenti ·
-
perf: optimize container lookup with early exitForse di nuovo libera @Ch-Abhinav-Chowdary l’ha presa 300 giorni fa e non c’è nessuna pull request aperta. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 35/100
testing-library/react-testing-library#1430 · 1 commento ·
-
`fireEvent.mouseEnter` does not forward `relatedTarget` (relatedTarget is the window instead)Forse già presa @swarnim02 l’ha presa 317 giorni fa. Aperta
Difficoltà 3/5 1-2 giorni Idoneità per principianti 55/100
Tutte le issue di testing-library/react-testing-library
Issue simili
-
Tool errors containing cycles or BigInt crash getErrorMessage and replace the original failureApertafactory-active factory-automatic task-bug-reproduction-success task-identify-harness-labels-done task-identify-issue-type-done
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
vercel/ai#22796 · 2 commenti ·
I maintainer di solito rispondono entro 1 giorno
-
documentation
Difficoltà 2/5 Mezza giornata Idoneità per principianti 78/100
facioquo/stock-indicators-dotnet#2300 ·
I maintainer di solito rispondono entro 1 giorno
-
fix(knowledge): list-promotion-candidates prints a stack trace on a malformed claim inventoryApertagood first issue needs-triage priority: low
Difficoltà 2/5 1-3 ore Idoneità per principianti 82/100
melodic-software/claude-code-plugins#6982 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
val() setter uses the first control's type for a mixed selectionForse già presa Una pull request collegata a questa issue è aperta o già unita. Aperta
Difficoltà 2/5 Mezza giornata Idoneità per principianti 85/100
I maintainer di solito rispondono entro 1 giorno
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 75/100
douglascrockford/Misty#35 ·
I maintainer di solito rispondono entro 1 giorno