fix(core): not appending notification configs properly (#12934)

aws-amplify · Feb 1, 2024 · d7900dc · d7900dc
2 parents b893b0c + cb7040f
commit d7900dc
Show file tree

Hide file tree

Showing 2 changed files with 40 additions and 0 deletions.
diff --git a/packages/core/__tests__/parseAWSExports.test.ts b/packages/core/__tests__/parseAWSExports.test.ts
@@ -262,4 +262,43 @@ describe('parseAWSExports', () => {
 			'Invalid config parameter.'
 		);
 	});
+	it('should append Notification configs when both Push and InApp configs are available', () => {
+		const testConfig = {
+			aws_project_region: 'us-west-2',
+			aws_user_pools_id: userPoolId,
+			Notifications: {
+				Push: {
+					AWSPinpoint: {
+						appId: "appId",
+						region: "region"
+					}
+				},
+				InAppMessaging: {
+					AWSPinpoint: {
+						appId: "appId",
+						region: "region"
+					}
+				}
+			}
+		};
+
+		expect(parseAWSExports(testConfig)).toMatchObject(
+			{
+				Notifications: {
+					PushNotification: {
+						Pinpoint: {
+							appId: "appId",
+							region: "region"
+						}
+					},
+					InAppMessaging: {
+						Pinpoint: {
+							appId: "appId",
+							region: "region"
+						}
+					}
+				}	
+			}
+		);
+	});
 });
diff --git a/packages/core/src/parseAWSExports.ts b/packages/core/src/parseAWSExports.ts
@@ -107,6 +107,7 @@ export const parseAWSExports = (
 		if (Push?.AWSPinpoint) {
 			const { appId, region } = Push.AWSPinpoint;
 			amplifyConfig.Notifications = {
+				...amplifyConfig.Notifications,
 				PushNotification: {
 					Pinpoint: {
 						appId,