fix: throw AmplifyUserError instead of Error

aws-amplify · Dec 14, 2024 · 847124a · 847124a
1 parent 6c8d04f
commit 847124a
Show file tree

Hide file tree

Showing 2 changed files with 16 additions and 12 deletions.
diff --git a/packages/backend-function/src/factory.test.ts b/packages/backend-function/src/factory.test.ts
@@ -714,9 +714,10 @@ void describe('AmplifyFunctionFactory', () => {
             entry: './test-assets/default-lambda/handler.ts',
             ephemeralStorageSizeMB: 511,
           }).getInstance(getInstanceProps),
-        new Error(
-          'ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive'
-        )
+        new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
+          message: `Invalid function ephemeralStorageSizeMB of 511`,
+          resolution: `ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive`,
+        })
       );
     });
 
@@ -727,9 +728,10 @@ void describe('AmplifyFunctionFactory', () => {
             entry: './test-assets/default-lambda/handler.ts',
             ephemeralStorageSizeMB: 10241,
           }).getInstance(getInstanceProps),
-        new Error(
-          'ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive'
-        )
+        new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
+          message: `Invalid function ephemeralStorageSizeMB of 10241`,
+          resolution: `ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive`,
+        })
       );
     });
 
@@ -740,9 +742,10 @@ void describe('AmplifyFunctionFactory', () => {
             entry: './test-assets/default-lambda/handler.ts',
             ephemeralStorageSizeMB: 512.5,
           }).getInstance(getInstanceProps),
-        new Error(
-          'ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive'
-        )
+        new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
+          message: `Invalid function ephemeralStorageSizeMB of 512.5`,
+          resolution: `ephemeralStorageSizeMB must be a whole number between 512 and 10240 inclusive`,
+        })
       );
     });
   });

diff --git a/packages/backend-function/src/factory.ts b/packages/backend-function/src/factory.ts
@@ -346,9 +346,10 @@ class FunctionFactory implements ConstructFactory<AmplifyFunction> {
         ephemeralStorageSizeMax
       )
     ) {
-      throw new Error(
-        `ephemeralStorageSizeMB must be a whole number between ${ephemeralStorageSizeMin} and ${ephemeralStorageSizeMax} inclusive`
-      );
+      throw new AmplifyUserError('InvalidEphemeralStorageSizeMBError', {
+        message: `Invalid function ephemeralStorageSizeMB of ${this.props.ephemeralStorageSizeMB}`,
+        resolution: `ephemeralStorageSizeMB must be a whole number between ${ephemeralStorageSizeMin} and ${ephemeralStorageSizeMax} inclusive`,
+      });
     }
     return this.props.ephemeralStorageSizeMB;
   };