Segregate search documents test folders to enable min-max tests (Azure#12643)

Author: KarishmaGhiya

sdk/search/search-documents/package.json

@@ -19,7 +19,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 5000000 --full-trace dist-esm/test/**/*.spec.js --harmony",
+    "integration-test:node": "nyc mocha -r esm --require source-map-support/register --reporter ../../../common/tools/mocha-multi-reporter.js --timeout 5000000 --full-trace dist-test/index.node.js --harmony",
     "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 search-lintReport.html || exit 0",

sdk/search/search-documents/rollup.base.config.js

@@ -36,7 +36,12 @@ export function nodeConfig(test = false) {
 
   if (test) {
     // Entry points - test files under the `test` folder(common for both browser and node), node specific test files
-    baseConfig.input = ["dist-esm/test/*.spec.js", "dist-esm/test/node/*.spec.js"];
+    baseConfig.input = [
+      "dist-esm/test/public/*.spec.js",
+      "dist-esm/test/internal/*.spec.js",
+      "dist-esm/test/public/node/*.spec.js",
+      "dist-esm/test/internal/node/*.spec.js"
+    ];
     baseConfig.plugins.unshift(multiEntry({ exports: false }));
 
     // different output file
@@ -94,13 +99,19 @@ export function browserConfig(test = false) {
           "@opentelemetry/api": ["CanonicalCode", "SpanKind", "TraceFlags"]
         }
       }),
+      // disable the plugin in the rollup config to cut-out errors
       viz({ filename: "dist-browser/browser-stats.html", sourcemap: false })
     ]
   };
 
   if (test) {
     // Entry points - test files under the `test` folder(common for both browser and node), browser specific test files
-    baseConfig.input = ["dist-esm/test/*.spec.js", "dist-esm/test/browser/*.spec.js"];
+    baseConfig.input = [
+      "dist-esm/test/public/*.spec.js",
+      "dist-esm/test/internal/*.spec.js",
+      "dist-esm/test/public/browser/*.spec.js",
+      "dist-esm/test/internal/browser/*.spec.js"
+    ];
     baseConfig.plugins.unshift(multiEntry({ exports: false }));
     baseConfig.output.file = "dist-test/index.browser.js";
 

…rch/search-documents/test/base64.spec.ts …h-documents/test/internal/base64.spec.tssdk/search/search-documents/test/base64.spec.ts renamed to sdk/search/search-documents/test/internal/base64.spec.ts sdk/search/search-documents/test/base64.spec.ts renamed to sdk/search/search-documents/test/internal/base64.spec.ts

@@ -2,7 +2,7 @@
 // Licensed under the MIT license.
 
 import { assert } from "chai";
-import { encode, decode } from "../src/base64";
+import { encode, decode } from "../../src/base64";
 
 describe("base64", () => {
   it("strings can roundtrip", () => {

…ch-documents/test/geographyPoint.spec.ts …nts/test/internal/geographyPoint.spec.tssdk/search/search-documents/test/geographyPoint.spec.ts renamed to sdk/search/search-documents/test/internal/geographyPoint.spec.ts sdk/search/search-documents/test/geographyPoint.spec.ts renamed to sdk/search/search-documents/test/internal/geographyPoint.spec.ts

@@ -3,7 +3,7 @@
 
 import { assert } from "chai";
 import * as sinon from "sinon";
-import GeographyPoint from "../src/geographyPoint";
+import GeographyPoint from "../../src/geographyPoint";
 
 describe("geographyPoint", () => {
   it("JSON.stringify", () => {

…rch-documents/test/serialization.spec.ts …ents/test/internal/serialization.spec.tssdk/search/search-documents/test/serialization.spec.ts renamed to sdk/search/search-documents/test/internal/serialization.spec.ts sdk/search/search-documents/test/serialization.spec.ts renamed to sdk/search/search-documents/test/internal/serialization.spec.ts

@@ -3,8 +3,8 @@
 
 import { assert } from "chai";
 import * as sinon from "sinon";
-import { serialize, deserialize } from "../src/serialization";
-import GeographyPoint from "../src/geographyPoint";
+import { serialize, deserialize } from "../../src/serialization";
+import GeographyPoint from "../../src/geographyPoint";
 
 describe("serialization.serialize", () => {
   it("nested", () => {

…ch/search-documents/test/tracing.spec.ts …-documents/test/internal/tracing.spec.tssdk/search/search-documents/test/tracing.spec.ts renamed to sdk/search/search-documents/test/internal/tracing.spec.ts sdk/search/search-documents/test/tracing.spec.ts renamed to sdk/search/search-documents/test/internal/tracing.spec.ts

@@ -3,7 +3,7 @@
 
 import { assert } from "chai";
 import * as sinon from "sinon";
-import { createSpan } from "../src/tracing";
+import { createSpan } from "../../src/tracing";
 import { setTracer, TestTracer, TestSpan } from "@azure/core-tracing";
 import { SpanKind, TraceFlags } from "@opentelemetry/api";
 import { OperationOptions } from "@azure/core-http";

…documents/test/node/searchClient.spec.ts …ts/test/public/node/searchClient.spec.tssdk/search/search-documents/test/node/searchClient.spec.ts renamed to sdk/search/search-documents/test/public/node/searchClient.spec.ts sdk/search/search-documents/test/node/searchClient.spec.ts renamed to sdk/search/search-documents/test/public/node/searchClient.spec.ts

@@ -13,7 +13,7 @@ import {
   SearchIndexClient,
   AutocompleteResult,
   IndexDocumentsBatch
-} from "../../src/index";
+} from "../../../src";
 import { Hotel } from "../utils/interfaces";
 import { createIndex, populateIndex, WAIT_TIME, createRandomIndexName } from "../utils/setup";
 import { delay } from "@azure/core-http";

…ents/test/node/searchIndexClient.spec.ts …st/public/node/searchIndexClient.spec.tssdk/search/search-documents/test/node/searchIndexClient.spec.ts renamed to sdk/search/search-documents/test/public/node/searchIndexClient.spec.ts sdk/search/search-documents/test/node/searchIndexClient.spec.ts renamed to sdk/search/search-documents/test/public/node/searchIndexClient.spec.ts

@@ -4,7 +4,7 @@
 /* eslint-disable no-invalid-this */
 import { isPlaybackMode, record, Recorder, isLiveMode } from "@azure/test-utils-recorder";
 import { assert } from "chai";
-import { SearchIndexClient, SynonymMap, SearchIndex } from "../../src/index";
+import { SearchIndexClient, SynonymMap, SearchIndex } from "../../../src";
 import { Hotel } from "../utils/interfaces";
 import { createClients, environmentSetup } from "../utils/recordedClient";
 import {

…ts/test/node/searchIndexerClient.spec.ts …/public/node/searchIndexerClient.spec.tssdk/search/search-documents/test/node/searchIndexerClient.spec.ts renamed to sdk/search/search-documents/test/public/node/searchIndexerClient.spec.ts sdk/search/search-documents/test/node/searchIndexerClient.spec.ts renamed to sdk/search/search-documents/test/public/node/searchIndexerClient.spec.ts

@@ -10,7 +10,7 @@ import {
   SearchIndexerDataSourceConnection,
   SearchIndexerSkillset,
   SearchIndexer
-} from "../../src/index";
+} from "../../../src";
 import { Hotel } from "../utils/interfaces";
 import { createClients, environmentSetup } from "../utils/recordedClient";
 import {

…arch/search-documents/test/odata.spec.ts …arch-documents/test/public/odata.spec.tssdk/search/search-documents/test/odata.spec.ts renamed to sdk/search/search-documents/test/public/odata.spec.ts sdk/search/search-documents/test/odata.spec.ts renamed to sdk/search/search-documents/test/public/odata.spec.ts

@@ -3,7 +3,7 @@
 
 import { assert } from "chai";
 import * as sinon from "sinon";
-import { odata } from "../src/odata";
+import { odata } from "../../src";
 
 describe("odata", () => {
   it("simple string isn't changed", () => {