fix(MetricFactory): Metric Search with multiple namespaces are wrongly wrapped (#637)

This change splits the namespaces to wrap them individually.

Fixes #623

---

_By submitting this pull request, I confirm that my contribution is made
under the terms of the Apache-2.0 license_

Co-authored-by: Martin Sommer <[email protected]>

Author: SeeSharpSoft

lib/common/metric/MetricFactory.ts

@@ -149,11 +149,14 @@ export class MetricFactory {
     const finalPeriod =
       period ?? this.globalDefaults.period ?? DefaultMetricPeriod;
     const searchNamespace = this.getNamespaceWithFallback(namespace);
+    const namespacesWithQuotations = searchNamespace
+      .split(",")
+      .map((namespace) => `"${namespace.trim()}"`);
     const keysWithQuotations = Object.keys(dimensionsMap).map(
       (key) => `"${key}"`,
     );
     const namespacePlusDimensionKeys = [
-      `"${searchNamespace}"`,
+      ...namespacesWithQuotations,
       ...keysWithQuotations,
     ].join(",");
     const metricSchema = `{${namespacePlusDimensionKeys}}`;

test/common/metric/MetricFactory.test.ts

@@ -328,6 +328,15 @@ test("snapshot test: createMetricSearch", () => {
   );
 
   expect(metricWithEmptyDimensions).toMatchSnapshot();
+
+  const metricWithMultipleNamespaces = metricFactory.createMetricSearch(
+    "MyMetricPrefix-",
+    { DummyDimension: "DummyDimensionValue" },
+    MetricStatistic.SUM,
+    "DummyNamespaceOverride1, DummyNamespaceOverride2 ,DummyNamespaceOverride3",
+  );
+
+  expect(metricWithMultipleNamespaces).toMatchSnapshot();
 });
 
 test("snapshot test: createMetricAnomalyDetection", () => {