Improve types to understand arrays of arrays

[bahmutov]

it.each([
  [1, 'foo'],
  [2, 'bar']
])('title', (a, b) => {
  // a should be a number
  // b should be a string
})

[DlgSHi]

@bahmutov Hi Gleb, is any chance to merge this fix? Or introduce types? Thanks in advance

[bahmutov]

I added an example spec showing the problem. The solution in https://github.com/lastType/cypress-each/pull/1/files solves type check in that example, but breaks every other spec :( if you run npm run lint.

[DlgSHi]

@bahmutov maybe it is because the simple typo ?

(this:Contexte,arg1:any,arg2:any)

instead of

(this:Context,arg1:any,arg2:any)

Context without e

[DlgSHi]

I've tried to apply that change and:
just added new type.

type TestCallback4<T extends readonly any[]> = T extends []
    ? (this: Context, arg1: any, arg2: any) => void
    : Parameters<(...res: [...T, any, any]) => void> extends [...infer R]
    ? R extends readonly [...T, any, any]
      ? (this: Context, ...res: [...R]) => void
      : never
    : never

Applied all other changes suggested, and removed cypress/**/*.js file from type check (why the *.js file should be there?) and it is working as expected :)

[bahmutov]

Mind opening a pr with the fix?Sent from my iPhoneOn Apr 25, 2023, at 08:53, Igor Dlugosh ***@***.***> wrote: I've tried to apply that change and: just added new type. type TestCallback4<T extends readonly any[]> = T extends [] ? (this: Context, arg1: any, arg2: any) => void : Parameters<(...res: [...T, any, any]) => void> extends [...infer R] ? R extends readonly [...T, any, any] ? (this: Context, ...res: [...R]) => void : never : never Applied all other changes suggested, and removed cypress/**/*.js file from type check (why the *.js file should be there?) and it is working as expected :) —Reply to this email directly, view it on GitHub, or unsubscribe.You are receiving this because you were mentioned.Message ID: ***@***.***>

[DlgSHi]

Created #75

[bahmutov]

done