Skip to content

Commit ad1793e

Browse files
committed
fix: update test descriptions to clarify exception handling behavior in AppSyncGraphQLResolver
1 parent aac7f9f commit ad1793e

File tree

1 file changed

+6
-6
lines changed

1 file changed

+6
-6
lines changed

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

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -762,7 +762,7 @@ describe('Class: AppSyncGraphQLResolver', () => {
762762
message: 'Aggregation failed',
763763
},
764764
])(
765-
'should register handler for %s',
765+
'should invoke exception handler for %s',
766766
async ({
767767
errorClass,
768768
message,
@@ -801,7 +801,7 @@ describe('Class: AppSyncGraphQLResolver', () => {
801801
}
802802
);
803803

804-
it('should handle multiple different error types with specific handlers', async () => {
804+
it('should handle multiple different error types with specific exception handlers', async () => {
805805
// Prepare
806806
const app = new AppSyncGraphQLResolver();
807807

@@ -950,7 +950,7 @@ describe('Class: AppSyncGraphQLResolver', () => {
950950
);
951951
});
952952

953-
it('should work with sync exception handlers', async () => {
953+
it('should invoke sync exception handlers and return their result', async () => {
954954
// Prepare
955955
const app = new AppSyncGraphQLResolver();
956956

@@ -976,11 +976,11 @@ describe('Class: AppSyncGraphQLResolver', () => {
976976
expect(result).toEqual({
977977
message: 'Sync validation failed',
978978
details: 'Sync error test',
979-
type: 'async_validation_error',
979+
type: 'sync_validation_error',
980980
});
981981
});
982982

983-
it('should not interfere with ResolverNotFoundException', async () => {
983+
it('does not catch ResolverNotFoundException with unrelated exception handlers', async () => {
984984
// Prepare
985985
const app = new AppSyncGraphQLResolver();
986986

@@ -1001,7 +1001,7 @@ describe('Class: AppSyncGraphQLResolver', () => {
10011001
).rejects.toThrow('No resolver found for Query-nonExistentResolver');
10021002
});
10031003

1004-
it('should work as a method decorator', async () => {
1004+
it('invokes exception handler when used as a method decorator', async () => {
10051005
// Prepare
10061006
const app = new AppSyncGraphQLResolver();
10071007

0 commit comments

Comments
 (0)