Merge pull request #90 from kellyselden/global-option-fix

still use global options when someone destructures

Author: jprichardson

test/write-file-sync.test.js

@@ -47,6 +47,20 @@ describe('+ writeFileSync()', function () {
       // restore default
       jf.spaces = null
     })
+
+    it('still uses global when context lost', function () {
+      var file = path.join(TEST_DIR, 'somefile.json')
+      var obj = {name: 'JP'}
+      jf.spaces = 2
+      var writeFileSync = jf.writeFileSync
+      writeFileSync(file, obj)
+
+      var data = fs.readFileSync(file, 'utf8')
+      assert.equal(data, '{\n  "name": "JP"\n}\n')
+
+      // restore default
+      jf.spaces = null
+    })
   })
 
   describe('> when JSON replacer is set', function () {

test/write-file.test.js

@@ -55,6 +55,23 @@ describe('+ writeFile()', function () {
         done()
       })
     })
+
+    it('still uses global when context lost', function (done) {
+      var file = path.join(TEST_DIR, 'somefile.json')
+      var obj = {name: 'JP'}
+      jf.spaces = 2
+      var writeFile = jf.writeFile
+      writeFile(file, obj, function (err) {
+        assert.ifError(err)
+
+        var data = fs.readFileSync(file, 'utf8')
+        assert.equal(data, '{\n  "name": "JP"\n}\n')
+
+        // restore default
+        jf.spaces = null
+        done()
+      })
+    })
   })
 
   describe('> when JSON replacer is set', function () {