From f3f62e2872105cb29a5e7abfc923bb54c2a5a984 Mon Sep 17 00:00:00 2001 From: gabrieledm Date: Wed, 26 Aug 2026 15:33:01 +0200 Subject: [PATCH 1/2] test: used setup function in 'when the data is cached' tests --- .../src/caching/useCache.test.ts | 18 +++++++++++------- 1 file changed, 11 insertions(+), 7 deletions(-) diff --git a/packages/tron-wallet-snap/src/caching/useCache.test.ts b/packages/tron-wallet-snap/src/caching/useCache.test.ts index c6ef774c..fbfd725c 100644 --- a/packages/tron-wallet-snap/src/caching/useCache.test.ts +++ b/packages/tron-wallet-snap/src/caching/useCache.test.ts @@ -174,15 +174,19 @@ describe('useCache', () => { describe('when the data is cached', () => { it('should return the cached result', async () => { - // Init the cache with some data - jest.spyOn(cache, 'get').mockResolvedValue('test'); + await withUseCache( + async ({ actualExecutionSpy, cache, cachedTestFunction }) => { + // Init the cache with some data + jest.spyOn(cache, 'get').mockResolvedValue('test'); - const result = await cachedTestFunction(); + const result = await cachedTestFunction(); - expect(result).toBe('test'); - expect(cache.get).toHaveBeenCalledTimes(1); - expect(actualExecutionSpy).not.toHaveBeenCalled(); - expect(cache.set).not.toHaveBeenCalled(); + expect(result).toBe('test'); + expect(cache.get).toHaveBeenCalledTimes(1); + expect(actualExecutionSpy).not.toHaveBeenCalled(); + expect(cache.set).not.toHaveBeenCalled(); + }, + ); }); }); From 450560974fee616515c94ba31a016c27abf67362 Mon Sep 17 00:00:00 2001 From: gabrieledm Date: Wed, 26 Aug 2026 15:42:38 +0200 Subject: [PATCH 2/2] test: removed usage of spyOn for cache mock --- packages/tron-wallet-snap/src/caching/useCache.test.ts | 3 ++- 1 file changed, 2 insertions(+), 1 deletion(-) diff --git a/packages/tron-wallet-snap/src/caching/useCache.test.ts b/packages/tron-wallet-snap/src/caching/useCache.test.ts index fbfd725c..f01f182c 100644 --- a/packages/tron-wallet-snap/src/caching/useCache.test.ts +++ b/packages/tron-wallet-snap/src/caching/useCache.test.ts @@ -177,7 +177,8 @@ describe('useCache', () => { await withUseCache( async ({ actualExecutionSpy, cache, cachedTestFunction }) => { // Init the cache with some data - jest.spyOn(cache, 'get').mockResolvedValue('test'); + cache.get.mockResolvedValue('test'); + // jest.spyOn(cache, 'get').mockResolvedValue('test'); const result = await cachedTestFunction();