Setup for custom queries clashes with AllByAttribute type
Maintainer antworten meist innerhalb von 4 Tagen
Dieses Issue hat noch niemand übernommen.
Bewertung
- Schwierigkeit
- 2/5
- Geschätzter Aufwand
- 1-3 Stunden
- Anfängerfreundlichkeit
- 48/100
- Issue-Typ
- Dokumentation
- Klarheit
- Größtenteils klar
- Aktivitätsstatus
- Veraltet
- Tech-Stack
- typescript
- Bereich
- documentation
Rechercherichtung
Beginne mit der im Issue verlinkten Seite zur Einrichtung benutzerdefinierter Abfragen und vergleiche deren JavaScript-Beispiel mit der gemeldeten TypeScript-Konvertierung. Aktualisiere das Beispiel so, dass die benutzerdefinierte Abfrage dem dokumentierten Typ und der Verwendung von buildQueries entspricht, und überprüfe anschließend, dass die TypeScript-Version den gemeldeten Typfehler nicht mehr erzeugt.
Vom Indexierungsmodell aus dem Issue-Text verfasst.
Beschreibung
Describe the bug
https://testing-library.com/docs/react-testing-library/setup#add-custom-queries
Existing setup for custom queries documentation does not work with typescript
To Reproduce
converting the following to typescript
const queryAllByDataCy = (...args) =>
queryHelpers.queryAllByAttribute('data-cy', ...args)
const queryAllByDataCy: AllByAttribute = (_, ...args) =>
queryHelpers.queryAllByAttribute('data-cy', ...args)
Produces a type error in the buildQueries function
type error
Argument of type 'AllByAttribute' is not assignable to parameter of type 'GetAllBy<[container: HTMLElement, id: Matcher, options?: MatcherOptions | undefined]>'.
Types of parameters 'attribute' and 'container' are incompatible.
Type 'HTMLElement' is not assignable to type 'string'.
buildQueries(queryAllByDataCy, getMultipleError, getMissingError)
Expected behavior
When converting the provided example to typescript the setup example should be in a format that conforms to the libraries type AllByAttribute.
This can be done by using partial application the container argument on queryHelpers.queryAllByAttribute and then updating the type of AllByAttribute to omit the partial argument
e.g.
js only:
const queryAllByDataCy = queryHelpers.queryAllByAttribute,bind(null, 'data-cy')
ts only:
type QueryAllByDataCy = (
container: HTMLElement,
id: Matcher,
options?: MatcherOptions
) => HTMLElement[]
const queryAllByDataCy: QueryAllByDataCy = queryHelpers.queryAllByAttribute,bind(null, 'data-cy')
Screenshots
If applicable, add screenshots to help explain your problem.
Desktop (please complete the following information):
- OS: Ubuntu 20.04.2 LTS
- Typescript ^4.2.4
- Node 15.14.0
- Vorherrschende Sprache
- JavaScript
- Sterne
- 479
- Forks
- 738
- Ø Merge
- 5 T. 1 Std.
- Gemergte PRs (30 T.)
- 2
Entwicklungsumgebung
- Kein Dockerfile und keine Docker-Compose-Datei
- Keine Pull-Request-Vorlage
- Beitragsleitfaden lesen
Erste Schritte
- Lesen Sie das ganze Issue und danach den Beitragsleitfaden des Projekts.
- Schreiben Sie ins Issue, dass Sie es übernehmen — das erspart doppelte Arbeit.
- Forken Sie das Repository und arbeiten Sie in einem Branch.
- Öffnen Sie einen Pull Request, der die Issue-Nummer nennt.
Mehr aus testing-library/testing-library-docs
-
Broken Chrome Extension Link in Queries DocsEvtl. vergeben @rajanpanth hat das vor 61 Tagen übernommen. Offen
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 72/100
testing-library/testing-library-docs#1499 ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Schwierigkeit 4/5 3-5 Tage Anfängerfreundlichkeit 48/100
testing-library/testing-library-docs#1528 ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Framework wrappers and DOM testing libraryEvtl. vergeben @jacklaurencegaray hat das vor 423 Tagen übernommen. Offen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 38/100
testing-library/testing-library-docs#1493 · 3 Kommentare ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 55/100
testing-library/testing-library-docs#1486 · 1 Reaktion ·
Maintainer antworten meist innerhalb von 4 Tagen
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 48/100
testing-library/testing-library-docs#1485 · 1 Kommentar ·
Maintainer antworten meist innerhalb von 4 Tagen
Alle Issues in testing-library/testing-library-docs
Ähnliche Issues
-
app_installations reports an error on every repo or suborg change when the plugin is not configuredOffen
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 64/100
github-community-projects/safe-settings#1111 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 75/100
Maintainer antworten meist innerhalb von 1 Tag
-
needs triage
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 82/100
pbakaus/impeccable#1055 ·
Maintainer antworten meist innerhalb von 1 Tag
-
Schwierigkeit 1/5 Unter einer Stunde Anfängerfreundlichkeit 82/100
guaguastandup/zotero-pdf2zh#368 ·
-
Add: EUROTV NEWSOffenchannels:add check:passed
Schwierigkeit 2/5 1-3 Stunden Anfängerfreundlichkeit 70/100