fix: trigger analytics migration on update/configure notifications (#…

…11285)

* fix: trigger analytics migration on update/configure notifications

packages/amplify-category-notifications/src/__tests__/notifications.test.ts

@@ -0,0 +1,42 @@
+import { $TSContext } from 'amplify-cli-core';
+import { migrationCheck } from '../migrations/index';
+import * as apiAnalyticsClient from '../plugin-client-api-analytics';
+
+const mockContext = {
+  input: { command: undefined },
+} as unknown as $TSContext;
+
+jest.mock('../plugin-client-api-analytics', () => ({ invokeAnalyticsMigrations: jest.fn() }));
+
+describe('notifications tests', () => {
+  beforeEach(() => {
+    jest.resetAllMocks();
+  });
+
+  test('notification migrations calls analytics migration', async () => {
+    mockContext.input.command = 'add';
+    await migrationCheck(mockContext);
+    expect(apiAnalyticsClient.invokeAnalyticsMigrations).toBeCalled();
+    jest.resetAllMocks();
+
+    mockContext.input.command = 'configure';
+    await migrationCheck(mockContext);
+    expect(apiAnalyticsClient.invokeAnalyticsMigrations).toBeCalled();
+    jest.resetAllMocks();
+
+    mockContext.input.command = 'update';
+    await migrationCheck(mockContext);
+    expect(apiAnalyticsClient.invokeAnalyticsMigrations).toBeCalled();
+    jest.resetAllMocks();
+
+    mockContext.input.command = 'push';
+    await migrationCheck(mockContext);
+    expect(apiAnalyticsClient.invokeAnalyticsMigrations).toBeCalled();
+    jest.resetAllMocks();
+
+    mockContext.input.command = 'remove';
+    await migrationCheck(mockContext);
+    expect(apiAnalyticsClient.invokeAnalyticsMigrations).not.toBeCalled();
+    jest.resetAllMocks();
+  });
+});

packages/amplify-category-notifications/src/migrations/index.ts

@@ -5,7 +5,7 @@ import { invokeAnalyticsMigrations } from '../plugin-client-api-analytics';
  * checks if the project has been migrated to the latest version
  */
 export const migrationCheck = async (context: $TSContext): Promise<void> => {
-  if (['add', 'update', 'push'].includes(context.input.command)) {
+  if (['add', 'configure', 'update', 'push'].includes(context.input.command)) {
     await invokeAnalyticsMigrations(context);
   }
 };