test: move to vitest
#269
test: move to vitest
#269
Conversation
test/test-utils.ts
Outdated
| @@ -134,6 +144,7 @@ function analyze(ast, parserOptions) { | |||
| key !== "tokens" && | |||
| key !== "trailingComments" && | |||
| typeof value === "object" && | |||
| // @ts-expect-error -- ignore | |||
| (typeof value.type === "string" || Array.isArray(value)) | |||
There was a problem hiding this comment.
It seems value is always null
There was a problem hiding this comment.
Should I update the logic here? It seems that this part is currently dead code.
There was a problem hiding this comment.
Yeah. I think getFallbackKeys should probably always return [].
However, it's fine if you fix it in this PR or we fix it in another PR.
There was a problem hiding this comment.
I found similar logic in src:
vue-eslint-parser/src/ast/traverse.ts
Line 39 in 62321db
I think we can fix it in another PR with more discussions.
There was a problem hiding this comment.
I think that src/ast/traverse.ts's fallbackKeysFilter is correct as a process. It assigns to value at L49 😄
vue-eslint-parser/src/ast/traverse.ts
Line 49 in 62321db
There was a problem hiding this comment.
But, I think it might be a good idea to refactor it to make it easier to read.
|
Thank you for working on migrating the testing infrastructure!!! |
|
I've updated some of the type definitions in Additionally, I used some |
No description provided.