Merge pull request #197 from iambumblehead/resolve-cannot-assign-read…

…only-property

added failing test

CHANGELOG.md

@@ -1,5 +1,10 @@
 # changelog
 
+ * 2.2.1 _Apr.03.2023_
+   * [use Object.defineProperty](https://github.com/iambumblehead/esmock/pull/197) to write mock definitions protected on inherited prototype chain
+ * 2.2.0 _Mar.23.2023_
+   * [throw error when](https://github.com/iambumblehead/esmock/pull/193) esmock.strictest is called with empty mock definition, @koshic @Swivelgames 
+   * [update jest and jest-light-runner packages](https://github.com/iambumblehead/esmock/pull/194) at unit-tests, @koshic
  * 2.1.0 _Nov.29.2022_
    * [add node v19](https://github.com/iambumblehead/esmock/pull/189) to ci-test pipeline
    * [use live default export](https://github.com/iambumblehead/esmock/pull/189) to populate enumerable properties of mock definition

package.json

@@ -1,7 +1,7 @@
 {
   "name": "esmock",
   "type": "module",
-  "version": "2.2.0",
+  "version": "2.2.1",
   "license": "ISC",
   "readmeFilename": "README.md",
   "description": "provides native ESM import mocking for unit tests",

src/esmockModule.js

@@ -27,7 +27,10 @@ const isPlainObj = o => Object.getPrototypeOf(o) === objProto
 const esmockModuleMerge = (defLive, def) => isPlainObj(defLive)
   ? Object.assign({}, defLive, def)
   : Object.assign(Object.keys(defLive).reduce(
-    (prev, k) => (prev[k] = defLive[k], prev), Object.create(defLive)), def)
+    (prev, k) => (Object.defineProperty(prev, k, {
+      value: defLive[k],
+      writable: true
+    }), prev), Object.create(defLive)), def)
 
 const esmockModuleMergeDefault = (defLive, def) =>
   (isObj(defLive) && isObj(def)) ? esmockModuleMerge(defLive, def) : def

tests/local/usesNodeProcess.js

@@ -0,0 +1,3 @@
+import nodeprocess from 'node:process'
+
+export default nodeprocess

tests/tests-node/esmock.node.test.js

@@ -5,6 +5,15 @@ import esmock from 'esmock'
 import sinon from 'sinon'
 import esmockCache from '../../src/esmockCache.js'
 
+test('should mock node:process', async () => {
+  // has direct and in-direct calls to `process.cwd()`
+  const thingBeingTested = await esmock('../local/usesNodeProcess.js', {}, {
+    'node:process': { cwd: () => 'tempDir' }
+  })
+
+  assert.strictEqual(thingBeingTested.cwd(), 'tempDir')
+})
+
 test('should mock package, even when package is not installed', async () => {
   const component = await esmock(`../local/notinstalledVueComponent.js`, {
     vue: {