Skip to content

Commit 847124a

Browse files
committed
fix: throw AmplifyUserError instead of Error
1 parent 6c8d04f commit 847124a

File tree

2 files changed

+16
-12
lines changed

2 files changed

+16
-12
lines changed

packages/backend-function/src/factory.test.ts

Lines changed: 12 additions & 9 deletions
Original file line numberDiff line numberDiff line change
@@ -714,9 +714,10 @@ void describe('AmplifyFunctionFactory', () => {
714714
entry: './test-assets/default-lambda/handler.ts',
715715
ephemeralStorageSizeMB: 511,
716716
}).getInstance(getInstanceProps),
717-
new Error(
718-
'ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive'
719-
)
717+
new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
718+
message: `Invalid function ephemeralStorageSizeMB of 511`,
719+
resolution: `ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive`,
720+
})
720721
);
721722
});
722723

@@ -727,9 +728,10 @@ void describe('AmplifyFunctionFactory', () => {
727728
entry: './test-assets/default-lambda/handler.ts',
728729
ephemeralStorageSizeMB: 10241,
729730
}).getInstance(getInstanceProps),
730-
new Error(
731-
'ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive'
732-
)
731+
new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
732+
message: `Invalid function ephemeralStorageSizeMB of 10241`,
733+
resolution: `ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive`,
734+
})
733735
);
734736
});
735737

@@ -740,9 +742,10 @@ void describe('AmplifyFunctionFactory', () => {
740742
entry: './test-assets/default-lambda/handler.ts',
741743
ephemeralStorageSizeMB: 512.5,
742744
}).getInstance(getInstanceProps),
743-
new Error(
744-
'ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive'
745-
)
745+
new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
746+
message: `Invalid function ephemeralStorageSizeMB of 512.5`,
747+
resolution: `ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive`,
748+
})
746749
);
747750
});
748751
});

packages/backend-function/src/factory.ts

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -346,9 +346,10 @@ class FunctionFactory implements ConstructFactory<AmplifyFunction> {
346346
ephemeralStorageSizeMax
347347
)
348348
) {
349-
throw new Error(
350-
`ephemeralStorageSizeMB must be a whole number between ${ephemeralStorageSizeMin} and ${ephemeralStorageSizeMax} inclusive`
351-
);
349+
throw new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
350+
message: `Invalid function ephemeralStorageSizeMB of ${this.props.ephemeralStorageSizeMB}`,
351+
resolution: `ephemeralStorageSizeMB must be a whole number between ${ephemeralStorageSizeMin} and ${ephemeralStorageSizeMax} inclusive`,
352+
});
352353
}
353354
return this.props.ephemeralStorageSizeMB;
354355
};

0 commit comments

Comments
 (0)