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]

🦋 Changeset detected

Latest commit: 90cb6c3

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
openzeppelin-solidity	Patch

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another 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.}