feat: allow the output file to not include the hostname (#57)

Author: mvegter

README.md

@@ -33,7 +33,7 @@ Convert coverage from the format outputted by [puppeteer](https://developers.goo
         page.coverage.stopJSCoverage(),
         page.coverage.stopCSSCoverage(),
       ]);
-      pti.write([...jsCoverage, ...cssCoverage], { storagePath: './.nyc_output' })
+      pti.write([...jsCoverage, ...cssCoverage], { includeHostname: true , storagePath: './.nyc_output' })
       await browser.close()
     })()
     ```

lib/output-files.js

@@ -14,6 +14,7 @@ let iterator = {}
 class OutputFiles {
   constructor (coverageInfo, options = {}) {
     this.storagePath = options.storagePath || './.nyc_output/js'
+    this.includeHostname = options.hasOwnProperty('includeHostname') ? options.includeHostname : true
 
     // Clone coverageInfo to prevent mutating the passed in data
     this.coverageInfo = clone(coverageInfo)
@@ -32,7 +33,7 @@ class OutputFiles {
 
     let postProtocolPath = urlPath.pathname.substring(1)
 
-    if (urlPath.hostname) {
+    if (urlPath.hostname && this.includeHostname) {
       let hostnameAndPort = urlPath.hostname
       if (urlPath.port) {
         hostnameAndPort = hostnameAndPort + '_' + urlPath.port

lib/puppeteer-to-istanbul.js

@@ -9,6 +9,7 @@ let jsonPart = {}
 class PuppeteerToIstanbul {
   constructor (coverageInfo, options = {}) {
     this.storagePath = options.storagePath || './.nyc_output'
+    this.includeHostname = options.hasOwnProperty('includeHostname') ? options.includeHostname : true
 
     this.coverageInfo = coverageInfo
     this.options = options

test/output-files.js

@@ -79,6 +79,16 @@ describe('output-files', () => {
     coverageInfo[3].url.should.include('js/views/record.js')
   })
 
+  it('appropriately handles modules required via http/https, with hostName excluded', () => {
+    const fixture = require('./fixtures/http-es6-modules.json')
+    const coverageInfo = OutputFiles(fixture, { includeHostname: false }).getTransformedCoverage().map(info => ({...info, url: info.url.replace(/\\/g, '/')}))
+
+    coverageInfo[0].url.should.include('js/index.js')
+    coverageInfo[1].url.should.include('js/utils/doc_ready.js')
+    coverageInfo[2].url.should.include('js/models/record.js')
+    coverageInfo[3].url.should.include('js/views/record.js')
+  })
+
   it('maintains original url in output', () => {
     const fixture = require('./fixtures/http-es6-modules.json')
     const coverageInfo = OutputFiles(fixture).getTransformedCoverage()