6350d35b3cThe fixer for non-ascii-character does not typidally do the right thing. It removes the entire node, not the offending character. I discovered this when it removed the entire contents of a file and I wasn't sure which auto-fix rule was doing it. This commit adds a minimal test for the rule. The tests require that auto-fix results be supplied, so if someone wants to re-add auto-fixing to the rule, we'll have tests that it does the right thing. PR-URL: https://github.com/nodejs/node/pull/38413 Reviewed-By: Colin Ihrig <cjihrig@gmail.com> Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com> Reviewed-By: James M Snell <jasnell@gmail.com> |
||
|---|---|---|
| .. | ||
| alphabetize-errors.js | ||
| async-iife-no-unused-result.js | ||
| crypto-check.js | ||
| documented-errors.js | ||
| eslint-check.js | ||
| inspector-check.js | ||
| lowercase-name-for-primitive.js | ||
| no-array-destructuring.js | ||
| no-duplicate-requires.js | ||
| no-unescaped-regexp-dot.js | ||
| non-ascii-character.js | ||
| prefer-assert-iferror.js | ||
| prefer-assert-methods.js | ||
| prefer-common-mustnotcall.js | ||
| prefer-common-mustsucceed.js | ||
| prefer-primordials.js | ||
| prefer-util-format-errors.js | ||
| require-common-first.js | ||
| required-modules.js | ||
| rules-utils.js | ||