fixup: add common.mustCall

nodejs · May 4, 2022 · e54262e · e54262e
1 parent 7c5679f
commit e54262e
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/test/parallel/test-vm-global-setter.js b/test/parallel/test-vm-global-setter.js
@@ -1,5 +1,5 @@
 'use strict';
-require('../common');
+const common = require('../common');
 const assert = require('assert');
 const vm = require('vm');
 
@@ -15,13 +15,15 @@ assert.strictEqual(descriptor.configurable, true);
 // Regression test for GH-42962. This assignment should not throw.
 window.globalProxy.onhashchange = () => {};
 
+assert.strictEqual(window.globalProxy.onhashchange, 42);
+
 function createWindow() {
   const obj = {};
   vm.createContext(obj);
   Object.defineProperty(obj, 'onhashchange', {
     // eslint-disable-next-line getter-return
-    get() {},
-    set() {},
+    get: common.mustCall(() => 42),
+    set: common.mustCall(),
     configurable: true
   });