Skip to content

Commit

Permalink
fix: Type definitions for findByRole() and findAllByRole() (#173)
Browse files Browse the repository at this point in the history
  • Loading branch information
shibukawa authored Feb 21, 2021
1 parent f08d6ed commit 6344133
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions types/index.d.ts
Original file line number Diff line number Diff line change
Expand Up @@ -3,6 +3,7 @@
import {
configure,
Matcher,
ByRoleMatcher,
MatcherOptions as DTLMatcherOptions,
ByRoleOptions as DTLByRoleOptions,
SelectorMatcherOptions as DTLSelectorMatcherOptions,
Expand Down Expand Up @@ -211,7 +212,7 @@ declare global {
* @see https://github.com/testing-library/cypress-testing-library#usage
* @see https://github.com/testing-library/dom-testing-library#table-of-contents
*/
findByRole(id: Matcher, options?: ByRoleOptions): Chainable<JQuery>
findByRole(id: ByRoleMatcher, options?: ByRoleOptions): Chainable<JQuery>

/**
* dom-testing-library helpers for Cypress
Expand All @@ -222,7 +223,7 @@ declare global {
* @see https://github.com/testing-library/cypress-testing-library#usage
* @see https://github.com/testing-library/dom-testing-library#table-of-contents
*/
findAllByRole(id: Matcher, options?: ByRoleOptions): Chainable<JQuery>
findAllByRole(id: ByRoleMatcher, options?: ByRoleOptions): Chainable<JQuery>

/**
* dom-testing-library helpers for Cypress
Expand Down

0 comments on commit 6344133

Please # to comment.