fix(rule-tester): handle window root path

[yeonjuan]

PR Checklist

• Addresses an existing open issue: fixes #000 (There is no open issue, let me know if I should create one.)
• That issue was marked as accepting prs
• Steps in Contributing were taken

Overview

While testing, I accidentally raised the wrong PRs to this repository. Sorry for the unnecessary notifications. 😢
Eventually, I figured out the cause using a PC with window OS that I hadn't used in a long time. 😄

[typescript-eslint]

Thanks for the PR, @yeonjuan!

typescript-eslint is a 100% community driven project, and we are incredibly grateful that you are contributing to that community.

The core maintainers work on this in their personal time, so please understand that it may not be possible for them to review your work immediately.

Thanks again!

---

🙏 Please, if you or your company is finding typescript-eslint valuable, help us sustain the project by sponsoring it transparently on https://opencollective.com/typescript-eslint.

[netlify]

✅ Deploy Preview for typescript-eslint ready!

Name	Link
🔨 Latest commit	ca95619
🔍 Latest deploy log	https://app.netlify.com/sites/typescript-eslint/deploys/67869e863c45d10008d9a721
😎 Deploy Preview	https://deploy-preview-10654--typescript-eslint.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.
Lighthouse	1 paths audited Performance: 97 (🔴 down 1 from production) Accessibility: 100 (no change from production) Best Practices: 92 (no change from production) SEO: 98 (no change from production) PWA: 80 (no change from production) View the detailed breakdown and full score reports

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[nx-cloud]

View your CI Pipeline Execution ↗ for commit ca95619.

Command	Status	Duration	Result
nx run-many --target=build --exclude website --...	✅ Succeeded	2s	View ↗
nx run-many --target=clean	✅ Succeeded	12s	View ↗

---

☁️ Nx Cloud last updated this comment at 2025-01-14 17:48:03 UTC

[yeonjuan]

If a window file path format is used in the filename of the test case, such as C:\\, the basePath must be changed so that the linter can find the cofnig file.

typescript-eslint/packages/eslint-plugin-internal/tests/rules/debug-namespace.test.ts

Lines 44 to 46 in ea6fbea

	],
	filename: 'C:\\Code\\typescript-eslint\\packages\\example\\file.ts',
	output: "const log = debug('typescript-eslint:example:file');",

[yeonjuan]

BTW, the value of the messages when Linter does not find a config for a given filename is as follows

[
 { "ruleId": null, "severity": 1, "message": "No matching configuration found for C:\\Code....", "line": 0, "column": 0 }
]

@ronami, I think it would be nice to print an error message when this happens, or have an assertion for that. What do you think? If you agree, I'll open a new issue.

I'm thinking of adding an assertion or logging before this.

typescript-eslint/packages/rule-tester/src/RuleTester.ts

Lines 997 to 999 in ea6fbea

	const hasMessageOfThisRule = messages.some(m => m.ruleId === ruleName);

[JoshuaKGoldberg]

FWIW I think I'm in favor 🙂

[ronami]

Thanks! ❤️

This would help make the main branch green.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 87.00%. Comparing base (a175189) to head (ca95619).
Report is 15 commits behind head on main.

Additional details and impacted files

@@           Coverage Diff           @@
##             main   #10654   +/-   ##
=======================================
  Coverage   87.00%   87.00%           
=======================================
  Files         447      448    +1     
  Lines       15586    15618   +32     
  Branches     4542     4551    +9     
=======================================
+ Hits        13560    13589   +29     
- Misses       1671     1673    +2     
- Partials      355      356    +1     

Flag	Coverage Δ
unittest	87.00% <ø> (+<0.01%)	⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Files with missing lines	Coverage Δ
packages/rule-tester/src/RuleTester.ts	74.57% <ø> (ø)

... and 1 file with indirect coverage changes

[JoshuaKGoldberg]

LGTM, thanks!

shakes fist at Windows paths