small lint fix

Author: dozoisch

karma.conf.js

@@ -19,7 +19,7 @@ module.exports = function (config) {
     ],
 
     preprocessors: {
-      "test/index.js": ["webpack", "sourcemap",],
+      "test/index.js": ["webpack", "sourcemap"],
     },
 
     webpack: webpackConfig,

src/recaptcha-wrapper.js

@@ -1,6 +1,4 @@
 "use strict";
-import React, { PropTypes } from "react";
-
 import ReCAPTCHA from "./recaptcha";
 import makeAsyncScriptLoader from "react-async-script";
 

src/recaptcha.js

@@ -1,12 +1,12 @@
-/*global grecaptcha*/
 "use strict";
 import React, { PropTypes } from "react";
 
 const ReCAPTCHA = React.createClass({
-  displayName: "react-reCAPTCHA",
+  displayName: "reCAPTCHA",
   propTypes: {
     sitekey: PropTypes.string.isRequired,
     onChange: PropTypes.func.isRequired,
+    grecaptcha: PropTypes.object,
     theme: PropTypes.oneOf(["dark", "light"]),
     type: PropTypes.oneOf(["image", "audio"]),
     tabindex: PropTypes.number,

test/recaptcha-spec.js

@@ -12,7 +12,7 @@ describe("ReCAPTCHA", function () {
 
   it("Rendered Component should be a div", function () {
     let instance = ReactTestUtils.renderIntoDocument(
-      <ReCAPTCHA siteKey="xxx" onloadCallbackName={callbackName} />
+      <ReCAPTCHA sitekey="xxx" onloadCallbackName={callbackName} />
     );
     assert.equal(instance.getDOMNode().nodeName, "DIV");
   });
@@ -21,29 +21,25 @@ describe("ReCAPTCHA", function () {
       className: "TheClassName",
       id: "superdefinedId",
     };
-    let className = "TheClassName";
     let instance = ReactTestUtils.renderIntoDocument(
-      <ReCAPTCHA siteKey="xxx" {...props} />
+      <ReCAPTCHA sitekey="xxx" {...props} />
     );
     assert.equal(instance.getDOMNode().id, props.id);
     assert.match(instance.getDOMNode().className, new RegExp(props.className));
   });
 
-  it("should register a callback on the window when grecaptcha is not loaded", function () {
-    let instance = ReactTestUtils.renderIntoDocument(
-      <ReCAPTCHA siteKey="xxx" />
-    );
-    assert.isNotNull(window[callbackName]);
-  });
   it("should call grecaptcha.render, when it is already loaded", function (done) {
     let grecaptchaMock = {
-      render() {
+      render(node, options) {
+        assert.isNotNull(node);
+        assert.equal(options.sitekey, "xxx");
         done();
       },
     };
     let instance = ReactTestUtils.renderIntoDocument(
-      <ReCAPTCHA siteKey="xxx" grecaptcha={grecaptchaMock} />
+      <ReCAPTCHA sitekey="xxx" grecaptcha={grecaptchaMock} />
     );
+    assert.ok(instance);
   });
   it("reset, should call grecaptcha.reset with the widget id", function (done) {
     let grecaptchaMock = {
@@ -57,7 +53,7 @@ describe("ReCAPTCHA", function () {
       },
     };
     let instance = ReactTestUtils.renderIntoDocument(
-      <ReCAPTCHA siteKey="xxx" grecaptcha={grecaptchaMock} />
+      <ReCAPTCHA sitekey="xxx" grecaptcha={grecaptchaMock} />
     );
     instance.reset();
   });

tools/build.js

@@ -1,8 +1,6 @@
 /* eslint no-process-exit: 0 */
 import "colors";
-import path from "path";
 import lib from "./lib/build";
-import { exec, spawn } from "child-process-promise";
 
 export default function Build(noExitOnFailure) {
   let result = Promise.all([

tools/lib/build.js

@@ -1,7 +1,6 @@
 import "colors";
 import path from "path";
-import fsp from "fs-promise";
-import { exec, spawn } from "child-process-promise";
+import { exec } from "child-process-promise";
 
 const repoRoot = path.resolve(__dirname, "../../");
 const lib = path.join(repoRoot, "lib");

webpack/strategies/test.js

@@ -14,4 +14,4 @@ export default (config, options) => {
     return config;
   }
   return config;
-}
+};

webpack/webpack.config.js

@@ -60,4 +60,4 @@ export default (options) => {
   return strategies.reduce((conf, strategy) => {
     return strategy(conf, options);
   }, config);
-}
+};