Merge pull request #15 from stevelacy/master

Features: update module, add tests

Author: PatrickJS

.eslintrc

@@ -0,0 +1,23 @@
+{
+  "env": {
+    "node": true
+  },
+  "rules": {
+    "strict": 0,
+    "camelcase": 0,
+    "curly": 0,
+    "indent": [2, 2, { "SwitchCase": 1 }],
+    "eol-last": 1,
+    "no-shadow": 0,
+    "no-redeclare": 2,
+    "no-extra-bind": 1,
+    "no-empty": 0,
+    "no-process-exit": 1,
+    "no-underscore-dangle": 0,
+    "no-use-before-define": 0,
+    "no-unused-vars": 0,
+    "consistent-return": 0,
+    "no-inner-declarations": 1,
+    "no-loop-func": 1
+  }
+}

.travis.yml

@@ -0,0 +1,5 @@
+sudo: false
+language: node_js
+node_js:
+  - "4"
+  - "6"

README.md

@@ -116,6 +116,12 @@ Don't forget to polyfill require if you want to use it in Node.js. See the webpa
         <br />
         <a href="https://github.com/">Kent C. Dodds</a>
       </td>
+      <td align="center">
+        <img width="150" height="150"
+        src="https://avatars.githubusercontent.com/stevelacy?v=3">
+        <br />
+        <a href="https://github.com/stevelacy">Steve Lacy</a>
+      </td>
     <tr>
   <tbody>
 </table>

package.json

@@ -1,15 +1,17 @@
 {
-	"name": "json5-loader",
-	"version": "0.6.0",
-	"author": "Tobias Koppers @sokra",
-	"description": "json5 loader module for webpack",
-	"peerDependencies": {
-		"json5": ">= 0.2.0 <= 0.5"
-	},
-	"licenses": [
-		{
-			"type": "MIT",
-			"url": "http://www.opensource.org/licenses/mit-license.php"
-		}
-	]
+  "name": "json5-loader",
+  "version": "1.0.0",
+  "author": "Tobias Koppers @sokra",
+  "description": "json5 loader module for webpack",
+  "dependencies": {
+    "json5": "^0.5.0"
+  },
+  "license": "MIT",
+  "devDependencies": {
+    "mocha": "^3.0.2",
+    "should": "^11.1.0"
+  },
+  "scripts": {
+    "test": "mocha test/*.js"
+  }
 }

test/loader.js

@@ -0,0 +1,41 @@
+'use strict';
+
+var should = require('should');
+var PROJECT_NAME = require('../package.json').name;
+var loader = require('..');
+var staticJson5 = "{name: 'test'}";
+
+describe(PROJECT_NAME, function () {
+  it('should export the loader', function (done) {
+    should(loader).be.type('function');
+    done();
+  });
+
+  it('should export a default for es2015 support', function (done) {
+    should.exist(loader.default);
+    should(loader.default).be.type('function');
+    done();
+  });
+
+  it('should convert to requires', function (done) {
+    var content = loader.call({}, staticJson5);
+    should(content).be.eql('module.exports = {\n\t"name": "test"\n}');
+    done();
+  });
+
+  it('should fire cacheable', function (done) {
+    var cacheable = function () {
+      done();
+    };
+    var content = loader.call({cacheable: cacheable}, staticJson5);
+    should(content).be.eql('module.exports = {\n\t"name": "test"\n}');
+  });
+
+  it('should catch invalid JSON5', function (done) {
+    var brokenJson5 = '{broken: json5}';
+    should(function () {
+      loader.call({}, brokenJson5);
+    }).throw('Error using JSON5 parsing');
+    done();
+  });
+});