Skip to content

Commit

Permalink
[perf] Migrate @azure-tests/perf-monitor-query to ESM (#32256)
Browse files Browse the repository at this point in the history
### Packages impacted by this PR

- @azure-tests/perf-monitor-query 

### Issues associated with this PR

- #31338

### Describe the problem that is addressed by this PR

Migrating @azure-tests/perf-monitor-query 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)
  • Loading branch information
mpodwysocki authored Dec 17, 2024
1 parent 4c39cfa commit 708295a
Show file tree
Hide file tree
Showing 8 changed files with 71 additions and 32 deletions.
59 changes: 45 additions & 14 deletions sdk/monitor/perf-tests/monitor-query/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -3,45 +3,76 @@
"sdk-type": "perf-test",
"version": "1.0.0",
"description": "",
"main": "",
"main": "./dist/commonjs/index.js",
"module": "./dist/esm/index.js",
"keywords": [],
"author": "",
"license": "ISC",
"author": "Microsoft Corporation",
"license": "MIT",
"dependencies": {
"@azure-tools/test-perf": "^1.0.0",
"@azure/identity": "^4.0.1",
"@azure/identity": "^4.5.0",
"@azure/monitor-query": "1.3.1",
"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"
}
Original file line number Diff line number Diff line change
@@ -1,9 +1,9 @@
// Copyright (c) Microsoft Corporation.
// Licensed under the MIT License.

import { MetricsQueryTest } from "./metricQuery.spec";
import { LogQueryBatchTest } from "./logQueryBatch.spec";
import { LogQueryTest } from "./logQuery.spec";
import { MetricsQueryTest } from "./metricQuery.spec.js";
import { LogQueryBatchTest } from "./logQueryBatch.spec.js";
import { LogQueryTest } from "./logQuery.spec.js";
import { createPerfProgram } from "@azure-tools/test-perf";

const perfProgram = createPerfProgram(MetricsQueryTest, LogQueryBatchTest, LogQueryTest);
Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
// Copyright (c) Microsoft Corporation.
// Licensed under the MIT License.

import { PerfOptionDictionary, getEnvVar } from "@azure-tools/test-perf";
import { MonitorQueryLog } from "./monitorQueryLog.spec";
import { MonitorQueryLog } from "./monitorQueryLog.spec.js";

type MonitorQueryTestOptions = Record<string, unknown>;

Expand Down
Original file line number Diff line number Diff line change
@@ -1,7 +1,8 @@
// Copyright (c) Microsoft Corporation.
// Licensed under the MIT License.

import { PerfOptionDictionary, getEnvVar } from "@azure-tools/test-perf";
import { MonitorQueryLog } from "./monitorQueryLog.spec";
import { MonitorQueryLog } from "./monitorQueryLog.spec.js";
import { QueryBatch } from "@azure/monitor-query";

type MonitorQueryTestOptions = Record<string, unknown>;
Expand Down
Original file line number Diff line number Diff line change
@@ -1,8 +1,9 @@
// Copyright (c) Microsoft Corporation.
// Licensed under the MIT License.

import { PerfOptionDictionary, getEnvVar } from "@azure-tools/test-perf";
import { AggregationType } from "@azure/monitor-query";
import { MonitorQueryMetrics } from "./monitorQueryMetrics.spec";
import { MonitorQueryMetrics } from "./monitorQueryMetrics.spec.js";

type MonitorQueryTestOptions = Record<string, unknown>;

Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,7 @@
import { PerfTest } from "@azure-tools/test-perf";
import { LogsQueryClient } from "@azure/monitor-query";
import { DefaultAzureCredential } from "@azure/identity";

// Expects the .env file at the same level
import * as dotenv from "dotenv";
dotenv.config();
import "dotenv/config";

export abstract class MonitorQueryLog<TOptions> extends PerfTest<TOptions> {
client: LogsQueryClient;
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -4,10 +4,7 @@
import { PerfTest } from "@azure-tools/test-perf";
import { MetricsQueryClient } from "@azure/monitor-query";
import { DefaultAzureCredential } from "@azure/identity";

// Expects the .env file at the same level
import * as dotenv from "dotenv";
dotenv.config();
import "dotenv/config";

export abstract class MonitorQueryMetrics<TOptions> extends PerfTest<TOptions> {
client: MetricsQueryClient;
Expand Down
19 changes: 15 additions & 4 deletions sdk/monitor/perf-tests/monitor-query/tsconfig.json
Original file line number Diff line number Diff line change
@@ -1,10 +1,21 @@
{
"extends": "../../../../tsconfig",
"compilerOptions": {
"module": "CommonJS",
"lib": ["ES6", "ESNext.AsyncIterable"],
"noEmit": true
"module": "NodeNext",
"moduleResolution": "NodeNext",
"rootDir": ".",
"paths": {
"@azure-tests/perf-monitor-query": ["./src/index"]
}
},
"compileOnSave": true,
"include": ["./test/**/*.ts"]
"include": [
"src/**/*.ts",
"src/**/*.mts",
"src/**/*.cts",
"samples-dev/**/*.ts",
"test/**/*.ts",
"test/**/*.mts",
"test/**/*.cts"
]
}

0 comments on commit 708295a

Please sign in to comment.