☂️ eslint-plugin-vitest

[Boshen]

Warning

This comment is maintained by CI. Do not edit this comment directly.
To update comment template, see https://github.com/oxc-project/oxc/tree/main/tasks/lint_rules

This is tracking issue for eslint-plugin-vitest.

There are 51(+ 1 deprecated) rules.

• 1/8 recommended rules are remaining as TODO
  • 1 of which have pending fixes
• 9/43 not recommended rules are remaining as TODO

To get started, run the following command:

just new-vitest-rule <RULE_NAME>

Then register the rule in crates/oxc_linter/src/rules.rs and also declare_all_lint_rules at the bottom.

Recommended rules

✅: 7, 🚫: 0, ⏳: 1 / total: 8

Status	Name
✅	vitest/expect-expect
✅	vitest/no-commented-out-tests
✅	vitest/no-identical-title
✅	vitest/no-import-node-test
✅⏳	vitest/require-local-test-context-for-concurrent-snapshots
✅	vitest/valid-describe-callback
✅	vitest/valid-expect
	vitest/valid-title

✅ = Implemented, 🚫 = No need to implement, ⏳ = Fix pending

Not recommended rules

✅: 34, 🚫: 0, ⏳: 0 / total: 43

Status	Name
	vitest/consistent-test-filename
✅	vitest/consistent-test-it
✅	vitest/max-expects
✅	vitest/max-nested-describe
✅	vitest/no-alias-methods
✅	vitest/no-conditional-expect
✅	vitest/no-conditional-in-test
✅	vitest/no-conditional-tests
✅	vitest/no-disabled-tests
✅	vitest/no-duplicate-hooks
✅	vitest/no-focused-tests
✅	vitest/no-hooks
✅	vitest/no-interpolation-in-snapshots
	vitest/no-large-snapshots
✅	vitest/no-mocks-import
✅	vitest/no-restricted-matchers
	vitest/no-restricted-vi-methods
✅	vitest/no-standalone-expect
✅	vitest/no-test-prefixes
✅	vitest/no-test-return-statement
	vitest/prefer-called-with
✅	vitest/prefer-comparison-matcher
✅	vitest/prefer-each
✅	vitest/prefer-equality-matcher
	vitest/prefer-expect-assertions
✅	vitest/prefer-expect-resolves
✅	vitest/prefer-hooks-in-order
✅	vitest/prefer-hooks-on-top
✅	vitest/prefer-lowercase-title
✅	vitest/prefer-mock-promise-shorthand
	vitest/prefer-snapshot-hint
	vitest/prefer-spy-on
✅	vitest/prefer-strict-equal
✅	vitest/prefer-to-be
✅	vitest/prefer-to-be-falsy
✅	vitest/prefer-to-be-object
✅	vitest/prefer-to-be-truthy
	vitest/prefer-to-contain
✅	vitest/prefer-to-have-length
✅	vitest/prefer-todo
	vitest/require-hook
✅	vitest/require-to-throw-message
✅	vitest/require-top-level-describe

✅ = Implemented, 🚫 = No need to implement, ⏳ = Fix pending

Deprecated rules

✅: 0, 🚫: 1, ⏳: 0 / total: 1

Status	Name
🚫	vitest/no-done-callback

✅ = Implemented, 🚫 = No need to implement, ⏳ = Fix pending

[shulaoda]

Does vitest/no-commented-out-tests seem to be duplicated with jest/no-commented-out-tests?

[Boshen]

We need to map jest to vitest rules in tasks/lint_rules, they have the same implementation.

[shulaoda]

Justfile seems to not have the command just new-vitest-rule. Should we add it or use just new-jest-rule? @Boshen

[mysteryven]

Justfile seems to not have the command just new-vitest-rule. Should we add it or use just new-jest-rule?

Add this is welcome.