Test invalid json (#7)

* add codecov

* test invalid json

Author: justjavac

src/index.js

@@ -3,13 +3,17 @@ const validateOptions = require('schema-utils')
 
 const schema = require('./options.json')
 
-function shouldInline(limit = 10240, size) {
+const DEFAULT_OPTIONS = {
+  limit: 10240,
+}
+
+function shouldInline(limit, size) {
   return size <= parseInt(limit, 10)
 }
 
 // https://v8.dev/blog/cost-of-javascript-2019#json
 module.exports = function(source) {
-  const options = getOptions(this) || {}
+  const options = Object.assign({}, DEFAULT_OPTIONS, getOptions(this))
 
   validateOptions(schema, options, 'JSON Perf Loader')
 

test/fixtures/file-invalid.json

@@ -0,0 +1 @@
+{

test/fixtures/fixture-invalid.js

@@ -0,0 +1,2 @@
+/* eslint-disable */
+import json from './file-invalid.json';

test/helpers/compiler.js

@@ -15,7 +15,7 @@ const modules = config => {
             type: 'javascript/auto',
             use: {
               loader: path.resolve(__dirname, '../../src'),
-              options: config.loader.options || {},
+              options: config.loader.options,
             },
           },
         ]

test/loader-invalid.test.js

@@ -0,0 +1,18 @@
+const webpack = require('./helpers/compiler')
+
+describe('Loader invaild', () => {
+  it('should throw error', async () => {
+    const config = {
+      loader: {
+        test: /\.json$/,
+        type: 'javascript/auto',
+      },
+    }
+
+    const stats = await webpack('fixture-invalid.js', config)
+    const { modules, errors, warnings } = stats.toJson()
+
+    expect(errors.length).toBeGreaterThan(0)
+    expect(warnings).toEqual([])
+  })
+})

test/loader.test.js

@@ -6,7 +6,6 @@ describe('Loader', () => {
       loader: {
         test: /\.json$/,
         type: 'javascript/auto',
-        options: {},
       },
     }