Merge pull request #1798 from marvinhagemeister/process_leakage

Fix binding tests leaking process values
sass · Nov 13, 2016 · fdda254 · fdda254
2 parents 5c7fb7f + 17921a9
commit fdda254
Showing 1 changed file with 9 additions and 3 deletions.
diff --git a/test/binding.js b/test/binding.js
@@ -53,7 +53,9 @@ describe('binding', function() {
       });
 
       afterEach(function() {
-        process.arch = prevValue;
+        Object.defineProperty(process, 'arch', {
+          value: prevValue,
+        });
       });
 
       it('should error', function() {
@@ -83,7 +85,9 @@ describe('binding', function() {
       });
 
       afterEach(function() {
-        process.platform = prevValue;
+        Object.defineProperty(process, 'platform', {
+          value: prevValue,
+        });
       });
 
       it('should error', function() {
@@ -113,7 +117,9 @@ describe('binding', function() {
       });
 
       afterEach(function() {
-        process.versions.modules = prevValue;
+        Object.defineProperty(process.versions, 'modules', {
+          value: prevValue,
+        });
       });
 
       it('should error', function() {