🐛 Bug: `setup` should be TDD `setup`

[brettz9]

Bug Report Checklist

• I have read and agree to Mocha's Code of Conduct and Contributing Guidelines
• I have searched for related issues and issues with the faq label, but none matched my issue.
• I have 'smoke tested' the code to be tested by running it outside the real test suite to get a better sense of whether the problem is in the code under test, my usage of Mocha, or Mocha itself.
• I want to provide a PR to resolve this

Expected

In doing a final examination of the new ESM export code, I discovered there is one small fix needing to be made.

The method setup should be the TDD setup, not mocha.setup.

Actual

setup is mocha.setup, so setup cannot be used for tdd mode.

Minimal, Reproducible Example

For:

suite('My suite (context)', function () {
  suiteTeardown(() => {
    console.log('after');
  });
  teardown(() => {
    console.log('after each');
  });
  suiteSetup(() => {
    console.log('before');
  });
  setup(() => {
    console.log('before each');
  });
  test('my test (specify)', function () {
    expect(true).to.equal(true);
  });
});

..."before each" does not run.

Versions

node_modules/.bin/mocha --version: 11.7.4

node --version: v22.16.0

Additional Info

No response

[mark-wiemer]

Thanks for the report @brettz9, do you know of a version where the code did behave as you expect?

[brettz9]

No, this is a new feature not yet released (which I submitted as part of #5527 ). I can submit a PR to address it (it's a simple fix), but see you all require the issue to be accepted first.

[mark-wiemer]

Ah, that makes sense. Then yes, please do submit a PR, thank you!