Fix linting

Author: sindresorhus

create-ava-rule.js

@@ -45,7 +45,9 @@ const avaVariableDeclaratorAst = {
 function isTestFunctionCall(node) {
 	if (node.type === 'Identifier') {
 		return node.name === 'test';
-	} else if (node.type === 'MemberExpression') {
+	}
+
+	if (node.type === 'MemberExpression') {
 		return isTestFunctionCall(node.object);
 	}
 

test/no-skip-assert.js

@@ -16,7 +16,7 @@ ruleTester.run('no-skip-assert', rule, {
 		header + 'test(t => { t.is(1, 1); });',
 		header + 'test.skip(t => { t.is(1, 1); });',
 		header + 'test(t => { notT.skip.is(1, 1); });',
-			// Shouldn't be triggered since it's not a test file
+		// Shouldn't be triggered since it's not a test file
 		'test(t => { t.skip.is(1, 1); });'
 	],
 	invalid: [

test/no-skip-test.js

@@ -19,7 +19,7 @@ ruleTester.run('no-skip-test', rule, {
 		header + 'test(t => { t.pass(); }); test(t => { t.pass(); });',
 		header + 'test(t => { t.skip.is(1, 2); });',
 		header + 'notTest.skip();',
-			// Shouldn't be triggered since it's not a test file
+		// Shouldn't be triggered since it's not a test file
 		'test.skip(t => {});'
 	],
 	invalid: [

test/no-statement-after-end.js

@@ -29,7 +29,7 @@ ruleTester.run('no-statement-after-end', rule, {
 		cbTest('if (t.context.a === 1) { return t.end(); } \n t.is(1, 1); t.end();'),
 		cbTest('return t.end();'),
 		cbTest('t.end(); return;'),
-			// Valid because it is not a test file (no header)
+		// Valid because it is not a test file (no header)
 		cbTest('t.end(); t.is(1, 1);', false)
 	],
 	invalid: [

test/no-todo-test.js

@@ -16,7 +16,7 @@ ruleTester.run('no-todo-test', rule, {
 		header + 'test("my test name", t => { t.pass(); });',
 		header + 'test.only("my test name", t => { t.pass(); });',
 		header + 'notTest.todo(t => { t.pass(); });',
-			// Shouldn't be triggered since it's not a test file
+		// Shouldn't be triggered since it's not a test file
 		'test.todo("my test name");'
 	],
 	invalid: [

test/package.js

@@ -2,16 +2,15 @@ import fs from 'fs';
 import path from 'path';
 import test from 'ava';
 import pify from 'pify';
-import index from '../';
+import index from '..';
 
 test('Every rule is defined in index file', async t => {
 	const ruleDir = path.resolve(__dirname, '../rules');
 	const files = await pify(fs.readdir)(ruleDir);
 
 	const rules = files.filter(file =>
 		file.indexOf('.js') === file.length - 3 &&
-		!fs.readFileSync(path.join(ruleDir, file), 'utf8').includes('deprecated: true')
-	);
+		!fs.readFileSync(path.join(ruleDir, file), 'utf8').includes('deprecated: true'));
 
 	rules.forEach(file => {
 		const name = file.slice(0, -3);

test/test-ended.js

@@ -22,9 +22,9 @@ ruleTester.run('test-ended', rule, {
 		header + 'test.cb.only(t => { t.end(); });',
 		header + 'test.cb.skip.only(t => { t.end(); });',
 		header + 'test.only.cb.skip(t => { t.end(); });',
-			// Shouldn't be triggered since it's not a callback test
+		// Shouldn't be triggered since it's not a callback test
 		header + 'test(t => { t.pass(); });',
-			// Shouldn't be triggered since it's not a test file
+		// Shouldn't be triggered since it's not a test file
 		'test.cb(t => {});'
 	],
 	invalid: [

test/test-title.js

@@ -13,7 +13,7 @@ const header = `const test = require('ava');\n`;
 
 ruleTester.run('test-title', rule, {
 	valid: [
-			// Default options should be `['if-multiple']`
+		// Default options should be `['if-multiple']`
 		header + 'test(t => { t.pass(); t.end(); });',
 		{
 			code: header + 'test("my test name", t => { t.pass(); t.end(); });',
@@ -67,7 +67,7 @@ ruleTester.run('test-title', rule, {
 			code: header + 'test(macroFn, arg1, arg2);',
 			options: ['always']
 		},
-			// Shouldn't be triggered since it's not a test file
+		// Shouldn't be triggered since it's not a test file
 		{
 			code: 'test(t => {});',
 			options: ['always']

test/use-t.js

@@ -29,11 +29,11 @@ ruleTester.run('use-t', rule, {
 		header + 'test(() => {});',
 		header + 'test(t => {});',
 		header + 'test.cb(t => {});',
-			// Header + 'test("test name", t => {});',
+		// Header + 'test("test name", t => {});',
 		header + 'test(function (t) {});',
 		header + 'test(testFunction);',
 		header + 'test.todo("test name");',
-			// Shouldn't be triggered since it's not a test file
+		// Shouldn't be triggered since it's not a test file
 		'test(foo => {});'
 	],
 	invalid: [

test/use-true-false.js

@@ -53,7 +53,7 @@ ruleTester.run('use-true-false', rule, {
 		testCase('t.falsy(value + value)'),
 		testCase('t.truthy()'),
 		testCase('t.falsy()'),
-			// Shouldn't be triggered since it's not a test file
+		// Shouldn't be triggered since it's not a test file
 		testCase('t.truthy(value === 1)', false)
 	],
 	invalid: [