eslint rule for no async in describe blocks via eslint-plugin-mocha

[gonzaotc]

Fixes #4943

PR Checklist

• Tests
• Documentation
• Changeset entry (run npx changeset add)

[changeset-bot]

⚠️ No Changeset found

Latest commit: 289dd79

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

[socket-security]

Review the following changes in direct dependencies. Learn more about Socket for GitHub.

Diff	Package	Supply Chain Security	Vulnerability	Quality	Maintenance	License
	npm/​eslint-plugin-mocha@​11.2.0

View full report

[coderabbitai]

Walkthrough

This pull request introduces the eslint-plugin-mocha ESLint plugin to enforce linting rules for Mocha test constructs. The plugin is added as a dev dependency, configured in the ESLint configuration with the mocha/no-async-suite rule set to error severity, and a changeset is created to document the patch-level release. Additionally, test files are updated to remove async keywords from Mocha describe blocks to comply with the newly enforced linting rule.

Suggested labels

tests

🚥 Pre-merge checks | ✅ 4 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 0.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (4 passed)

Check name	Status	Explanation
Title check	✅ Passed	The PR title clearly and specifically describes the main change: adding an ESLint rule to prevent async functions in Mocha describe blocks via eslint-plugin-mocha.
Linked Issues check	✅ Passed	The PR fully implements the objective from issue #4943: adds eslint-plugin-mocha dependency, enables the no-async-describe rule in ESLint config, and removes async keywords from describe blocks.
Out of Scope Changes check	✅ Passed	All changes are directly related to the linked issue objective: adding eslint-plugin-mocha, configuring the no-async-describe rule, and updating test files to comply with the new rule.
Description check	✅ Passed	The PR description references the GitHub issue being fixed and includes a changeset entry, directly relating to the changeset additions in the PR.

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}