Upgrade prettier dev-dependency to v2.5.1 in test-utils (Azure#19720)

Upgraded prettier dev-dependency version to latest 2.5.1.
Files were re-formatted as well. There are only format changes in this PR, no manual changes except for package.json files.

Main format changes with Prettier 2.x in this PR include:

Trailing commas by default.
Whitespace added after every function keyword.

Author: JonathanCrd

common/config/rush/pnpm-lock.yaml

@@ -2,7 +2,7 @@
 process.env.CHROME_BIN = require("puppeteer").executablePath();
 require("dotenv").config({ path: "../.env" });
 
-module.exports = function(config) {
+module.exports = function (config) {
   config.set({
     // base path that will be used to resolve all patterns (eg. files, exclude)
     basePath: "./",
@@ -16,7 +16,7 @@ module.exports = function(config) {
     // list of files / patterns to load in the browser
     files: [
       "dist-test/index.browser.js",
-      { pattern: "dist-test/index.browser.js.map", type: "html", included: false, served: true }
+      { pattern: "dist-test/index.browser.js.map", type: "html", included: false, served: true },
     ],
 
     // list of files / patterns to exclude
@@ -25,7 +25,7 @@ module.exports = function(config) {
     // preprocess matching files before serving them to the browser
     // available preprocessors: https://npmjs.org/browse/keyword/karma-preprocessor
     preprocessors: {
-      "**/*.js": ["sourcemap", "env"]
+      "**/*.js": ["sourcemap", "env"],
       // IMPORTANT: COMMENT following line if you want to debug in your browsers!!
       // Preprocess source file to calculate code coverage, however this will make source file unreadable
       //"dist-test/index.browser.js": ["coverage"]
@@ -58,8 +58,8 @@ module.exports = function(config) {
     customLaunchers: {
       ChromeHeadlessNoSandbox: {
         base: "ChromeHeadless",
-        flags: ["--no-sandbox", "--disable-web-security"]
-      }
+        flags: ["--no-sandbox", "--disable-web-security"],
+      },
     },
 
     // Continuous Integration mode
@@ -76,6 +76,6 @@ module.exports = function(config) {
     browserConsoleLogOptions: {
       // We would usually hide the logs from the tests, but we don't need to do this inside of the recorder package because we are not recording the tests.
       // // terminal: process.env.TEST_MODE !== "record"
-    }
+    },
   });
 };

sdk/test-utils/perf/karma.conf.js

@@ -77,7 +77,7 @@
     "karma-chrome-launcher": "^3.0.0",
     "karma-coverage": "^2.0.0",
     "karma-env-preprocessor": "^0.1.1",
-    "prettier": "^1.16.4",
+    "prettier": "^2.5.1",
     "rimraf": "^3.0.0",
     "typescript": "~4.2.0",
     "ts-node": "^8.3.0"

sdk/test-utils/perf/package.json

@@ -5,7 +5,7 @@ import { AbortSignalLike, AbortController } from "@azure/abort-controller";
 import {
   TestProxyHttpClient,
   TestProxyHttpClientV1,
-  testProxyHttpPolicy
+  testProxyHttpPolicy,
 } from "./testProxyHttpClient";
 import { HttpClient } from "@azure/core-http";
 import { Pipeline } from "@azure/core-rest-pipeline";
@@ -15,9 +15,9 @@ import { PerfParallel } from "./parallel";
 /**
  * Enables writing perf tests where the number of operations are dynamic for the method/call being tested.
  */
-export abstract class BatchPerfTest<TOptions = Record<string, unknown>> extends PerfTestBase<
-  TOptions
-> {
+export abstract class BatchPerfTest<
+  TOptions = Record<string, unknown>
+> extends PerfTestBase<TOptions> {
   private readonly testProxy!: string;
   public testProxyHttpClient!: TestProxyHttpClient;
   public testProxyHttpClientV1!: TestProxyHttpClientV1;

sdk/test-utils/perf/src/batchPerfTest.ts

@@ -85,51 +85,51 @@ export interface DefaultPerfOptions {
 export const defaultPerfOptions: PerfOptionDictionary<DefaultPerfOptions> = {
   help: {
     description: "Shows all of the available options",
-    shortName: "h"
+    shortName: "h",
   },
   parallel: {
     description: "How many of the same test to call at the same time",
     shortName: "p",
-    defaultValue: 1
+    defaultValue: 1,
   },
   duration: {
     description: "When to stop calling tests at all",
     shortName: "d",
-    defaultValue: 10
+    defaultValue: 10,
   },
   warmup: {
     description: "Duration of warmup in seconds",
     shortName: "w",
-    defaultValue: 5
+    defaultValue: 5,
   },
   iterations: {
     description: "Times to repeat the whole process, after warmup",
     shortName: "i",
-    defaultValue: 1
+    defaultValue: 1,
   },
   "no-cleanup": {
-    description: "Disables test cleanup"
+    description: "Disables test cleanup",
   },
   "test-proxies": {
     description: "URIs of TestProxy servers (separated by ';')",
-    defaultValue: undefined
+    defaultValue: undefined,
   },
   insecure: {
     description:
       "Applied when test-proxies option is defined, connects with https(insecurely by disabling SSL validation)",
     shortName: "ins",
-    defaultValue: false
+    defaultValue: false,
   },
   "milliseconds-to-log": {
     description: "Log frequency in milliseconds",
     shortName: "mtl",
-    defaultValue: 1000
+    defaultValue: 1000,
   },
   "list-transitive-dependencies": {
     description: "List all dependencies, instead of only direct ones, before test run",
     shortName: "ltd",
-    defaultValue: false
-  }
+    defaultValue: false,
+  },
 };
 
 /**
@@ -173,7 +173,7 @@ export function parsePerfOption<TOptions>(
     result[optionName as keyof TOptions] = {
       ...option,
       longName,
-      value
+      value,
     };
   }
 
@@ -211,7 +211,7 @@ export function validateOptions<TOptions>(options: PerfOptionDictionary<TOptions
 function getBooleanOptionDetails<TOptions>(options: PerfOptionDictionary<TOptions>) {
   const booleanProps: { boolean: string[]; default: { [key: string]: boolean } } = {
     boolean: [],
-    default: {}
+    default: {},
   };
 
   for (const key in options) {

sdk/test-utils/perf/src/options.ts

@@ -8,7 +8,7 @@ import {
   DefaultPerfOptions,
   defaultPerfOptions,
   validateOptions,
-  ParsedPerfOptions
+  ParsedPerfOptions,
 } from "./options";
 import { PerfParallel } from "./parallel";
 import { AbortController } from "@azure/abort-controller";
@@ -40,7 +40,7 @@ export abstract class PerfTestBase<TOptions = Record<string, unknown>> {
     if (this.options) {
       validateOptions({
         ...this.options,
-        ...defaultPerfOptions
+        ...defaultPerfOptions,
       });
     }
 
@@ -54,7 +54,7 @@ export abstract class PerfTestBase<TOptions = Record<string, unknown>> {
     //   ```
     return parsePerfOption({
       ...this.options,
-      ...defaultPerfOptions
+      ...defaultPerfOptions,
     } as PerfOptionDictionary<TOptions & DefaultPerfOptions>);
   }
 

sdk/test-utils/perf/src/perfTestBase.ts

@@ -3,7 +3,7 @@ import {
   RequestPolicy,
   RequestPolicyOptions,
   WebResource,
-  HttpOperationResponse
+  HttpOperationResponse,
 } from "@azure/core-http";
 import * as url from "url";
 

sdk/test-utils/perf/src/policy.ts

@@ -7,7 +7,7 @@ import {
   PerfOptionDictionary,
   parsePerfOption,
   defaultPerfOptions,
-  DefaultPerfOptions
+  DefaultPerfOptions,
 } from "./options";
 import { PerfParallel } from "./parallel";
 import { exec } from "child_process";
@@ -103,19 +103,19 @@ export class PerfProgram {
     const weightedAverage = totalOperations / operationsPerSecond;
     console.log(
       `Completed ${totalOperations.toLocaleString(undefined, {
-        maximumFractionDigits: 0
+        maximumFractionDigits: 0,
       })} ` +
         `operations in a weighted-average of ` +
         `${weightedAverage.toLocaleString(undefined, {
           maximumFractionDigits: 2,
-          minimumFractionDigits: 2
+          minimumFractionDigits: 2,
         })}s ` +
         `(${operationsPerSecond.toLocaleString(undefined, {
-          maximumFractionDigits: 2
+          maximumFractionDigits: 2,
         })} ops/s, ` +
         `${secondsPerOperation.toLocaleString(undefined, {
           maximumFractionDigits: 3,
-          minimumFractionDigits: 3
+          minimumFractionDigits: 3,
         })} s/op)`
     );
   }
@@ -141,8 +141,9 @@ export class PerfProgram {
     // of operations running.
     const millisecondsToLog = Number(this.parsedDefaultOptions["milliseconds-to-log"].value);
     console.log(
-      `\n=== ${title} mode, iteration ${iterationIndex + 1}. Logs every ${millisecondsToLog /
-        1000}s ===`
+      `\n=== ${title} mode, iteration ${iterationIndex + 1}. Logs every ${
+        millisecondsToLog / 1000
+      }s ===`
     );
     console.log(`ElapsedTime\tCurrent\t\tTotal\t\tAverage`);
     let lastCompleted = 0;
@@ -177,7 +178,7 @@ export class PerfProgram {
       this.tests.map((test, i = 0) => {
         parallels[i] = {
           completedOperations: 0,
-          lastMillisecondsElapsed: 0
+          lastMillisecondsElapsed: 0,
         };
         return test.runAll(parallels[i], durationMilliseconds, abortController);
       })