[perf] Migrate @azure-tests/perf-ai-metrics-advisor to ESM (#32209)

### Packages impacted by this PR - @azure-tests/perf-ai-metrics-advisor ### Issues associated with this PR - #31338 ### Describe the problem that is addressed by this PR Migrates @azure-tests/perf-ai-metrics-advisor to ESM via automation. ### What are the possible designs available to address the problem? If there are more than one possible design, why was the one in this PR chosen? ### Are there test cases added in this PR? _(If not, why?)_ ### Provide a list of related PRs _(if any)_ ### Command used to generate this PR:**_(Applicable only to SDK release request PRs)_ ### Checklists - [ ] Added impacted package name to the issue description - [ ] Does this PR needs any fixes in the SDK Generator?** _(If so, create an Issue in the [Autorest/typescript](https://github.com/Azure/autorest.typescript) repository and link it here)_ - [ ] Added a changelog (if necessary)
Azure · Dec 17, 2024 · 84169fa · 84169fa
1 parent 6ab19e8
commit 84169fa
Show file tree

Hide file tree

Showing 7 changed files with 72 additions and 25 deletions.
diff --git a/sdk/metricsadvisor/perf-tests/ai-metrics-advisor/package.json b/sdk/metricsadvisor/perf-tests/ai-metrics-advisor/package.json
@@ -3,44 +3,75 @@
   "sdk-type": "perf-test",
   "version": "1.0.0",
   "description": "",
-  "main": "",
+  "main": "./dist/commonjs/index.js",
   "keywords": [],
-  "author": "",
-  "license": "ISC",
+  "author": "Microsoft Corporation",
+  "license": "MIT",
   "dependencies": {
     "@azure-tools/test-perf": "^1.0.0",
     "@azure/ai-metrics-advisor": "^1.0.0",
-    "dotenv": "^16.0.0"
+    "dotenv": "^16.0.0",
+    "tslib": "^2.8.1"
   },
   "devDependencies": {
     "@azure/dev-tool": "^1.0.0",
     "@types/node": "^18.0.0",
     "eslint": "^9.9.0",
-    "ts-node": "^10.0.0",
-    "tslib": "^2.2.0",
     "typescript": "~5.6.2"
   },
   "private": true,
   "scripts": {
-    "build": "npm run clean && tsc -p .",
+    "build": "npm run clean && dev-tool run build-package",
     "build:samples": "echo skipped",
     "build:test": "echo skipped",
-    "check-format": "dev-tool run vendored prettier --list-different --config ../../../../.prettierrc.json --ignore-path ../../../../.prettierignore \"test/**/*.ts\" \"*.{js,json}\"",
+    "check-format": "dev-tool run vendored prettier --list-different --config ../../../../.prettierrc.json --ignore-path ../../../../.prettierignore \"src/**/*.ts\" \"*.{js,json}\"",
     "clean": "dev-tool run vendored rimraf --glob dist dist-esm test-dist types *.tgz *.log",
-    "format": "dev-tool run vendored prettier --write --config ../../../../.prettierrc.json --ignore-path ../../../../.prettierignore \"test/**/*.ts\" \"*.{js,json}\"",
+    "format": "dev-tool run vendored prettier --write --config ../../../../.prettierrc.json --ignore-path ../../../../.prettierignore \"src/**/*.ts\" \"*.{js,json}\"",
     "integration-test": "echo skipped",
     "integration-test:browser": "echo skipped",
     "integration-test:node": "echo skipped",
-    "lint": "eslint -c ../../../../common/tools/eslint-plugin-azure-sdk/eslint.perftests.config.mjs test",
-    "lint:fix": "eslint -c ../../../../common/tools/eslint-plugin-azure-sdk/eslint.perftests.config.mjs test --fix --fix-type [problem,suggestion]",
+    "lint": "dev-tool run vendored eslint -c ../../../../common/tools/eslint-plugin-azure-sdk/eslint.perftests.config.mjs src",
+    "lint:fix": "dev-tool run vendored eslint -c ../../../../common/tools/eslint-plugin-azure-sdk/eslint.perftests.config.mjs src --fix --fix-type [problem,suggestion]",
     "pack": "npm pack 2>&1",
-    "perf-test:node": "npm run build && node dist-esm/index.spec.js",
+    "perf-test:node": "npm run build && node dist/esm/index.js",
     "test": "echo skipped",
     "test:browser": "echo skipped",
     "test:node": "echo skipped",
     "unit-test": "echo skipped",
     "unit-test:browser": "echo skipped",
     "unit-test:node": "echo skipped",
     "update-snippets": "echo skipped"
-  }
+  },
+  "type": "module",
+  "tshy": {
+    "exports": {
+      "./package.json": "./package.json",
+      ".": "./src/index.ts"
+    },
+    "dialects": [
+      "esm",
+      "commonjs"
+    ],
+    "selfLink": false
+  },
+  "files": [
+    "dist/",
+    "README.md",
+    "LICENSE"
+  ],
+  "exports": {
+    "./package.json": "./package.json",
+    ".": {
+      "import": {
+        "types": "./dist/esm/index.d.ts",
+        "default": "./dist/esm/index.js"
+      },
+      "require": {
+        "types": "./dist/commonjs/index.d.ts",
+        "default": "./dist/commonjs/index.js"
+      }
+    }
+  },
+  "types": "./dist/commonjs/index.d.ts",
+  "module": "./dist/esm/index.js"
 }
diff --git a/...sts/ai-metrics-advisor/test/index.spec.ts → ...erf-tests/ai-metrics-advisor/src/index.ts b/...sts/ai-metrics-advisor/test/index.spec.ts → ...erf-tests/ai-metrics-advisor/src/index.ts
@@ -2,9 +2,9 @@
 // Licensed under the MIT License.
 
 import { createPerfProgram } from "@azure-tools/test-perf";
-import { AnomaliesListTest } from "./listAnomalies.spec";
-import { IncidentsListTest } from "./listIncidents.spec";
-import { RootCauseTest } from "./rootCauses.spec";
+import { AnomaliesListTest } from "./listAnomalies.spec.js";
+import { IncidentsListTest } from "./listIncidents.spec.js";
+import { RootCauseTest } from "./rootCauses.spec.js";
 
 const perfProgram = createPerfProgram(AnomaliesListTest, IncidentsListTest, RootCauseTest);
 perfProgram.run();
diff --git a/...etrics-advisor/test/listAnomalies.spec.ts → ...metrics-advisor/src/listAnomalies.spec.ts b/...etrics-advisor/test/listAnomalies.spec.ts → ...metrics-advisor/src/listAnomalies.spec.ts
@@ -1,7 +1,9 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
+
 import { PerfOptionDictionary, getEnvVar } from "@azure-tools/test-perf";
-import { MetricsAdvisorTest } from "./metricsAdvisor.spec";
+import { MetricsAdvisorTest } from "./metricsAdvisor.spec.js";
+
 type MetricsAdvisorTestOptions = Record<string, unknown>;
 
 export class AnomaliesListTest extends MetricsAdvisorTest<MetricsAdvisorTestOptions> {

diff --git a/...etrics-advisor/test/listIncidents.spec.ts → ...metrics-advisor/src/listIncidents.spec.ts b/...etrics-advisor/test/listIncidents.spec.ts → ...metrics-advisor/src/listIncidents.spec.ts
@@ -1,7 +1,9 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
+
 import { PerfOptionDictionary, getEnvVar } from "@azure-tools/test-perf";
-import { MetricsAdvisorTest } from "./metricsAdvisor.spec";
+import { MetricsAdvisorTest } from "./metricsAdvisor.spec.js";
+
 type MetricsAdvisorTestOptions = Record<string, unknown>;
 
 export class IncidentsListTest extends MetricsAdvisorTest<MetricsAdvisorTestOptions> {

diff --git a/...trics-advisor/test/metricsAdvisor.spec.ts → ...etrics-advisor/src/metricsAdvisor.spec.ts b/...trics-advisor/test/metricsAdvisor.spec.ts → ...etrics-advisor/src/metricsAdvisor.spec.ts
@@ -5,8 +5,7 @@ import { PerfTest, getEnvVar } from "@azure-tools/test-perf";
 import { MetricsAdvisorClient, MetricsAdvisorKeyCredential } from "@azure/ai-metrics-advisor";
 
 // Expects the .env file at the same level
-import * as dotenv from "dotenv";
-dotenv.config();
+import "dotenv/config";
 
 export abstract class MetricsAdvisorTest<TOptions> extends PerfTest<TOptions> {
   client: MetricsAdvisorClient;

diff --git a/...i-metrics-advisor/test/rootCauses.spec.ts → ...ai-metrics-advisor/src/rootCauses.spec.ts b/...i-metrics-advisor/test/rootCauses.spec.ts → ...ai-metrics-advisor/src/rootCauses.spec.ts
@@ -1,7 +1,9 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
+
 import { PerfOptionDictionary, getEnvVar } from "@azure-tools/test-perf";
-import { MetricsAdvisorTest } from "./metricsAdvisor.spec";
+import { MetricsAdvisorTest } from "./metricsAdvisor.spec.js";
+
 type MetricsAdvisorTestOptions = Record<string, unknown>;
 
 export class RootCauseTest extends MetricsAdvisorTest<MetricsAdvisorTestOptions> {

diff --git a/sdk/metricsadvisor/perf-tests/ai-metrics-advisor/tsconfig.json b/sdk/metricsadvisor/perf-tests/ai-metrics-advisor/tsconfig.json
@@ -1,10 +1,21 @@
 {
   "extends": "../../../../tsconfig",
   "compilerOptions": {
-    "module": "CommonJS",
-    "lib": ["ES6", "ESNext.AsyncIterable"],
-    "noEmit": true
+    "module": "NodeNext",
+    "moduleResolution": "NodeNext",
+    "rootDir": ".",
+    "paths": {
+      "@azure-tests/perf-ai-metrics-advisor": ["./src/index.ts"]
+    }
   },
   "compileOnSave": true,
-  "include": ["./test/**/*.ts"]
+  "include": [
+    "src/**/*.ts",
+    "src/**/*.mts",
+    "src/**/*.cts",
+    "samples-dev/**/*.ts",
+    "test/**/*.ts",
+    "test/**/*.mts",
+    "test/**/*.cts"
+  ]
 }