fix: revert reusing spy when calling spyOn

tinylibs · Sep 9, 2024 · 9a54c10 · 9a54c10
1 parent 88cf76b
commit 9a54c10
Show file tree

Hide file tree

Showing 2 changed files with 17 additions and 17 deletions.
diff --git a/src/spyOn.ts b/src/spyOn.ts
@@ -116,24 +116,24 @@ export function internalSpyOn<T, K extends string & keyof T>(
 
   if (!mock) mock = origin
 
-  let fn: SpyInternal
-  if (origin && S in origin) {
-    fn = origin as SpyInternal
-  } else {
-    fn = createInternalSpy(mock)
-    if (accessType === 'value') {
-      prototype(fn, origin)
-    }
-
-    const state = fn[S]
-    defineValue(state, 'restore', restore)
-    defineValue(state, 'getOriginal', () => (ssr ? origin() : origin))
-    defineValue(state, 'willCall', (newCb: Procedure) => {
-      state.impl = newCb
-      return fn
-    })
+  // let fn: SpyInternal
+  // if (origin && S in origin) {
+  //   fn = origin as SpyInternal
+  // } else {
+  let fn = createInternalSpy(mock)
+  if (accessType === 'value') {
+    prototype(fn, origin)
   }
 
+  const state = fn[S]
+  defineValue(state, 'restore', restore)
+  defineValue(state, 'getOriginal', () => (ssr ? origin() : origin))
+  defineValue(state, 'willCall', (newCb: Procedure) => {
+    state.impl = newCb
+    return fn
+  })
+  // }
+
   reassign(
     ssr
       ? () => {

diff --git a/test/index.test.ts b/test/index.test.ts
@@ -756,7 +756,7 @@ test('next in a row', () => {
   expect(cb()).toBe(undefined)
 })
 
-test('spying twice and unspying restores original method', () => {
+test.todo('spying twice and unspying restores original method', () => {
   const obj = {
     method: () => 1,
   }