fix: update debug messages to include event type in RouteHandlerRegistry and Router

Author: arnabrahman

packages/event-handler/src/appsync-graphql/RouteHandlerRegistry.ts

@@ -41,7 +41,7 @@ class RouteHandlerRegistry {
   public register(options: RouteHandlerOptions): void {
     const { fieldName, handler, typeName } = options;
     this.#logger.debug(
-      `Adding resolver ${handler.name} for field ${typeName}.${fieldName}`
+      `Adding ${this.#eventType} resolver for field ${typeName}.${fieldName}`
     );
     const cacheKey = this.#makeKey(typeName, fieldName);
     if (this.resolvers.has(cacheKey)) {
@@ -67,7 +67,7 @@ class RouteHandlerRegistry {
     fieldName: string
   ): RouteHandlerOptions | undefined {
     this.#logger.debug(
-      `Looking for resolver for type=${typeName}, field=${fieldName}`
+      `Looking for ${this.#eventType} resolver for type=${typeName}, field=${fieldName}`
     );
     return this.resolvers.get(this.#makeKey(typeName, fieldName));
   }

packages/event-handler/tests/unit/appsync-graphql/AppSyncGraphQLResolver.test.ts

@@ -112,6 +112,18 @@ describe('Class: AppSyncGraphQLResolver', () => {
     );
 
     // Assess
+    expect(console.debug).toHaveBeenNthCalledWith(
+      1,
+      'Adding onMutation resolver for field Mutation.addPost'
+    );
+    expect(console.debug).toHaveBeenNthCalledWith(
+      2,
+      'Looking for onQuery resolver for type=Mutation, field=addPost'
+    );
+    expect(console.debug).toHaveBeenNthCalledWith(
+      3,
+      'Looking for onMutation resolver for type=Mutation, field=addPost'
+    );
     expect(result).toEqual({
       id: '123',
       title: 'Post Title',

packages/event-handler/tests/unit/appsync-graphql/Router.test.ts

@@ -19,11 +19,11 @@ describe('Class: Router', () => {
     // Assess
     expect(console.debug).toHaveBeenNthCalledWith(
       1,
-      `Adding resolver ${getPost.name} for field Query.getPost`
+      'Adding onQuery resolver for field Query.getPost'
     );
     expect(console.debug).toHaveBeenNthCalledWith(
       2,
-      `Adding resolver ${addPost.name} for field Mutation.addPost`
+      'Adding onMutation resolver for field Mutation.addPost'
     );
   });
 
@@ -64,19 +64,19 @@ describe('Class: Router', () => {
     // Assess
     expect(console.debug).toHaveBeenNthCalledWith(
       1,
-      'Adding resolver getPost for field Query.getPost'
+      'Adding onQuery resolver for field Query.getPost'
     );
     expect(console.debug).toHaveBeenNthCalledWith(
       2,
-      'Adding resolver getAuthor for field Query.getAuthor'
+      'Adding onQuery resolver for field Query.getAuthor'
     );
     expect(console.debug).toHaveBeenNthCalledWith(
       3,
-      'Adding resolver addPost for field Mutation.addPost'
+      'Adding onMutation resolver for field Mutation.addPost'
     );
     expect(console.debug).toHaveBeenNthCalledWith(
       4,
-      'Adding resolver updatePost for field Mutation.updatePost'
+      'Adding onMutation resolver for field Mutation.updatePost'
     );
 
     // verify that class scope is preserved after decorating
@@ -106,11 +106,11 @@ describe('Class: Router', () => {
     // Assess
     expect(console.debug).toHaveBeenNthCalledWith(
       1,
-      'Adding resolver getLocations for field Query.locations'
+      'Adding onQuery resolver for field Query.locations'
     );
     expect(console.debug).toHaveBeenNthCalledWith(
       2,
-      'Adding resolver getLocations for field Query.listLocations'
+      'Adding onQuery resolver for field Query.listLocations'
     );
 
     expect(response).toEqual([