From 213206f03a35472fc894e7feb10044aa79df3e50 Mon Sep 17 00:00:00 2001
From: "Benjamin E. Coe" <bencoe@gmail.com>
Date: Thu, 4 May 2017 23:28:11 -0700
Subject: [PATCH] fix: we should not create a cache folder if cache is false
 (#567)

---
 index.js                  |  2 +-
 package.json              |  2 +-
 test/{src => }/nyc-bin.js | 24 +++++++++++++++++++++---
 test/nyc.js               |  1 +
 4 files changed, 24 insertions(+), 5 deletions(-)
 rename test/{src => }/nyc-bin.js (96%)

diff --git a/index.js b/index.js
index 397a527f7..4c93cd3bd 100755
--- a/index.js
+++ b/index.js
@@ -327,7 +327,7 @@ NYC.prototype.clearCache = function () {
 
 NYC.prototype.createTempDirectory = function () {
   mkdirp.sync(this.tempDirectory())
-  mkdirp.sync(this.cacheDirectory)
+  if (this.cache) mkdirp.sync(this.cacheDirectory)
 
   if (this._showProcessTree) {
     mkdirp.sync(this.processInfoDirectory())
diff --git a/package.json b/package.json
index 9b3e815be..d5c0493ee 100644
--- a/package.json
+++ b/package.json
@@ -10,7 +10,7 @@
     "clean": "rimraf ./.nyc_output ./node_modules/.cache ./.self_coverage ./test/fixtures/.nyc_output ./test/fixtures/node_modules/.cache *covered.js ./lib/*covered.js",
     "build": "node ./build-tests",
     "instrument": "node ./build-self-coverage.js",
-    "test-integration": "tap -t120 --no-cov -b ./test/build/*.js && mocha --timeout=15000 ./test/src/nyc-bin.js",
+    "test-integration": "tap -t120 --no-cov -b ./test/build/*.js && mocha --timeout=15000 ./test/nyc-bin.js",
     "test-mocha": "node ./bin/nyc --no-clean --silent --temp-directory=./.self_coverage mocha ./test/nyc.js ./test/process-args.js",
     "report": "node ./bin/nyc  --temp-directory ./.self_coverage/ -r text -r lcov report",
     "release": "standard-version"
diff --git a/test/src/nyc-bin.js b/test/nyc-bin.js
similarity index 96%
rename from test/src/nyc-bin.js
rename to test/nyc-bin.js
index 3c20a1064..f005f6da5 100644
--- a/test/src/nyc-bin.js
+++ b/test/nyc-bin.js
@@ -2,10 +2,10 @@
 
 var _ = require('lodash')
 var path = require('path')
-var bin = path.resolve(__dirname, '../../bin/nyc')
-var fixturesCLI = path.resolve(__dirname, '../fixtures/cli')
+var bin = path.resolve(__dirname, '../bin/nyc')
+var fixturesCLI = path.resolve(__dirname, './fixtures/cli')
 var fakebin = path.resolve(fixturesCLI, 'fakebin')
-var fixturesHooks = path.resolve(__dirname, '../fixtures/hooks')
+var fixturesHooks = path.resolve(__dirname, './fixtures/hooks')
 var fs = require('fs')
 var glob = require('glob')
 var isWindows = require('is-windows')()
@@ -689,6 +689,24 @@ describe('the nyc cli', function () {
     })
   })
 
+  // see: https://github.com/istanbuljs/nyc/issues/563
+  it('does not create .cache folder if cache is "false"', function (done) {
+    var args = [bin, '--cache=false', process.execPath, './index.js']
+
+    var proc = spawn(process.execPath, args, {
+      cwd: process.cwd(),
+      env: env
+    })
+
+    rimraf.sync('./node_modules/.cache')
+
+    proc.on('close', function (code) {
+      code.should.equal(0)
+      fs.existsSync('./node_modules/.cache').should.equal(false)
+      done()
+    })
+  })
+
   it('allows alternative high and low watermarks to be configured', function (done) {
     var args = [
       bin,
diff --git a/test/nyc.js b/test/nyc.js
index fe455e07c..ce8239b46 100644
--- a/test/nyc.js
+++ b/test/nyc.js
@@ -1,6 +1,7 @@
 /* global describe, it */
 
 const NYC = require('../')
+
 require('chai').should()
 
 describe('NYC', function () {