Update prefer-power-assert to accept only true (#118)

Author: bdougherty

docs/rules/prefer-power-assert.md

@@ -2,13 +2,13 @@
 
 Translations: [Français](https://github.com/avajs/ava-docs/blob/master/fr_FR/related/eslint-plugin-ava/docs/rules/prefer-power-assert.md)
 
-- [`t.truthy()`](https://github.com/sindresorhus/ava#truthyvalue-message) __(You can do [most things](https://github.com/sindresorhus/ava#enhanced-assertion-messages) with this one)__
-- [`t.deepEqual()`](https://github.com/sindresorhus/ava#deepequalvalue-expected-message)
-- [`t.notDeepEqual()`](https://github.com/sindresorhus/ava#notdeepequalvalue-expected-message)
-- [`t.throws()`](https://github.com/sindresorhus/ava#throwsfunctionpromise-error-message)
-- [`t.notThrows()`](https://github.com/sindresorhus/ava#notthrowsfunctionpromise-message)
-- [`t.pass()`](https://github.com/sindresorhus/ava#passmessage)
-- [`t.fail()`](https://github.com/sindresorhus/ava#failmessage)
+- [`t.true()`](https://github.com/avajs/ava#truevalue-message)
+- [`t.deepEqual()`](https://github.com/avajs/ava#deepequalvalue-expected-message)
+- [`t.notDeepEqual()`](https://github.com/avajs/ava#notdeepequalvalue-expected-message)
+- [`t.throws()`](https://github.com/avajs/ava#throwsfunctionpromise-error-message)
+- [`t.notThrows()`](https://github.com/avajs/ava#notthrowsfunctionpromise-message)
+- [`t.pass()`](https://github.com/avajs/ava#passmessage)
+- [`t.fail()`](https://github.com/avajs/ava#failmessage)
 
 Useful for people wanting to fully embrace the power of [power-assert](https://github.com/power-assert-js/power-assert).
 
@@ -19,7 +19,13 @@ Useful for people wanting to fully embrace the power of [power-assert](https://g
 import test from 'ava';
 
 test(t => {
+	t.truthy(foo);
+	t.falsy(foo);
+	t.false(foo === bar);
 	t.is(foo, bar);
+	t.not(foo, bar);
+	t.regex(foo, bar);
+	t.ifError(error);
 });
 ```
 
@@ -30,6 +36,10 @@ test(t => {
 import test from 'ava';
 
 test(t => {
-	t.truthy(foo === bar);
+	t.true(foo === bar);
+	t.deepEqual(foo, bar);
+	t.notDeepEqual(foo, bar);
+	t.throws(foo);
+	t.notThrows(bar);
 });
 ```

rules/prefer-power-assert.js

@@ -4,8 +4,8 @@ const deepStrictEqual = require('deep-strict-equal');
 const createAvaRule = require('../create-ava-rule');
 
 const notAllowed = [
+	'truthy',
 	'falsy',
-	'true',
 	'false',
 	'is',
 	'not',

test/prefer-power-assert.js

@@ -34,8 +34,8 @@ function testNotAllowedMethod(methodName) {
 }
 
 const notAllowedMethods = [
+	'truthy(foo)',
 	'falsy(foo)',
-	'true(foo)',
 	'false(foo)',
 	'is(foo, bar)',
 	'not(foo, bar)',
@@ -65,11 +65,13 @@ function testAllowedMethod(methodName) {
 }
 
 const allowedMethods = [
-	'truthy(foo)',
+	'true(foo)',
 	'deepEqual(foo, bar)',
 	'notDeepEqual(foo, bar)',
 	'throws(block)',
-	'notThrows(block)'
+	'notThrows(block)',
+	'pass(foo)',
+	'fail(foo)'
 ];
 
 for (const methodName of allowedMethods) {
@@ -81,12 +83,12 @@ function testWithModifier(modifier) {
 		ruleTester.run('prefer-power-assert', rule, {
 			valid: [
 				{
-					code: `import test from 'ava';\n test.${modifier}(t => { t.truthy(foo); });`
+					code: `import test from 'ava';\n test.${modifier}(t => { t.true(foo); });`
 				}
 			],
 			invalid: [
 				{
-					code: `import test from 'ava';\n test.${modifier}(t => { t.falsy(foo); });`,
+					code: `import test from 'ava';\n test.${modifier}(t => { t.is(foo); });`,
 					errors
 				}
 			]
@@ -103,12 +105,12 @@ function testDeclaration(declaration) {
 		ruleTester.run('prefer-power-assert', rule, {
 			valid: [
 				{
-					code: `${declaration}\n test(t => { t.truthy(foo); });`
+					code: `${declaration}\n test(t => { t.true(foo); });`
 				}
 			],
 			invalid: [
 				{
-					code: `${declaration}\n test(t => { t.falsy(foo); });`,
+					code: `${declaration}\n test(t => { t.is(foo); });`,
 					errors
 				}
 			]
@@ -153,7 +155,7 @@ test('misc', () => {
 	ruleTester.run('prefer-power-assert', rule, {
 		valid: [
 			{
-				code: `import test from 'ava';\n test.cb(function (t) { t.truthy(foo); t.end(); });`
+				code: `import test from 'ava';\n test.cb(function (t) { t.true(foo); t.end(); });`
 			},
 			// shouldn't be triggered since it's not a test file
 			{