chore: fix tests

logto-io · Sep 9, 2022 · 9deef19 · 9deef19
1 parent dfc8277
commit 9deef19
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 10 deletions.
diff --git a/packages/react/src/hooks/index.test.tsx b/packages/react/src/hooks/index.test.tsx
@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-empty-function */
 import LogtoClient from '@logto/browser';
 import { renderHook, act } from '@testing-library/react-hooks';
 import { ComponentType } from 'react';
@@ -8,7 +7,7 @@ import { LogtoProvider } from '../provider';
 
 const isAuthenticated = jest.fn(() => false);
 const isSignInRedirected = jest.fn(() => false);
-const handleSignInCallback = jest.fn(async () => 0);
+const handleSignInCallback = jest.fn().mockResolvedValue(true);
 const getAccessToken = jest.fn(() => {
   throw new Error('not authenticated');
 });
@@ -20,8 +19,8 @@ jest.mock('@logto/browser', () => {
       isSignInRedirected,
       handleSignInCallback,
       getAccessToken,
-      signIn: jest.fn(async () => {}),
-      signOut: jest.fn(async () => {}),
+      signIn: jest.fn().mockResolvedValue(true),
+      signOut: jest.fn().mockResolvedValue(true),
     };
   });
 });
@@ -111,4 +110,3 @@ describe('useLogto', () => {
     });
   });
 });
-/* eslint-enable @typescript-eslint/no-empty-function */
diff --git a/packages/vue/src/index.test.ts b/packages/vue/src/index.test.ts
@@ -1,4 +1,3 @@
-/* eslint-disable @typescript-eslint/no-empty-function */
 import LogtoClient from '@logto/browser';
 import { App, readonly } from 'vue';
 
@@ -9,7 +8,7 @@ import { createPluginMethods } from './plugin';
 
 const isAuthenticated = jest.fn(async () => false);
 const isSignInRedirected = jest.fn(() => false);
-const handleSignInCallback = jest.fn(async () => {});
+const handleSignInCallback = jest.fn().mockResolvedValue(true);
 const getAccessToken = jest.fn(() => {
   throw new Error('not authenticated');
 });
@@ -24,8 +23,8 @@ jest.mock('@logto/browser', () => {
       isSignInRedirected,
       handleSignInCallback,
       getAccessToken,
-      signIn: jest.fn(async () => {}),
-      signOut: jest.fn(async () => {}),
+      signIn: jest.fn().mockResolvedValue(true),
+      signOut: jest.fn().mockResolvedValue(true),
     };
   });
 });
@@ -151,4 +150,3 @@ describe('useHandleSignInCallback', () => {
     expect(handleSignInCallback).toHaveBeenCalledTimes(1);
   });
 });
-/* eslint-enable @typescript-eslint/no-empty-function */