Use arrow functions in test suite

Using arrow functions in the test suite removes the "function" clutter
from the Jasmine DSL, making it easier to read.

Test plan: Run `npm test` and see everything continues to pass.

Author: wincent

src/__tests__/main-test.js

@@ -34,16 +34,16 @@ var source = [
   'module.exports = Component;'
 ].join('\n');
 
-describe('main', function() {
+describe('main', () => {
   var utils;
   var docgen;
 
-  beforeEach(function() {
+  beforeEach(() => {
     utils = require('../../tests/utils');
     docgen = require('../main');
   });
 
-  it('parses with default resolver/handlers', function() {
+  it('parses with default resolver/handlers', () => {
     var docs = docgen.parse(source);
     expect(docs).toEqual({
       description: 'Example component description',
@@ -63,7 +63,7 @@ describe('main', function() {
     });
   });
 
-  it('parses with custom handlers', function() {
+  it('parses with custom handlers', () => {
     var docs = docgen.parse(source, null, [
       docgen.handlers.componentDocblockHandler,
     ]);

src/__tests__/parse-test.js

@@ -12,11 +12,11 @@
 
 jest.autoMockOff();
 
-describe('parse', function() {
+describe('parse', () => {
   var utils;
   var parse;
 
-  beforeEach(function() {
+  beforeEach(() => {
     utils = require('../../tests/utils');
     parse = require('../parse');
   });
@@ -25,7 +25,7 @@ describe('parse', function() {
     return utils.parse(source).get('body', 0, 'expression');
   }
 
-  it('allows custom component definition resolvers', function() {
+  it('allows custom component definition resolvers', () => {
     var path = pathFromSource('({foo: "bar"})');
     var resolver = jest.genMockFunction().mockReturnValue(path);
     var handler = jest.genMockFunction();
@@ -35,7 +35,7 @@ describe('parse', function() {
     expect(handler.mock.calls[0][1]).toBe(path);
   });
 
-  it('errors if component definition is not found', function() {
+  it('errors if component definition is not found', () => {
     var resolver = jest.genMockFunction();
     expect(function() {
       parse('', resolver);

src/handlers/__tests__/componentDocblockHandler-test.js

@@ -13,7 +13,7 @@
 jest.autoMockOff();
 jest.mock('../../Documentation');
 
-describe('componentDocblockHandler', function() {
+describe('componentDocblockHandler', () => {
   var utils;
   var documentation;
   var componentDocblockHandler;
@@ -31,13 +31,13 @@ describe('componentDocblockHandler', function() {
     );
   }
 
-  beforeEach(function() {
+  beforeEach(() => {
     utils = require('../../../tests/utils');
     documentation = new (require('../../Documentation'));
     componentDocblockHandler = require('../componentDocblockHandler');
   });
 
-  it('finds docblocks for component definitions', function() {
+  it('finds docblocks for component definitions', () => {
     var definition = parse([
       '/**',
       ' * Component description',
@@ -49,7 +49,7 @@ describe('componentDocblockHandler', function() {
     expect(documentation.description).toBe('Component description');
   });
 
-  it('ignores other types of comments', function() {
+  it('ignores other types of comments', () => {
     var definition = parse([
       '/*',
       ' * This is not a docblock',
@@ -69,7 +69,7 @@ describe('componentDocblockHandler', function() {
     expect(documentation.description).toBe('');
   });
 
-  it('only considers the docblock directly above the definition', function() {
+  it('only considers the docblock directly above the definition', () => {
     var definition = parse([
       '/**',
       ' * This is the wrong docblock',

src/handlers/__tests__/defaultPropsHandler-test.js

@@ -13,7 +13,7 @@
 jest.autoMockOff();
 jest.mock('../../Documentation');
 
-describe('defaultPropsHandler', function() {
+describe('defaultPropsHandler', () => {
   var utils;
   var documentation;
   var defaultValueHandler;
@@ -22,13 +22,13 @@ describe('defaultPropsHandler', function() {
     return utils.parse(src).get('body', 0, 'expression');
   }
 
-  beforeEach(function() {
+  beforeEach(() => {
     utils = require('../../../tests/utils');
     documentation = new (require('../../Documentation'));
     defaultPropsHandler = require('../defaultPropsHandler');
   });
 
-  it ('should find prop default values that are literals', function() {
+  it ('should find prop default values that are literals', () => {
     var definition = parse([
       '({',
       '  getDefaultProps: function() {',

src/handlers/__tests__/propDocblockHandler-test.js

@@ -13,12 +13,12 @@
 jest.autoMockOff();
 jest.mock('../../Documentation');
 
-describe('propDocblockHandler', function() {
+describe('propDocblockHandler', () => {
   var utils;
   var documentation;
   var propDocblockHandler;
 
-  beforeEach(function() {
+  beforeEach(() => {
     utils = require('../../../tests/utils');
     documentation = new (require('../../Documentation'));
     propDocblockHandler = require('../propDocblockHandler');
@@ -33,7 +33,7 @@ describe('propDocblockHandler', function() {
     );
   }
 
-  it('finds docblocks for prop types', function() {
+  it('finds docblocks for prop types', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -61,7 +61,7 @@ describe('propDocblockHandler', function() {
     });
   });
 
-  it('can handle multline comments', function() {
+  it('can handle multline comments', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -85,7 +85,7 @@ describe('propDocblockHandler', function() {
     });
   });
 
-  it('ignores non-docblock comments', function() {
+  it('ignores non-docblock comments', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -115,7 +115,7 @@ describe('propDocblockHandler', function() {
     });
   });
 
-  it('only considers the comment with the property below it', function() {
+  it('only considers the comment with the property below it', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -139,7 +139,7 @@ describe('propDocblockHandler', function() {
     });
   });
 
-  it('understands and ignores the spread operator', function() {
+  it('understands and ignores the spread operator', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -160,7 +160,7 @@ describe('propDocblockHandler', function() {
     });
   });
 
-  it('resolves variables', function() {
+  it('resolves variables', () => {
     var definition = parse([
       'var Props = {',
       '  /**',
@@ -181,7 +181,7 @@ describe('propDocblockHandler', function() {
     });
   });
 
-  it('does not error if propTypes cannot be found', function() {
+  it('does not error if propTypes cannot be found', () => {
     var definition = parse([
       '({',
       '  fooBar: 42',

src/handlers/__tests__/propTypeHandler-test.js

@@ -13,13 +13,13 @@
 jest.autoMockOff();
 jest.mock('../../Documentation');
 
-describe('propTypeHandler', function() {
+describe('propTypeHandler', () => {
   var utils;
   var getPropTypeMock;
   var documentation;
   var propTypeHandler;
 
-  beforeEach(function() {
+  beforeEach(() => {
     utils = require('../../../tests/utils');
     getPropTypeMock = jest.genMockFunction().mockImplementation(() => ({}));
     jest.setMock('../../utils/getPropType', getPropTypeMock);
@@ -38,7 +38,7 @@ describe('propTypeHandler', function() {
     );
   }
 
-  it('passes the correct argument to getPropType', function() {
+  it('passes the correct argument to getPropType', () => {
     var definition = parse(
       '({propTypes: {foo: PropTypes.bool, abc: PropTypes.xyz}})'
     );
@@ -52,7 +52,7 @@ describe('propTypeHandler', function() {
     expect(getPropTypeMock).toBeCalledWith(xyzPath);
   });
 
-  it('finds definitions via React.PropTypes', function() {
+  it('finds definitions via React.PropTypes', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -76,7 +76,7 @@ describe('propTypeHandler', function() {
     });
   });
 
-  it('finds definitions via the ReactPropTypes module', function() {
+  it('finds definitions via the ReactPropTypes module', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -95,7 +95,7 @@ describe('propTypeHandler', function() {
     });
   });
 
-  it('detects whether a prop is required', function() {
+  it('detects whether a prop is required', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -119,7 +119,7 @@ describe('propTypeHandler', function() {
     });
   });
 
-  it('only considers definitions from React or ReactPropTypes', function() {
+  it('only considers definitions from React or ReactPropTypes', () => {
     var definition = parse([
       '({',
       '  propTypes: {',
@@ -145,7 +145,7 @@ describe('propTypeHandler', function() {
     });
   });
 
-  it('understands the spread operator', function() {
+  it('understands the spread operator', () => {
     var definition = parse([
       'var Foo = require("Foo.react");',
       'var props = {bar: PropTypes.bool};',
@@ -172,7 +172,7 @@ describe('propTypeHandler', function() {
     });
   });
 
-  it('resolves variables', function() {
+  it('resolves variables', () => {
     var definition = parse([
       'var props = {bar: PropTypes.bool};',
       '({',
@@ -189,7 +189,7 @@ describe('propTypeHandler', function() {
     });
   });
 
-  it('does not error if propTypes cannot be found', function() {
+  it('does not error if propTypes cannot be found', () => {
     var definition = parse([
       '({',
       '  fooBar: 42',

src/resolver/__tests__/findAllReactCreateClassCalls-test.js

@@ -12,7 +12,7 @@
 
 jest.autoMockOff();
 
-describe('React documentation parser', function() {
+describe('React documentation parser', () => {
   var findAllReactCreateClassCalls;
   var recast;
 
@@ -23,13 +23,13 @@ describe('React documentation parser', function() {
     );
   }
 
-  beforeEach(function() {
+  beforeEach(() => {
     findAllReactCreateClassCalls = require('../findAllReactCreateClassCalls');
     recast = require('recast');
   });
 
 
-  it('finds React.createClass', function() {
+  it('finds React.createClass', () => {
     var source = [
       'var React = require("React");',
       'var Component = React.createClass({});',
@@ -43,7 +43,7 @@ describe('React documentation parser', function() {
     expect(result[0].node.type).toBe('ObjectExpression');
   });
 
-  it('finds React.createClass, independent of the var name', function() {
+  it('finds React.createClass, independent of the var name', () => {
     var source = [
       'var R = require("React");',
       'var Component = R.createClass({});',
@@ -55,7 +55,7 @@ describe('React documentation parser', function() {
     expect(result.length).toBe(1);
   });
 
-  it('does not process X.createClass of other modules', function() {
+  it('does not process X.createClass of other modules', () => {
     var source = [
       'var R = require("NoReact");',
       'var Component = R.createClass({});',
@@ -67,7 +67,7 @@ describe('React documentation parser', function() {
     expect(result.length).toBe(0);
   });
 
-  it('finds assignments to exports', function() {
+  it('finds assignments to exports', () => {
     var source = [
       'var R = require("React");',
       'var Component = R.createClass({});',
@@ -80,7 +80,7 @@ describe('React documentation parser', function() {
     expect(result.length).toBe(1);
   });
 
-  it('accepts multiple definitions', function() {
+  it('accepts multiple definitions', () => {
     var source = [
       'var R = require("React");',
       'var ComponentA = R.createClass({});',