Split Tests into Unit / Perf and update all active tests to use commo…

…n test project (#1606)
microsoft · Jul 9, 2021 · fd67558 · fd67558
1 parent 16a4284
commit fd67558
Show file tree

Hide file tree

Showing 118 changed files with 525 additions and 75,276 deletions.
diff --git a/...up-es3/Tests/External/magic-string.umd.js → common/Tests/External/magic-string.umd.js b/...up-es3/Tests/External/magic-string.umd.js → common/Tests/External/magic-string.umd.js
diff --git a/common/Tests/Framework/package.json b/common/Tests/Framework/package.json
@@ -49,7 +49,6 @@
         "magic-string": "^0.25.7"
     },
     "dependencies": {
-        "@microsoft/applicationinsights-shims" : "2.0.0",
         "@microsoft/dynamicproto-js": "^1.1.4"
     }
 }
diff --git a/common/Tests/Framework/tsconfig.json b/common/Tests/Framework/tsconfig.json
@@ -8,7 +8,7 @@
       "target": "es5",
       "forceConsistentCasingInFileNames": true,
       "importHelpers": true,
-      "noEmitHelpers": true,
+      "noEmitHelpers": false,
       "skipLibCheck": true,
       "alwaysStrict": true,
       "declaration": true,

diff --git a/common/config/rush/command-line.json b/common/config/rush/command-line.json
@@ -11,6 +11,16 @@
             "ignoreMissingScript": false,
             "allowWarningsInSuccessfulBuild": true
         },
+        {
+            "commandKind": "bulk",
+            "name": "perftest",
+            "summary": "Run all performance based tests for all packages",
+            "description": "Runs performance tests for all projects",
+            "safeForSimultaneousRushProcesses": false,
+            "enableParallelism": false,
+            "ignoreMissingScript": true,
+            "allowWarningsInSuccessfulBuild": true
+        },
         {
             "commandKind": "bulk",
             "name": "lint",

diff --git a/common/config/rush/npm-shrinkwrap.json b/common/config/rush/npm-shrinkwrap.json
diff --git a/extensions/applicationinsights-analytics-js/Tests/Selenium/appinsights-analytics.tests.ts b/extensions/applicationinsights-analytics-js/Tests/Selenium/appinsights-analytics.tests.ts
diff --git a/...-js/Tests/AnalyticsExtensionSize.tests.ts → .../Unit/src/AnalyticsExtensionSize.tests.ts b/...-js/Tests/AnalyticsExtensionSize.tests.ts → .../Unit/src/AnalyticsExtensionSize.tests.ts
@@ -3,8 +3,8 @@ import * as pako from "pako";
 
 export class AnalyticsExtensionSizeCheck extends AITestClass {
     private readonly MAX_DEFLATE_SIZE = 20;
-    private readonly rawFilePath = "../../dist/applicationinsights-analytics-js.min.js";
-    private readonly prodFilePaath = "../../browser/applicationinsights-analytics-js.min.js"
+    private readonly rawFilePath = "../dist/applicationinsights-analytics-js.min.js";
+    private readonly prodFilePaath = "../browser/applicationinsights-analytics-js.min.js"
 
     public testInitialize() {
     }
@@ -31,7 +31,7 @@ export class AnalyticsExtensionSizeCheck extends AITestClass {
     private _fileSizeCheck(isProd: boolean): void {
         let _filePath = isProd? this.prodFilePaath : this.rawFilePath;
         let postfix = isProd? "" : "-raw";
-        let fileName = _filePath.split("..")[2];
+        let fileName = _filePath.split("..")[1];
         this.testCase({
             name: `Test applicationinsights-analytics-extension${postfix} deflate size`,
             test: () => {

diff --git a/...ics-js/Tests/ApplicationInsights.tests.ts → ...sts/Unit/src/ApplicationInsights.tests.ts b/...ics-js/Tests/ApplicationInsights.tests.ts → ...sts/Unit/src/ApplicationInsights.tests.ts
@@ -10,7 +10,7 @@ import {
 import { ITelemetryItem, AppInsightsCore, IPlugin, IConfiguration, IAppInsightsCore, setEnableEnvMocks, getLocation, dumpObj } from "@microsoft/applicationinsights-core-js";
 import { Sender } from "@microsoft/applicationinsights-channel-js"
 import { PropertiesPlugin } from "@microsoft/applicationinsights-properties-js";
-import { ApplicationInsights } from "../src/JavaScriptSDK/ApplicationInsights";
+import { ApplicationInsights } from "../../../src/JavaScriptSDK/ApplicationInsights";
 
 declare class ExceptionHelper {
     capture: (appInsights:IAppInsights) => void;

diff --git a/...cs-js/Tests/TelemetryItemCreator.tests.ts → ...ts/Unit/src/TelemetryItemCreator.tests.ts b/...cs-js/Tests/TelemetryItemCreator.tests.ts → ...ts/Unit/src/TelemetryItemCreator.tests.ts
@@ -5,7 +5,7 @@ import {
     TelemetryItemCreator,
     IPageViewTelemetry
 } from '@microsoft/applicationinsights-common';
-import { ApplicationInsights } from '../src/JavaScriptSDK/ApplicationInsights'
+import { ApplicationInsights } from '../../../src/JavaScriptSDK/ApplicationInsights'
 import { 
     IAppInsightsCore, AppInsightsCore,
     ITelemetryItem,

diff --git a/extensions/applicationinsights-analytics-js/Tests/Unit/src/appinsights-analytics.tests.ts b/extensions/applicationinsights-analytics-js/Tests/Unit/src/appinsights-analytics.tests.ts
@@ -0,0 +1,9 @@
+import { ApplicationInsightsTests } from './ApplicationInsights.tests';
+import { TelemetryItemCreatorTests } from './TelemetryItemCreator.tests';
+import { AnalyticsExtensionSizeCheck } from "./AnalyticsExtensionSize.tests";
+
+export function runTests() {
+    new ApplicationInsightsTests().registerTests();
+    new TelemetryItemCreatorTests().registerTests();
+    new AnalyticsExtensionSizeCheck().registerTests();
+}
diff --git a/...ts-analytics-js/Tests/Selenium/Tests.html → ...nsights-analytics-js/Tests/UnitTests.html b/...ts-analytics-js/Tests/Selenium/Tests.html → ...nsights-analytics-js/Tests/UnitTests.html
@@ -5,18 +5,18 @@
     <meta charset="utf-8">
     <meta http-equiv="Cache-control" content="no-Cache" />
     <title>Tests for Application Insights JavaScript Analytics Extension</title>
-    <link rel="stylesheet" href="../../../../common//Tests//External/qunit-1.23.1.css">
-    <script src="../../../../common/Tests/Selenium/ExceptionHelper.js"></script>
+    <link rel="stylesheet" href="../../../common/Tests//External/qunit-1.23.1.css">
+    <script src="../../../common/Tests/Selenium/ExceptionHelper.js"></script>
     <!-- <script src="http://sinonjs.org/releases/sinon-2.3.8.js" crossorigin="anonymous"></script> -->
-    <script src="../../../../common/Tests/External/sinon-7.3.1.js"></script> -->
+    <script src="../../../common/Tests/External/sinon-7.3.1.js"></script> -->
     <!-- <script src="http://cdnjs.cloudflare.com/ajax/libs/require.js/2.2.0/require.js" crossorigin="anonymous"></script> -->
-    <script src="../../../../common/Tests/External/require-2.2.0.js"></script>
-    <script src="../../../../common/Tests/Selenium/ModuleLoader.js"></script>
-    <script src="../../../../common/Tests/Selenium/SimpleSyncPromise.js"></script>
+    <script src="../../../common/Tests/External/require-2.2.0.js"></script>
+    <script src="../../../common/Tests/Selenium/ModuleLoader.js"></script>
+    <script src="../../../common/Tests/Selenium/SimpleSyncPromise.js"></script>
 
     <script>
         var modules = new ModuleLoader({
-            baseUrl: '../../',
+            baseUrl: '../',
             paths: {
                 qunit: "../../common/Tests/External/qunit-1.23.1",
                 "whatwg-fetch": "../../common/Tests/External/whatwg-fetch.3.0.0"
@@ -50,7 +50,7 @@
         // Load Properties
         modules.add("@microsoft/applicationinsights-properties-js", "./node_modules/@microsoft/applicationinsights-properties-js/browser/applicationinsights-properties-js");
 
-        var testModule = modules.add("Tests/Selenium/appinsights-analytics.tests", "./appinsights-analytics.tests.js")
+        var testModule = modules.add("Tests/Unit/src/appinsights-analytics.tests", "./Unit/dist/appinsights-analytics.tests.js")
         testModule.run = function (tests) {
             console && console.log("Starting tests");
             QUnit.start();