test(extension): Fix tests

Author: rekmarks

packages/extension/src/kernel-integration/handlers/execute-db-query.test.ts

@@ -3,10 +3,10 @@ import { describe, it, expect, vi } from 'vitest';
 import { executeDBQueryHandler } from './execute-db-query.ts';
 
 describe('executeDBQueryHandler', () => {
-  it('executes a database query', () => {
+  it('executes a database query', async () => {
     const mockExecuteDBQuery = vi.fn().mockReturnValueOnce([{ key: 'value' }]);
 
-    const result = executeDBQueryHandler.implementation(
+    const result = await executeDBQueryHandler.implementation(
       { executeDBQuery: mockExecuteDBQuery },
       {
         sql: 'test-query',
@@ -17,19 +17,17 @@ describe('executeDBQueryHandler', () => {
     expect(result).toStrictEqual([{ key: 'value' }]);
   });
 
-  it('should propagate errors from executeDBQuery', () => {
+  it('should propagate errors from executeDBQuery', async () => {
     const error = new Error('Query failed');
     const mockExecuteDBQuery = vi.fn().mockImplementationOnce(() => {
       throw error;
     });
 
-    // TODO:rekm Fix upstream types to allow sync handlers
-    // eslint-disable-next-line @typescript-eslint/promise-function-async
-    expect(() =>
+    await expect(
       executeDBQueryHandler.implementation(
         { executeDBQuery: mockExecuteDBQuery },
         { sql: 'test-query' },
       ),
-    ).toThrow(error);
+    ).rejects.toThrow(error);
   });
 });

packages/extension/src/kernel-integration/handlers/execute-db-query.ts

@@ -25,10 +25,10 @@ export const executeDBQueryHandler: Handler<
 > = {
   ...executeDBQuerySpec,
   hooks: { executeDBQuery: true },
-  implementation: (
+  implementation: async (
     { executeDBQuery }: ExecuteDBQueryHooks,
     params: { sql: string },
-  ): Record<string, string>[] => {
+  ): Promise<Record<string, string>[]> => {
     return executeDBQuery(params.sql);
   },
 };

packages/extension/src/kernel-integration/handlers/index.ts

@@ -15,7 +15,7 @@ export const handlers = {
   sendVatCommand: sendVatCommandHandler,
   executeDBQuery: executeDBQueryHandler,
   launchVat: launchVatHandler,
-  reloadConfig: reloadConfigHandler,
+  reload: reloadConfigHandler,
   restartVat: restartVatHandler,
   terminateVat: terminateVatHandler,
   terminateAllVats: terminateAllVatsHandler,