fix(vitest): skip processing getter in auto-mocked constructor call (#…

…4677)
vitest-dev · Dec 6, 2023 · cb7864a · cb7864a
1 parent 1e4aa8e
commit cb7864a
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 1 deletion.
diff --git a/packages/vitest/src/runtime/mocker.ts b/packages/vitest/src/runtime/mocker.ts
@@ -334,7 +334,11 @@ export class VitestMocker {
             // so that mock states between prototype/instances don't affect each other
             // (jest reference https://github.com/jestjs/jest/blob/2c3d2409879952157433de215ae0eee5188a4384/packages/jest-mock/src/index.ts#L678-L691)
             if (this instanceof newContainer[property]) {
-              for (const { key } of getAllMockableProperties(this, false, primitives)) {
+              for (const { key, descriptor } of getAllMockableProperties(this, false, primitives)) {
+                // skip getter since it's not mocked on prototype as well
+                if (descriptor.get)
+                  continue
+
                 const value = this[key]
                 const type = getType(value)
                 const isFunction = type.includes('Function') && typeof value === 'function'

diff --git a/test/core/src/mockedC.ts b/test/core/src/mockedC.ts
@@ -20,6 +20,10 @@ export class MockedC {
   }
 
   set getSetProp(_val: number) {}
+
+  get getExpectNotCalled(): number {
+    throw new Error('automocked constructor should not call this getter')
+  }
 }
 
 export async function asyncFunc(): Promise<string> {