fixup! fix: strip BOM when reading JSON files

index.js

@@ -135,6 +135,7 @@ class Installer {
   }
 }
 module.exports = Installer
+module.exports._readJson = readJson
 
 function readJson (jsonPath, name, ignoreMissing) {
   return readFileAsync(path.join(jsonPath, name), 'utf8')

test/lib/package-json-with-bom.json

@@ -0,0 +1,6 @@
+{
+  "name": "strong-spawn-npm",
+  "version": "1.0.0",
+  "description": "Reliably spawn npm™ on any platform",
+  "homepage": "https://github.com/strongloop/strong-spawn-npm"
+}

test/specs/index.js

@@ -1,5 +1,7 @@
 'use strict'
 
+const fs = require('fs')
+const path = require('path')
 const test = require('tap').test
 const requireInject = require('require-inject')
 const fixtureHelper = require('../lib/fixtureHelper.js')
@@ -295,3 +297,20 @@ test('skips lifecycle scripts with ignoreScripts is set', t => {
     t.end()
   })
 })
+
+test('handles JSON docs that contain a BOM', t => {
+  t.plan(2)
+  const Installer = requireInject('../../index.js', {/* just don't want to cache */})
+  const bomJSON = 'package-json-with-bom.json'
+  const bomJSONDir = path.resolve(__dirname, '../lib')
+  const actualJSON = {
+    name: 'strong-spawn-npm',
+    version: '1.0.0',
+    description: 'Reliably spawn npm™ on any platform',
+    homepage: 'https://github.com/strongloop/strong-spawn-npm'
+  }
+  // ensure that the file does indeed fail to be parsed by JSON.parse
+  t.throws(() => JSON.parse(fs.readFileSync(path.join(bomJSONDir, bomJSON), 'utf8')),
+           {message: 'Unexpected token \uFEFF in JSON at position 0'})
+  return Installer._readJson(bomJSONDir, bomJSON).then(obj => t.match(obj, actualJSON))
+})