Enabled no-skipped-tests eslint rule [warn]

- Not sure why, but this is set to off in our base plugin
- It masks issues where people have temporarily skipped tests to fix later and then forgot
- Enabling this to allow us to review all those places, but we should also try to leave it on and set to error
This commit is contained in:
Hannah Wolfe 2021-05-07 19:21:15 +01:00
parent 67ea6c44ef
commit 895bffec1f
No known key found for this signature in database
GPG Key ID: 9F8C7532D0A6BA55

View File

@ -23,6 +23,8 @@ module.exports = {
'no-unused-vars': 'off', 'no-unused-vars': 'off',
'no-useless-escape': 'off', 'no-useless-escape': 'off',
'ghost/mocha/no-skipped-tests': 'warn',
// TODO: remove these custom rules and fix problems in test files // TODO: remove these custom rules and fix problems in test files
'ghost/mocha/max-top-level-suites': 'off', 'ghost/mocha/max-top-level-suites': 'off',
'ghost/mocha/no-identical-title': 'off', 'ghost/mocha/no-identical-title': 'off',