scriptoriumadmin/Roo-Code
1
0
Fork 0
mirror of https://github.com/RooVetGit/Roo-Code.git synced 2026-09-14 23:21:19 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions

Fix TypeScript type checking errors in authentication tests

Browse source 
- Replace vi.MockedFunction<any> with any for simpler typing
- Add optional chaining for potentially undefined array access
- Ensure tests pass TypeScript strict checking
This commit is contained in:
Roo Code 2025-07-08 11:53:31 +00:00
parent 16eddd37eb
commit 1051bb684f
2 changed files with 18 additions and 18 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

18
apps/web/src/actions/analytics/__tests__/events-auth.test.ts
View file

@ -40,13 +40,13 @@ vi.mock('@/types', () => ({
}));
describe('Share Token Authorization in Events', () => {
let mockAnalytics: vi.MockedFunction<any>;
let mockAuthorizeAnalytics: vi.MockedFunction<any>;
let mockGetUsersById: vi.MockedFunction<any>;
let mockDb: vi.MockedFunction<any>;
let mockIsValidShareToken: vi.MockedFunction<any>;
let mockIsShareExpired: vi.MockedFunction<any>;
let mockAuth: vi.MockedFunction<any>;
let mockAnalytics: any;
let mockAuthorizeAnalytics: any;
let mockGetUsersById: any;
let mockDb: any;
let mockIsValidShareToken: any;
let mockIsShareExpired: any;
let mockAuth: any;
beforeEach(async () => {
vi.clearAllMocks();
@ -228,7 +228,7 @@ describe('Share Token Authorization in Events', () => {
});
expect(result.tasks).toHaveLength(1);
expect(result.tasks[0].taskId).toBe('task-123');
expect(result.tasks?.[0]?.taskId).toBe('task-123');
});
it('should allow access for valid public share', async () => {
@ -257,7 +257,7 @@ describe('Share Token Authorization in Events', () => {
});
expect(result.tasks).toHaveLength(1);
expect(result.tasks[0].taskId).toBe('task-123');
expect(result.tasks?.[0]?.taskId).toBe('task-123');
// Should not call auth for public shares
expect(mockAuth).not.toHaveBeenCalled();
});

18
apps/web/src/actions/analytics/__tests__/messages-auth.test.ts
View file

@ -39,12 +39,12 @@ vi.mock('@/types', () => ({
}));
describe('Share Token Authorization in Messages', () => {
let mockAnalytics: vi.MockedFunction<any>;
let mockAuthorizeAnalytics: vi.MockedFunction<any>;
let mockDb: vi.MockedFunction<any>;
let mockIsValidShareToken: vi.MockedFunction<any>;
let mockIsShareExpired: vi.MockedFunction<any>;
let mockAuth: vi.MockedFunction<any>;
let mockAnalytics: any;
let mockAuthorizeAnalytics: any;
let mockDb: any;
let mockIsValidShareToken: any;
let mockIsShareExpired: any;
let mockAuth: any;
beforeEach(async () => {
vi.clearAllMocks();
@ -242,8 +242,8 @@ describe('Share Token Authorization in Messages', () => {
);
expect(result).toHaveLength(2);
expect(result[0].taskId).toBe('task-123');
expect(result[1].taskId).toBe('task-123');
expect(result?.[0]?.taskId).toBe('task-123');
expect(result?.[1]?.taskId).toBe('task-123');
});
it('should allow access for valid public share', async () => {
@ -274,7 +274,7 @@ describe('Share Token Authorization in Messages', () => {
);
expect(result).toHaveLength(2);
expect(result[0].taskId).toBe('task-123');
expect(result?.[0]?.taskId).toBe('task-123');
// Should not call auth for public shares
expect(mockAuth).not.toHaveBeenCalled();
});