tidy tests

walaura · walaura · commit 38f5068684cc · 2019-12-05T16:25:42.000Z
diff --git a/scripts/eslint-rules/__tests__/no-production-logging-test.internal.js b/scripts/eslint-rules/__tests__/no-production-logging-test.internal.js
@@ -15,24 +15,24 @@ const ruleTester = new RuleTester();
 
 ruleTester.run('no-production-logging', rule, {
   valid: [
+    /* wrapped calls */
     {
-      code: 'if (__DEV__) {warningWithoutStack(test)}',
+      code: 'if (__DEV__) { warningWithoutStack(test) }',
     },
     {
-      code: 'if (__DEV__) {warning(test)}',
+      code: 'if (__DEV__) { warning(test) }',
     },
+    /* calls wrapped on an outer bound */
     {
       code: 'if (__DEV__) { if (potato) { while (true) { warning(test) }}}',
     },
+    /* don't do anything to arbitrary fn's or invariants */
     {
       code: 'normalFunctionCall(test)',
     },
     {
       code: 'invariant(test)',
     },
-    {
-      code: 'if (__DEV__) {normalFunctionCall(test)}',
-    },
   ],
   invalid: [
     {

Original file line number	Diff line number	Diff line change
`@@ -15,24 +15,24 @@ const ruleTester = new RuleTester();`
`15`	`15`
`16`	`16`	`ruleTester.run('no-production-logging', rule, {`
`17`	`17`	`valid: [`
	`18`	`+ /* wrapped calls */`
`18`	`19`	`{`
`19`		`- code: 'if (__DEV__) {warningWithoutStack(test)}',`
	`20`	`+ code: 'if (__DEV__) { warningWithoutStack(test) }',`
`20`	`21`	`},`
`21`	`22`	`{`
`22`		`- code: 'if (__DEV__) {warning(test)}',`
	`23`	`+ code: 'if (__DEV__) { warning(test) }',`
`23`	`24`	`},`
	`25`	`+ /* calls wrapped on an outer bound */`
`24`	`26`	`{`
`25`	`27`	`code: 'if (__DEV__) { if (potato) { while (true) { warning(test) }}}',`
`26`	`28`	`},`
	`29`	`+ /* don't do anything to arbitrary fn's or invariants */`
`27`	`30`	`{`
`28`	`31`	`code: 'normalFunctionCall(test)',`
`29`	`32`	`},`
`30`	`33`	`{`
`31`	`34`	`code: 'invariant(test)',`
`32`	`35`	`},`
`33`		`- {`
`34`		`- code: 'if (__DEV__) {normalFunctionCall(test)}',`
`35`		`- },`
`36`	`36`	`],`
`37`	`37`	`invalid: [`
`38`	`38`	`{`