test: Extend scope of eslint to __tests__

Author: tmcw

.eslintignore

@@ -1,2 +1,5 @@
-test/fixture/*
-src/default_theme/assets/*
+__tests__/fixture/*
+src/default_theme/*
+coverage/*
+lib/*
+declarations/*

.eslintrc

@@ -20,7 +20,7 @@
     "no-throw-literal": 2,
     "no-self-compare": 2,
     "no-void": 2,
-    "no-unused-vars": 2,
+    "no-unused-vars": 1,
     "wrap-iife": 2,
     "no-eq-null": 2,
     "strict": [2, "global"],
@@ -31,6 +31,7 @@
   },
   "extends": [
     "eslint:recommended",
+    "plugin:flowtype/recommended",
     "prettier"
   ],
   "env": {

__tests__/__snapshots__/bin-readme.js.snap

@@ -0,0 +1,59 @@
+// Jest Snapshot v1, https://goo.gl/fbAQLP
+
+exports[`readme command --readme-file 1`] = `
+"# A title
+
+# API
+
+<!-- Generated by documentation.js. Update this documentation by updating the source code. -->
+
+## foo
+
+A function with documentation.
+
+**Parameters**
+
+-   \`a\`  {string} blah
+
+Returns **[number](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number)** answer
+
+## bar
+
+A second function with docs
+
+**Parameters**
+
+-   \`b\`  
+
+# Another section
+"
+`;
+
+exports[`readme command updates README.md 1`] = `
+"# A title
+
+# API
+
+<!-- Generated by documentation.js. Update this documentation by updating the source code. -->
+
+## foo
+
+A function with documentation.
+
+**Parameters**
+
+-   \`a\`  {string} blah
+
+Returns **[number](https://developer.mozilla.org/en-US/docs/Web/JavaScript/Reference/Global_Objects/Number)** answer
+
+## bar
+
+A second function with docs
+
+**Parameters**
+
+-   \`b\`  
+
+# Another section
+"
+`;

__tests__/__snapshots__/test.js.snap

@@ -42,7 +42,7 @@ Array [
       "loc": SourceLocation {
         "end": Position {
           "column": 1,
-          "line": 13,
+          "line": 12,
         },
         "start": Position {
           "column": 0,
@@ -178,11 +178,11 @@ Array [
       "loc": SourceLocation {
         "end": Position {
           "column": 2,
-          "line": 12,
+          "line": 10,
         },
         "start": Position {
           "column": 0,
-          "line": 9,
+          "line": 7,
         },
       },
     },
@@ -244,11 +244,11 @@ Array [
     "loc": SourceLocation {
       "end": Position {
         "column": 3,
-        "line": 8,
+        "line": 6,
       },
       "start": Position {
         "column": 0,
-        "line": 5,
+        "line": 3,
       },
     },
     "members": Object {

__tests__/bin-readme.js

@@ -1,4 +1,3 @@
-'use strict';
 var path = require('path'),
   os = require('os'),
   exec = require('child_process').exec,
@@ -55,20 +54,10 @@ describe('readme command', function() {
     });
   });
 
-  var expectedFile = path.join(fixtures, 'README.output.md');
-  var expectedPath = path.join(fixtures, 'README.output.md');
-  var expected = fs.readFileSync(expectedFile, 'utf-8');
-
   test('updates README.md', function() {
     return documentation(['readme index.js -s API'], { cwd: d }).then(() => {
       var outputPath = path.join(d, 'README.md');
-
-      if (UPDATE) {
-        fs.writeFileSync(expectedPath, fs.readFileSync(outputPath, 'utf-8'));
-      }
-
-      var actual = fs.readFileSync(outputPath, 'utf-8');
-      expect(actual).toEqual(expected);
+      expect(fs.readFileSync(outputPath, 'utf-8')).toMatchSnapshot();
     });
   });
 
@@ -80,12 +69,8 @@ describe('readme command', function() {
     return documentation(['readme index.js -s API --readme-file other.md'], {
       cwd: d
     }).then(() => {
-      var actualPath = path.join(d, 'other.md');
-      if (UPDATE) {
-        fs.writeFileSync(actualPath, expected);
-      }
-      var actual = fs.readFileSync(actualPath, 'utf-8');
-      expect(actual).toEqual(expected);
+      var actual = fs.readFileSync(path.join(d, 'other.md'), 'utf8');
+      expect(actual).toMatchSnapshot();
     });
   });
 
@@ -119,7 +104,7 @@ describe('readme command', function() {
     return documentation(['readme index.js'], { cwd: d }).catch(err => {
       expect(err).toBeTruthy();
       expect(err.code !== 0).toBeTruthy();
-      expect(stderr.match(/Missing required argument/)).toBeTruthy();
+      expect(err.stderr.match(/Missing required argument/)).toBeTruthy();
     });
   });
 

__tests__/bin-watch-serve.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var path = require('path');
 var os = require('os');
 var get = require('./utils').get;

__tests__/bin.js

@@ -1,4 +1,4 @@
-'use strict';
+/* global jasmine */
 
 var path = require('path'),
   os = require('os'),

__tests__/fixture/external.input.js

@@ -8,7 +8,6 @@ require('./polyglot/blend.cpp');
  * I am in `external.input.js`.
  */
 function foo() {
-  'use strict';
   return 'bar';
 }
 

__tests__/fixture/flow-unnamed-params.input.js

@@ -1,7 +1,5 @@
 // @flow
 
-'use strict';
-
 /** x */
 let x: T => string;
 

__tests__/fixture/html/nested.input.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /**
  * Creates a new Klass
  * @extends Stream.Writable

__tests__/fixture/node_modules/external/lib/main.js

@@ -1,5 +1,4 @@
 /*eslint max-len: 0 */
-'use strict';
 
 var _formatType = require('../src/output/util/format_type'),
   LinkerStack = require('../src/output/util/linker_stack'),

__tests__/format_type.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var filterAccess = require('../../src/filter_access');
 
 test('filterAccess ignore', function() {

__tests__/lib/filter_access.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var flowDoctrine = require('../../src/flow_doctrine.js'),
   parse = require('../../src/parsers/javascript'),
   FLOW_TYPES = require('babel-types').FLOW_TYPES;

__tests__/lib/flow_doctrine.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var mock = require('mock-fs');
 var mockRepo = require('../../utils').mockRepo;
 var path = require('path');

__tests__/lib/git/find_git.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var mock = require('mock-fs');
 var mockRepo = require('../../utils').mockRepo;
 var getGithubURLPrefix = require('../../../src/git/url_prefix');

__tests__/lib/git/url_prefix.js

@@ -1,5 +1,3 @@
-'use strict';
-
 /* eslint no-unused-vars: 0 */
 
 var mock = require('mock-fs'),

__tests__/lib/github.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../src/parsers/javascript'),
   hierarchy = require('../../src/hierarchy');
 

__tests__/lib/hierarchy.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../../src/parsers/javascript'),
   inferName = require('../../../src/infer/name'),
   inferAccess = require('../../../src/infer/access');

__tests__/lib/infer/access.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../../src/parsers/javascript'),
   findTarget = require('../../../src/infer/finders').findTarget;
 

__tests__/lib/infer/finders.js

@@ -1,4 +1,3 @@
-'use strict';
 /*eslint-disable no-unused-vars*/
 var inferKind = require('../../../src/infer/kind'),
   parse = require('../../../src/parsers/javascript');

__tests__/lib/infer/kind.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../../src/parsers/javascript'),
   inferMembership = require('../../../src/infer/membership')();
 

__tests__/lib/infer/membership.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../../src/parsers/javascript'),
   inferName = require('../../../src/infer/name');
 

__tests__/lib/infer/name.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../../src/parsers/javascript'),
   inferParams = require('../../../src/infer/params');
 

__tests__/lib/infer/params.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../../src/parsers/javascript'),
   inferKind = require('../../../src/infer/kind'),
   inferType = require('../../../src/infer/type');

__tests__/lib/infer/type.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var path = require('path'), shallow = require('../../../src/input/shallow');
 
 test('shallow deps', function() {

__tests__/lib/input/shallow.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../src/parsers/javascript'),
   lintComments = require('../../src/lint').lintComments,
   formatLint = require('../../src/lint').formatLint;

__tests__/lib/lint.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var path = require('path'),
   _ = require('lodash'),
   mergeConfig = require('../../src/merge_config');

__tests__/lib/merge_config.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var nestTag = require('../../src/nest').nestTag;
 
 // Print a tree of tags in a way that's easy to test.

__tests__/lib/nest.js

@@ -1,4 +1,3 @@
-'use strict';
 var formatters = require('../../../../src/output/util/formatters')(getHref);
 
 test('formatters.parameters -- long form', function() {

__tests__/lib/output/util/formatters.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var parse = require('../../src/parsers/javascript'),
   remark = require('remark'),
   visit = require('unist-util-visit');

__tests__/lib/parse.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var remark = require('remark'),
   parse = require('../../../src/parsers/javascript');
 

__tests__/lib/parsers/javascript.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var fs = require('fs'),
   path = require('path'),
   remark = require('remark'),

__tests__/lib/parsers/polyglot.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var get = require('../utils').get;
 var File = require('vinyl');
 var getPort = require('get-port');

__tests__/lib/server.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var sort = require('../../src/sort'), path = require('path');
 
 test('sort stream alphanumeric', function() {

__tests__/lib/sort.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var walk = require('../../src/walk');
 
 test('walk', function() {

__tests__/lib/walk.js

@@ -1,4 +1,3 @@
-'use strict';
 var LinkerStack = require('../src/output/util/linker_stack');
 
 test('linkerStack', function() {

__tests__/linker.js

@@ -1,5 +1,3 @@
-'use strict';
-
 var documentationSchema = require('documentation-schema'),
   validate = require('json-schema'),
   documentation = require('../'),

__tests__/test.js

@@ -1,4 +1,3 @@
-'use strict';
 var http = require('http');
 import { walk } from '../src/walk';
 var concat = require('concat-stream');

__tests__/utils.js

@@ -2,8 +2,6 @@
 
 /* eslint no-console: 0 */
 
-'use strict';
-
 var yargs = require('yargs');
 var commands = require('../lib/commands');