test: bug repro for vm function redefinition

This commit adds a failing test case for the vm module. Currently, if runInContext() defines a function, and a later call to runInContext() redefines the same function, the original function is not overwritten. Refs: #548 PR-URL: #5528 Reviewed-By: James M Snell <[email protected]> Reviewed-By: Wyatt Preul <[email protected]> Reviewed-By: Rich Trott <[email protected]>
nodejs · Mar 4, 2016 · e38e2a6 · e38e2a6
1 parent 32e1f9d
commit e38e2a6
Showing 1 changed file with 11 additions and 0 deletions.
diff --git a/test/known_issues/test-vm-function-redefinition.js b/test/known_issues/test-vm-function-redefinition.js
@@ -0,0 +1,11 @@
+'use strict';
+// Refs: https://github.com/nodejs/node/issues/548
+require('../common');
+const assert = require('assert');
+const vm = require('vm');
+const context = vm.createContext();
+
+vm.runInContext('function test() { return 0; }', context);
+vm.runInContext('function test() { return 1; }', context);
+const result = vm.runInContext('test()', context);
+assert.strictEqual(result, 1);