Add an option to avoid fixing no-skip-test

docs/rules/no-skip-test.md

@@ -35,3 +35,15 @@ test('bar', t => {
 	t.pass();
 });
 ```
+
+## Options
+
+The rule takes the following options:
+
+* `fix`: whether to fix violations (default true)
+
+You can set the option in configuration like this:
+
+```js
+"ava/no-skip-test": ["error", { fix: false }]
+```

rules/no-skip-test.js

@@ -4,6 +4,8 @@ const createAvaRule = require('../create-ava-rule');
 const util = require('../util');
 
 const create = context => {
+	const [options] = context.options;
+	const {fix = true} = (options || {});
 	const ava = createAvaRule();
 
 	return ava.merge({
@@ -16,26 +18,39 @@ const create = context => {
 				context.report({
 					node: propertyNode,
 					message: 'No tests should be skipped.',
-					fix: fixer => {
-						return fixer.replaceTextRange.apply(null, util.removeTestModifier({
-							modifier: 'skip',
-							node,
-							context
-						}));
-					}
+					...(fix && {
+						fix: fixer => {
+							return fixer.replaceTextRange.apply(null, util.removeTestModifier({
+								modifier: 'skip',
+								node,
+								context
+							}));
+						}
+					})
 				});
 			}
 		})
 	});
 };
 
+const schema = [{
+	type: 'object',
+	properties: {
+		fix: {
+			type: 'boolean',
+			default: true
+		}
+	}
+}];
+
 module.exports = {
 	create,
 	meta: {
 		docs: {
 			url: util.getDocsUrl(__filename)
 		},
 		fixable: 'code',
-		type: 'suggestion'
+		type: 'suggestion',
+		schema
 	}
 };

test/no-skip-test.js

@@ -32,6 +32,19 @@ ruleTester.run('no-skip-test', rule, {
 				column: 6
 			}]
 		},
+		{
+			code: header + 'test.skip(t => { t.pass(); });',
+			output: header + 'test.skip(t => { t.pass(); });',
+			options: [{
+				fix: false
+			}],
+			errors: [{
+				message,
+				type: 'Identifier',
+				line: 2,
+				column: 6
+			}]
+		},
 		{
 			code: header + 'test.cb.skip(t => { t.pass(); t.end(); });',
 			output: header + 'test.cb(t => { t.pass(); t.end(); });',