update

Author: bichikim

packages/solid-test/src/trigger/__tests__/index.spec.ts

@@ -14,4 +14,22 @@ describe('trigger', () => {
     trigger.run()
     expect(callback).toHaveBeenCalled()
   })
+  it('should get changed count', () => {
+    const trigger = createTrigger()
+    const callback = vi.fn()
+
+    expect(trigger.changed).toBe(0)
+    trigger.target = callback
+    expect(trigger.changed).toBe(1)
+    trigger.target = callback
+    expect(trigger.changed).toBe(2)
+  })
+  it('should get target', () => {
+    const trigger = createTrigger()
+    const callback = vi.fn()
+
+    expect(trigger.target).toBeUndefined()
+    trigger.target = callback
+    expect(trigger.target).toBe(callback)
+  })
 })

packages/utils/src/json-stringfy/__tests__/index.spec.ts

@@ -1,5 +1,15 @@
 import {jsonStringify} from '../'
-import {describe, expect, it} from 'vitest'
+import {describe, expect, it, vi} from 'vitest'
+import {stringify} from 'safe-stable-stringify'
+
+vi.mock('safe-stable-stringify', async () => {
+  const module: any = await import('safe-stable-stringify')
+
+  return {
+    ...module,
+    stringify: vi.fn(module.stringify),
+  }
+})
 describe('jsonStringify', () => {
   it('should return string', async () => {
     const result = jsonStringify({
@@ -8,13 +18,23 @@ describe('jsonStringify', () => {
 
     expect(result).toBe('{"foo":"foo"}')
   })
-  // todo fix test
-  it.skip('should return string', async () => {
+  it('should return string with circular', async () => {
     const obj = {props: {} as any}
 
     obj.props = obj
     const result = jsonStringify(obj)
 
+    expect(result).toBe('{"props":"[Circular]"}')
+  })
+  it('should return empty defaultValue when error occurs', async () => {
+    // mock stringify to throw error
+    vi.mocked(stringify).mockImplementation(() => {
+      throw new Error('test')
+    })
+    const result = jsonStringify({
+      foo: '',
+    })
+
     expect(result).toBe('')
   })
 })