[Event hubs] Separate event hubs tests into public and internal and enable min-max tests (Azure#12798)

Author: KarishmaGhiya

eng/tools/dependency-testing/index.js

@@ -200,8 +200,8 @@ async function readAndReplaceSourceReferences(filePath, packageName) {
   var fileContent = await packageUtils.readFile(filePath);
   console.log("Reading filePath = " + filePath);
   testAssetsContent = fileContent.replace('path.resolve(path.join(process.cwd(), "test-assets"','path.resolve(path.join(process.cwd(),"..","..", "test-assets"');
-  // Regex for internal references = /* ["']+[../]*src[/a-z]+["'] */
-  var internalrefs = testAssetsContent.match(/[\"\']+[..//]*src[//a-zA-Z]+[\"\']+/g);
+  // Regex for internal references = /* ["']+[../]*src[/][a-z]+["'] */
+  var internalrefs = testAssetsContent.match(/[\"\']+[..//]*src[//][a-zA-Z]+[\"\']+/g);
   var writeContent = "";
   if (internalrefs) {
     console.log("internal refs = ");
@@ -211,7 +211,7 @@ async function readAndReplaceSourceReferences(filePath, packageName) {
   else {
     var replaceText = "\"" + packageName + "\"";
     //Regex for public api references to be replaced by package name
-    writeContent = testAssetsContent.replace(/[\"\']+[..//]*src[\"\']+/g, replaceText);
+    writeContent = testAssetsContent.replace(/[\"\']+[..//]*src[//]*[\"\']+/g, replaceText);
   }
   await packageUtils.writeFile(filePath, writeContent);
 }

sdk/eventhub/event-hubs/package.json

@@ -58,7 +58,7 @@
     "extract-api": "tsc -p . && api-extractor run --local",
     "format": "prettier --write --config ../../../.prettierrc.json --ignore-path ../../../.prettierignore \"src/**/*.ts\" \"test/**/*.ts\" \"*.{js,json}\"",
     "integration-test:browser": "karma start --single-run",
-    "integration-test:node": "nyc mocha -r esm --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 1200000 --full-trace dist-esm/test/*.spec.js dist-esm/test/**/*.spec.js",
+    "integration-test:node": "nyc mocha -r esm --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 1200000 --full-trace dist-esm/test/internal/*.spec.js dist-esm/test/public/*.spec.js dist-esm/test/public/**/*.spec.js dist-esm/test/internal/**/*.spec.js",
     "integration-test": "npm run integration-test:node && npm run integration-test:browser",
     "lint:fix": "eslint package.json api-extractor.json src test --ext .ts --fix --fix-type [problem,suggestion]",
     "lint": "eslint package.json api-extractor.json src test --ext .ts -f html -o event-hubs-lintReport.html || exit 0",

sdk/eventhub/event-hubs/rollup.base.config.js

@@ -43,7 +43,7 @@ export function nodeConfig(test = false) {
 
   if (test) {
     // entry point is every test file
-    baseConfig.input = "dist-esm/test/**/*.spec.js";
+    baseConfig.input = ["dist-esm/test/internal/**/*.spec.js", "dist-esm/test/public/**/*.spec.js"];
     baseConfig.plugins.unshift(multiEntry({ exports: false }));
 
     // different output file
@@ -143,7 +143,11 @@ export function browserConfig(test = false) {
   };
 
   if (test) {
-    baseConfig.input = ["dist-esm/test/*.spec.js", "dist-esm/test/impl/*.spec.js"];
+    baseConfig.input = [
+      "dist-esm/test/public/*.spec.js",
+      "dist-esm/test/internal/*.spec.js",
+      "dist-esm/test/internal/impl/*.spec.js"
+    ];
     baseConfig.plugins.unshift(multiEntry({ exports: false }));
     baseConfig.output.file = "test-browser/index.js";
 

sdk/eventhub/event-hubs/test/auth.spec.ts …ub/event-hubs/test/internal/auth.spec.tssdk/eventhub/event-hubs/test/auth.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/auth.spec.ts sdk/eventhub/event-hubs/test/auth.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/auth.spec.ts

@@ -2,11 +2,10 @@
 // Licensed under the MIT license.
 
 import { parseConnectionString } from "@azure/core-amqp";
-import { EventHubConsumerClient } from "../src/eventHubConsumerClient";
-import { EnvVarKeys, getEnvVars } from "./utils/testUtils";
+import { EventHubConsumerClient, EventHubProducerClient } from "../../src";
+import { EnvVarKeys, getEnvVars } from "../public/utils/testUtils";
 import chai from "chai";
-import { EventHubProducerClient } from "../src";
-import { SharedKeyCredential } from "../src/eventhubSharedKeyCredential";
+import { SharedKeyCredential } from "../../src/eventhubSharedKeyCredential";
 
 const should = chai.should();
 const env = getEnvVars();

…/eventhub/event-hubs/test/client.spec.ts …/event-hubs/test/internal/client.spec.tssdk/eventhub/event-hubs/test/client.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/client.spec.ts sdk/eventhub/event-hubs/test/client.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/client.spec.ts

@@ -14,12 +14,12 @@ import {
   EventHubProducerClient,
   EventHubConsumerClient,
   Subscription
-} from "../src";
-import { packageJsonInfo } from "../src/util/constants";
-import { EnvVarKeys, getEnvVars, isNode } from "./utils/testUtils";
+} from "../../src";
+import { packageJsonInfo } from "../../src/util/constants";
+import { EnvVarKeys, getEnvVars, isNode } from "../public/utils/testUtils";
 import { MessagingError } from "@azure/core-amqp";
-import { ConnectionContext } from "../src/connectionContext";
-import { getRuntimeInfo } from "../src/util/runtimeInfo";
+import { ConnectionContext } from "../../src/connectionContext";
+import { getRuntimeInfo } from "../../src/util/runtimeInfo";
 const env = getEnvVars();
 
 const testFailureMessage = "Test failure";

…/eventhub/event-hubs/test/config.spec.ts …/event-hubs/test/internal/config.spec.tssdk/eventhub/event-hubs/test/config.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/config.spec.ts sdk/eventhub/event-hubs/test/config.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/config.spec.ts

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 
-import { EventHubConnectionConfig } from "../src/eventhubConnectionConfig";
+import { EventHubConnectionConfig } from "../../src/eventhubConnectionConfig";
 import chai from "chai";
 const should = chai.should();
 

…/event-hubs/test/dataTransformer.spec.ts …bs/test/internal/dataTransformer.spec.tssdk/eventhub/event-hubs/test/dataTransformer.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/dataTransformer.spec.ts sdk/eventhub/event-hubs/test/dataTransformer.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/dataTransformer.spec.ts

@@ -6,7 +6,7 @@ import * as chai from "chai";
 const should = chai.should();
 import * as assert from "assert";
 import isBuffer from "is-buffer";
-import { defaultDataTransformer } from "../src/dataTransformer";
+import { defaultDataTransformer } from "../../src/dataTransformer";
 
 describe("DataTransformer", function() {
   const objectBody: any = {

…ubs/test/diagnostics/messageSpan.spec.ts …internal/diagnostics/messageSpan.spec.tssdk/eventhub/event-hubs/test/diagnostics/messageSpan.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/diagnostics/messageSpan.spec.ts sdk/eventhub/event-hubs/test/diagnostics/messageSpan.spec.ts renamed to sdk/eventhub/event-hubs/test/internal/diagnostics/messageSpan.spec.ts

@@ -2,10 +2,10 @@
 // Licensed under the MIT license.
 
 import chai from "chai";
-import { createMessageSpan } from "../../src/diagnostics/messageSpan";
+import { createMessageSpan } from "../../../src/diagnostics/messageSpan";
 import { TraceFlags, SpanContext } from "@opentelemetry/api";
 import { TestTracer, setTracer, getTracer } from "@azure/core-tracing";
-import { EventHubConnectionConfig } from "../../src/eventhubConnectionConfig";
+import { EventHubConnectionConfig } from "../../../src/eventhubConnectionConfig";
 
 const should = chai.should();
 const assert = chai.assert;