chore(test): use t.mock.module for mocking

[avivkeller]

Description

I kept wondering why, whenever I attempted to mock these modules with node:test mocking, I would always get an error that they weren't defined. Then it hit me: next-router isn't defined, because we don't use it 🤦.

So, this PR removes the mock for next-router, and uses node:test for mocking next-intl.

Validation

All tests should pass

Related Issues

Ref: https://github.com/nodejs/nodejs.org/pull/7623/files#r2032206240

Check List

• I have read the Contributing Guidelines and made commit messages that follow the guideline.
• I have run npm run format to ensure the code follows the style guide.
• I have run npm run test to check if all tests are passing.
• I have run npx turbo build to check if the website builds without errors.
• I've covered new added functionality with unit tests if necessary.

[vercel]

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name	Status	Preview	Updated (UTC)
nodejs-org	✅ Ready (Inspect)	Visit Preview	Apr 17, 2025 6:52pm

[Copilot]

Pull Request Overview

This PR cleans up the test harness by removing the unused mocks for next-router and next-intl and switching to using node:test's built-in mocking via t.module.mock. The changes include removing legacy mock files and loader modifications, and introducing a new setup.jsx file that sets up mocks for next-intl and next-intl/navigation.

Reviewed Changes

Copilot reviewed 5 out of 6 changed files in this pull request and generated no comments.

Show a summary per file

File	Description
apps/site/tests/setup.mjs	Removed the legacy setup that registered a custom module loader.
apps/site/tests/setup.jsx	Added new mocking setup using node:test's t.module.mock for next-intl and its navigation.
apps/site/tests/mocks/next-router.mjs	Removed the unused next-router mock.
apps/site/tests/mocks/next-intl.jsx	Removed the legacy next-intl mock file.
apps/site/tests/loader.mjs	Removed the custom module resolution hook, now unnecessary with t.module.mock.

Files not reviewed (1)

• apps/site/package.json: Language not supported

[codecov-commenter]

Codecov Report

Attention: Patch coverage is 96.42857% with 1 line in your changes missing coverage. Please review.

Project coverage is 74.61%. Comparing base (162cd26) to head (e81241f).
Report is 1 commits behind head on main.

✅ All tests successful. No failed tests found.

Files with missing lines	Patch %	Lines
apps/site/tests/setup.jsx	96.42%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##             main    #7660      +/-   ##
==========================================
+ Coverage   74.54%   74.61%   +0.06%     
==========================================
  Files          96       96              
  Lines        7673     7685      +12     
  Branches      194      192       -2     
==========================================
+ Hits         5720     5734      +14     
+ Misses       1951     1949       -2     
  Partials        2        2              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

[ovflowd]

Feel free to fast-track this. Giving a 👍

[github-actions]

Lighthouse Results

URL	Performance	Accessibility	Best Practices	SEO	Report
/en	🟢 99	🟢 100	🟢 100	🟢 91	🔗
/en/about	🟢 100	🟢 100	🟢 100	🟢 91	🔗
/en/about/previous-releases	🟢 99	🟢 100	🟢 100	🟢 92	🔗
/en/download	🟢 99	🟢 100	🟢 100	🟢 91	🔗
/en/blog	🟢 100	🟢 100	🟢 96	🟢 92	🔗