Also handle nested non-writable getters

lib/utils.js

@@ -675,7 +675,9 @@ exports.breakCircularDeps = inputObj => {
 
     seen.add(obj);
     for (const k in obj) {
-      if (Object.prototype.hasOwnProperty.call(obj, k)) {
+      const descriptor = Object.getOwnPropertyDescriptor(obj, k);
+
+      if (descriptor && descriptor.writable) {
         obj[k] = _breakCircularDeps(obj[k], k);
       }
     }

test/integration/fixtures/parallel/getter-error-object.mjs

@@ -0,0 +1,15 @@
+import {describe, it} from '../../../../index.js';
+
+describe('test1', () => {
+  it('test', async () => {
+    const error = new Error('Oh no!');
+
+    error.nested = {
+      get inner() {
+        return 'abc';
+      }
+    };
+
+    throw error;
+  });
+});

test/integration/parallel.spec.js

@@ -71,4 +71,16 @@ describe('parallel run', () => {
     assert.strictEqual(result.failures[0].err.message, 'Oh no!');
     assert.deepStrictEqual(result.failures[0].err.values, [ { toB: { toA: '[Circular]' } } ]);
   });
+
+  it('should correctly handle a non-writable getter reference in an exception', async () => {
+    const result = await runMochaJSONAsync('parallel/getter-error-object.mjs', [
+      '--parallel',
+      '--jobs',
+      '2',
+      require.resolve('./fixtures/parallel/testworkerid1.mjs')
+    ]);
+    assert.strictEqual(result.stats.failures, 1);
+    assert.strictEqual(result.stats.passes, 1);
+    assert.strictEqual(result.failures[0].err.message, 'Oh no!');
+  });
 });