fix(types): make third and fourth params optional for findAllBy and findBy built queries

Author: zorfling

types/__tests__/type-tests.ts

@@ -80,6 +80,11 @@ async function testQueryHelpers() {
   getByAutomationId(element, ['id', 'automationId'])
   findAllByAutomationId(element, 'id', {}, {timeout: 1000})
   findByAutomationId(element, 'id', {}, {timeout: 1000})
+  // test optional params too
+  findAllByAutomationId(element, 'id', {})
+  findByAutomationId(element, 'id', {})
+  findAllByAutomationId(element, 'id')
+  findByAutomationId(element, 'id')
 }
 
 async function testByRole() {

types/query-helpers.d.ts

@@ -1,5 +1,5 @@
-import { Matcher, MatcherOptions } from './matches'
-import { waitForOptions } from './wait-for'
+import {Matcher, MatcherOptions} from './matches'
+import {waitForOptions} from './wait-for'
 
 export interface SelectorMatcherOptions extends MatcherOptions {
   selector?: string
@@ -39,12 +39,12 @@ export type GetAllBy<Arguments extends any[]> = QueryMethod<
   HTMLElement[]
 >
 export type FindAllBy<Arguments extends any[]> = QueryMethod<
-  [Arguments[0], Arguments[1], waitForOptions],
+  [Arguments[0], Arguments[1]?, waitForOptions?],
   Promise<HTMLElement[]>
 >
 export type GetBy<Arguments extends any[]> = QueryMethod<Arguments, HTMLElement>
 export type FindBy<Arguments extends any[]> = QueryMethod<
-  [Arguments[0], Arguments[1], waitForOptions],
+  [Arguments[0], Arguments[1]?, waitForOptions?],
   Promise<HTMLElement>
 >