diff options
Diffstat (limited to 'tools/lint/eslint/eslint-plugin-mozilla/lib/configs/xpcshell-test.js')
| -rw-r--r-- | tools/lint/eslint/eslint-plugin-mozilla/lib/configs/xpcshell-test.js | 11 |
1 files changed, 11 insertions, 0 deletions
diff --git a/tools/lint/eslint/eslint-plugin-mozilla/lib/configs/xpcshell-test.js b/tools/lint/eslint/eslint-plugin-mozilla/lib/configs/xpcshell-test.js index 6a4d572911..d3c983999a 100644 --- a/tools/lint/eslint/eslint-plugin-mozilla/lib/configs/xpcshell-test.js +++ b/tools/lint/eslint/eslint-plugin-mozilla/lib/configs/xpcshell-test.js @@ -10,6 +10,16 @@ module.exports = { overrides: [ { + // Some directories have multiple kinds of tests, and some rules + // don't work well for plain mochitests, so disable those. + files: ["*.html", "*.xhtml"], + // plain/chrome mochitests don't automatically include Assert, so + // autofixing `ok()` to Assert.something is bad. + rules: { + "mozilla/no-comparison-or-assignment-inside-ok": "off", + }, + }, + { // If it is a head file, we turn off global unused variable checks, as it // would require searching the other test files to know if they are used or not. // This would be expensive and slow, and it isn't worth it for head files. @@ -44,6 +54,7 @@ module.exports = { "mozilla/import-headjs-globals": "error", "mozilla/mark-test-function-used": "error", "mozilla/no-arbitrary-setTimeout": "error", + "mozilla/no-comparison-or-assignment-inside-ok": "error", "mozilla/no-useless-run-test": "error", "no-shadow": "error", // Turn off no-unsanitized for tests, as we do want to be able to use |