DeprecationWarning: in testRunner

[Luxcium]

Bug Report

🔎 Search Terms

Mocha, Runner

🕗 Version & Regression Information

on the branch, main with mocha out of sync with is definition 9.0.2/8.2

💻 Code

running npm test in main branch

🙁 Actual behavior

[Worker XX] (node:386YYY) DeprecationWarning: "Runner(suite: Suite, delay: boolean)" is deprecated. Use "Runner(suite: Suite, {delay: boolean})" instead. (Use 'node --trace-deprecation ...' to show where the warning was created)

🙂 Expected behavior

• Code is up-to-date with the main branch
• Successfully run gulp runtests locally
• Successfully run gulp tests & gulp runtests-parallel --light=false locally
• fixed by Fix(DeprecationWarning): Work around in testRunner #44923
• fixed by Fix(DeprecationWarning): Work around in testRunner #44925

[Luxcium]

In Mocha the constructor was updated here on line 127/129

From function Runner(suite, delay) to function Runner(suite, opts)
which was around version 7.1.0

Therefore as @sandersn suggested this is something that should be first fixed in the type definition... and then this issue could be resolved without using a « workaround »

The @type/mocha related to this issue is located here on ine 1403

― Also posted by @Luxcium in #44925 (comment)

― and posted in (DefinitelyTyped/DefinitelyTyped#54416)

[jakebailey]

This was fixed in #46160.