[v9.3.x] AzureMonitor: Fix dimension migration (#62578)

AzureMonitor: Fix dimension migration (#62485)

Remove unneeded properties post migration

(cherry picked from commit f77853f91e)

Co-authored-by: Andreas Christou <andreas.christou@grafana.com>

public/app/plugins/datasource/grafana-azure-monitor-datasource/utils/migrateQuery.test.ts

@@ -151,6 +151,27 @@ describe('AzureMonitor: migrateQuery', () => {
         })
       );
     });
+    it('correctly removes outdated fields', () => {
+      const result = migrateQuery({
+        ...azureMonitorQueryV8,
+        azureMonitor: { dimension: 'testDimension', dimensionFilter: 'testFilter' },
+      });
+      expect(result).toMatchObject(
+        expect.objectContaining({
+          azureMonitor: expect.objectContaining({
+            dimensionFilters: [
+              {
+                dimension: 'testDimension',
+                operator: 'eq',
+                filters: ['testFilter'],
+              },
+            ],
+          }),
+        })
+      );
+      expect(result.azureMonitor).not.toHaveProperty('dimension');
+      expect(result.azureMonitor).not.toHaveProperty('dimensionFilter');
+    });
 
     it('correctly migrates a metric definition', () => {
       const result = migrateQuery({ ...azureMonitorQueryV8, azureMonitor: { metricDefinition: 'ms.ns/mn' } });

public/app/plugins/datasource/grafana-azure-monitor-datasource/utils/migrateQuery.ts

@@ -95,6 +95,9 @@ function migrateDimensionToDimensionFilter(query: AzureMonitorQuery): AzureMonit
     ]);
   }
 
+  delete workingQuery.azureMonitor?.dimension;
+  delete workingQuery.azureMonitor?.dimensionFilter;
+
   return workingQuery;
 }