Fix lint errors

Author: fkling

.eslintrc

@@ -1,11 +1,20 @@
 {
   "parser": "babel-eslint",
   "rules": {
-    "strict": 0,
     "comma-dangle": [1, "always-multiline"],
-    "quotes": [2, "single", "avoid-escape"]
+    "no-underscore-dangle": 0,
+    "quotes": [2, "single", "avoid-escape"],
+    "strict": 0
   },
   "env": {
     "node": true
+  },
+  "globals": {
+    "ASTNode": true,
+    "Handler": true,
+    "NodePath": true,
+    "PropTypeDescriptor": true,
+    "PropDescriptor": true,
+    "Resolver": true
   }
 }

src/Documentation.js

@@ -6,13 +6,10 @@
  *  LICENSE file in the root directory of this source tree. An additional grant
  *  of patent rights can be found in the PATENTS file in the same directory.
  *
+ *  @flow
+ *
  */
 
-/**
- * @flow
- */
-"use strict";
-
 class Documentation {
   _props: Object;
   _description: string;
@@ -49,7 +46,7 @@ class Documentation {
   toObject(): Object {
     var obj = {
       description: this._description,
-      props: this._props
+      props: this._props,
     };
 
     if (this._composes.length) {

src/__mocks__/Documentation.js

@@ -21,7 +21,7 @@ function Documentation() {
     },
     setDescription(descr) {
       this.description = descr;
-    }
+    },
   };
 }
 

src/handlers/__tests__/propTypeCompositionHandler-test.js

@@ -8,7 +8,7 @@
  *
  */
 
-/*global jest, describe, it, expect, beforeEach, afterEach*/
+/*global jest, describe, it, expect, beforeEach*/
 
 jest.autoMockOff();
 jest.mock('../../Documentation');

src/handlers/propTypeHandler.js

@@ -13,7 +13,6 @@
 import type Documentation from '../Documentation';
 
 import getMembers from '../utils/getMembers';
-import getNameOrValue from '../utils/getNameOrValue';
 import getPropType from '../utils/getPropType';
 import getPropertyName from '../utils/getPropertyName';
 import getMemberValuePath from '../utils/getMemberValuePath';
@@ -44,7 +43,7 @@ function isRequired(path) {
   );
 }
 
-function amendPropTypes(documentation, path, componentPath) {
+function amendPropTypes(documentation, path) {
   path.get('properties').each(function(propertyPath) {
     switch (propertyPath.node.type) {
       case types.Property.name:
@@ -86,5 +85,5 @@ export default function propTypeHandler(
   if (!propTypesPath) {
     return;
   }
-  amendPropTypes(documentation, propTypesPath, path);
+  amendPropTypes(documentation, propTypesPath);
 }

src/main.js

@@ -37,8 +37,8 @@ var defaultHandlers = [
  */
 function defaultParse( // eslint-disable-line no-unused-vars
   src: string,
-  resolver?: ?Resolver,
-  handlers?: ?Array<Handler>
+  resolver?: ?Resolver, // eslint-disable-line no-shadow
+  handlers?: ?Array<Handler> // eslint-disable-line no-shadow
 ): Array<Object>|Object {
   if (!resolver) {
     resolver = defaultResolver;

src/resolver/__tests__/findExportedComponentDefinition-test.js

@@ -291,7 +291,7 @@ describe('findExportedComponentDefinition', () => {
           `;
           expect(() => parse(source)).toThrow();
 
-          var source = `
+          source = `
             import React from 'React';
             var Component = class extends React.Component {};
             export {Component};
@@ -428,7 +428,7 @@ describe('findExportedComponentDefinition', () => {
           `;
           expect(() => parse(source)).toThrow();
 
-          var source = `
+          source = `
             import React from 'React';
             export var ComponentA = class extends React.Component {};
             var ComponentB  = class extends React.Component {};

src/utils/__tests__/docblock-test.js

@@ -8,9 +8,10 @@
  *
  */
 
-"use strict";
+/*global jest, describe, beforeEach, it, expect*/
 
-jest.autoMockOff();
+jest
+  .dontMock('../docblock');
 
 describe('docblock', () => {
 

src/utils/__tests__/getMemberValuePath-test.js

@@ -8,7 +8,7 @@
  *
  */
 
-/*global jest, describe, xdescribe, beforeEach, it, expect*/
+/*global jest, describe, beforeEach, it, expect*/
 
 jest
   .dontMock('../getMemberValuePath.js');

src/utils/__tests__/getMembers-test.js

@@ -8,30 +8,23 @@
  *
  */
 
-"use strict";
+/*global jest, describe, beforeEach, it, expect*/
 
 jest.autoMockOff();
 
 describe('getMembers', () => {
-  var recast;
+  var expression;
   var getMembers;
   var memberExpressionPath;
 
-  function parse(src) {
-    return new recast.types.NodePath(
-      recast.parse(src).program.body[0].expression
-    );
-  }
-
   beforeEach(() => {
     getMembers = require('../getMembers');
-    recast = require('recast');
-    memberExpressionPath = parse('foo.bar(123)(456)[baz][42]');
+    ({expression} = require('../../../tests/utils'));
+    memberExpressionPath = expression('foo.bar(123)(456)[baz][42]');
   });
 
 
   it('finds all "members" "inside" a MemberExpression', () => {
-    var b = recast.types.builders;
     var members = getMembers(memberExpressionPath);
 
     //bar(123)