Merge pull request #8 from sabeurthabti/master

fix linting errors

Author: maximenajim

.eslintrc-node

@@ -4,10 +4,11 @@ extends:
 ecmaFeatures:
   modules: false
 rules:
+  "filenames/filenames": 0
   "func-style":
     - 0
     - declaration
   "object-shorthand":
     - 0
   "no-extra-parens":
-    - 0
+    - 0

.eslintrc-test

@@ -13,6 +13,7 @@ rules:
   "no-unused-expressions": 0 # for `chai.expect`
   "max-len": [2, 150, 2, {ignorePattern: "^\\s*(?:it|describe)\\(.*"}]
   "max-statements": 0
+  "filenames/filenames": 0
   "max-nested-callbacks": 0
   "prefer-arrow-callback": 0
   "func-style":

lib/index.js

@@ -3,7 +3,7 @@
 const Module = require("module");
 const require_ = Module.prototype.require;
 const InstantiateReactComponent = require("react/lib/instantiateReactComponent");
-const EscapeTextContentForBrowser = require("react/lib/escapeTextContentForBrowser");
+const escapeTextContentForBrowser = require("react/lib/escapeTextContentForBrowser");
 
 const get = require("lodash/get");
 const set = require("lodash/set");
@@ -87,7 +87,7 @@ class InstantiateReactComponentOptimizer {
       templateAttrs.forEach((attrKey) => {
         const _attrKey = attrKey.replace(".", "__");
         set(curEl.props, attrKey, templateAttrValues[_attrKey]);
-        templateAttrValues[_attrKey] = EscapeTextContentForBrowser(templateAttrValues[_attrKey]);
+        templateAttrValues[_attrKey] = escapeTextContentForBrowser(templateAttrValues[_attrKey]);
       });
       return compiled(templateAttrValues);
     };

package.json

@@ -36,6 +36,7 @@
     "chai": "^3.5.0",
     "eslint": "^1.10.3",
     "eslint-config-defaults": "^8.0.1",
+    "eslint-plugin-filenames": "^1.1.0",
     "intercept-stdout": "^0.1.2",
     "istanbul": "^0.4.3",
     "mocha": "^2.4.5",

test/spec/index.spec.js

@@ -311,6 +311,7 @@ describe("react-component-cache", function () {
   });
 
   it("should templatize properly even when prop names overlap with '_'", () => {
+    /*eslint-disable camelcase*/
     clearRequireCache();
 
     let renderCount = 0;
@@ -333,13 +334,14 @@ describe("react-component-cache", function () {
       }
     }
 
-    let props = {foo: {bar: "Hello World X!"}, foo_bar: "Hello World Y!"};
+    const props = {foo: {bar: "Hello World X!"}, foo_bar: "Hello World Y!"};
     // Cache Miss
     expect(ReactDomServer.renderToString(React.createFactory(HelloWorld)(props))).to.contains("Hello World X!");
     expect(renderCount).to.equal(1);
     // Cache Hit
     expect(ReactDomServer.renderToString(React.createFactory(HelloWorld)(props))).to.contains("Hello World Y!");
     expect(renderCount).to.equal(1);
+    /*eslint-enable camelcase*/
   });
 
   it("should should throw error when templateAttr is function or object", () => {