[App Config] Separate app config tests into public and internal to enable min-max tests

sdk/appconfiguration/app-configuration/test/helpers.spec.ts → sdk/appconfiguration/app-configuration/test/internal/helpers.spec.ts

@@ -10,9 +10,9 @@ import {
   transformKeyValue,
   transformKeyValueResponseWithStatusCode,
   transformKeyValueResponse
-} from "../src/internal/helpers";
+} from "../../src/internal/helpers";
 import * as assert from "assert";
-import { ConfigurationSetting, HttpResponseField, HttpResponseFields } from "../src";
+import { ConfigurationSetting, HttpResponseField, HttpResponseFields } from "../../src";
 import { HttpHeaders } from "@azure/core-http";
 
 describe("helper methods", () => {

sdk/appconfiguration/app-configuration/test/internal/http.spec.ts

@@ -5,12 +5,17 @@ import { parseSyncToken, SyncTokens } from "../../src/internal/synctokenpolicy";
 import * as assert from "assert";
 import { AppConfigurationClient } from "../../src";
 import nock from "nock";
-import { getUserAgentPrefix, packageVersion } from "../../src/appConfigurationClient";
+import {
+  getUserAgentPrefix,
+  InternalAppConfigurationClientOptions,
+  packageVersion
+} from "../../src/appConfigurationClient";
 import {
   createAppConfigurationClientForTests,
   assertThrowsRestError,
   startRecorder
-} from "../testHelpers";
+} from "../utils/testHelpers";
+//import InternalAppConfigurationClientOptions
 import * as chai from "chai";
 import { Recorder } from "@azure/test-utils-recorder";
 
@@ -103,7 +108,9 @@ describe("http request related tests", function() {
 
     beforeEach(function() {
       recorder = startRecorder(this);
-      client = createAppConfigurationClientForTests() || this.skip();
+      client =
+        createAppConfigurationClientForTests<InternalAppConfigurationClientOptions>({}) ||
+        this.skip();
     });
 
     afterEach(async function() {
@@ -146,7 +153,7 @@ describe("http request related tests", function() {
       syncTokens = new SyncTokens();
 
       client =
-        createAppConfigurationClientForTests({
+        createAppConfigurationClientForTests<InternalAppConfigurationClientOptions>({
           syncTokens: syncTokens
         }) || this.skip();
 

sdk/appconfiguration/app-configuration/test/package.spec.ts → sdk/appconfiguration/app-configuration/test/internal/package.spec.ts

@@ -2,7 +2,7 @@ import { join } from "path";
 import * as assert from "assert";
 import { isNode } from "@azure/core-http";
 
-import { packageVersion } from "../src/appConfigurationClient";
+import { packageVersion } from "../../src/appConfigurationClient";
 
 describe("packagejson related tests", () => {
   // if this test is failing you need to update the contant `packageVersion` referenced above

sdk/appconfiguration/app-configuration/test/tracingHelpers.spec.ts → sdk/appconfiguration/app-configuration/test/internal/tracingHelpers.spec.ts

@@ -1,7 +1,7 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT License.
 
-import { Spanner } from "../src/internal/tracingHelpers";
+import { Spanner } from "../../src/internal/tracingHelpers";
 import { RestError } from "@azure/core-http";
 import { getTracer } from "@azure/core-tracing";
 import { SpanKind, CanonicalCode } from "@opentelemetry/api";

sdk/appconfiguration/app-configuration/test/policies/throttlingRetryPolicyTests.spec.ts

@@ -12,7 +12,7 @@ import {
   RestError
 } from "@azure/core-http";
 import { ThrottlingRetryPolicy, getDelayInMs } from "../../src/policies/throttlingRetryPolicy";
-import { assertThrowsRestError } from "../testHelpers";
+import { assertThrowsRestError } from "../utils/testHelpers";
 
 describe("ThrottlingRetryPolicy", () => {
   class PassThroughPolicy {

sdk/appconfiguration/app-configuration/test/auth.spec.ts → sdk/appconfiguration/app-configuration/test/public/auth.spec.ts

@@ -1,8 +1,12 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 
-import { AppConfigurationClient } from "../src";
-import { startRecorder, getTokenAuthenticationCredential, CredsAndEndpoint } from "./testHelpers";
+import { AppConfigurationClient } from "../../src";
+import {
+  startRecorder,
+  getTokenAuthenticationCredential,
+  CredsAndEndpoint
+} from "../utils/testHelpers";
 import * as assert from "assert";
 import { Recorder, isPlaybackMode } from "@azure/test-utils-recorder";
 

sdk/appconfiguration/app-configuration/test/etags.spec.ts → sdk/appconfiguration/app-configuration/test/public/etags.spec.ts

@@ -1,13 +1,13 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 
-import { AppConfigurationClient } from "../src";
+import { AppConfigurationClient } from "../../src";
 import {
   startRecorder,
   createAppConfigurationClientForTests,
   deleteKeyCompletely,
   assertThrowsRestError
-} from "./testHelpers";
+} from "../utils/testHelpers";
 import * as assert from "assert";
 import { Recorder } from "@azure/test-utils-recorder";
 

sdk/appconfiguration/app-configuration/test/index.readonlytests.spec.ts → sdk/appconfiguration/app-configuration/test/public/index.readonlytests.spec.ts

@@ -4,8 +4,8 @@ import {
   deleteKeyCompletely,
   assertThrowsAbortError,
   startRecorder
-} from "./testHelpers";
-import { AppConfigurationClient } from "../src";
+} from "../utils/testHelpers";
+import { AppConfigurationClient } from "../../src";
 import * as assert from "assert";
 import { Recorder } from "@azure/test-utils-recorder";
 

sdk/appconfiguration/app-configuration/test/index.spec.ts → sdk/appconfiguration/app-configuration/test/public/index.spec.ts

@@ -10,8 +10,8 @@ import {
   assertThrowsRestError,
   assertThrowsAbortError,
   startRecorder
-} from "./testHelpers";
-import { AppConfigurationClient, ConfigurationSetting } from "../src";
+} from "../utils/testHelpers";
+import { AppConfigurationClient, ConfigurationSetting } from "../../src";
 import { delay } from "@azure/core-http";
 import { Recorder } from "@azure/test-utils-recorder";
 

sdk/appconfiguration/app-configuration/test/throwOrNotThrow.spec.ts → sdk/appconfiguration/app-configuration/test/public/throwOrNotThrow.spec.ts

@@ -1,10 +1,10 @@
-import { AppConfigurationClient, ConfigurationSetting } from "../src";
+import { AppConfigurationClient, ConfigurationSetting } from "../../src";
 import {
   createAppConfigurationClientForTests,
   deleteKeyCompletely,
   assertThrowsRestError,
   startRecorder
-} from "./testHelpers";
+} from "../utils/testHelpers";
 import * as assert from "assert";
 import { Recorder } from "@azure/test-utils-recorder";
 

sdk/appconfiguration/app-configuration/test/testHelpers.ts → sdk/appconfiguration/app-configuration/test/utils/testHelpers.ts

@@ -1,9 +1,9 @@
 // Copyright (c) Microsoft Corporation.
 // Licensed under the MIT license.
 
-import { AppConfigurationClient } from "../src";
+import { AppConfigurationClient, AppConfigurationClientOptions } from "../../src";
 import { PagedAsyncIterableIterator } from "@azure/core-paging";
-import { ConfigurationSetting, ListConfigurationSettingPage, ListRevisionsPage } from "../src";
+import { ConfigurationSetting, ListConfigurationSettingPage, ListRevisionsPage } from "../../src";
 import { env, isPlaybackMode, RecorderEnvironmentSetup, record } from "@azure/test-utils-recorder";
 import * as assert from "assert";
 
@@ -12,7 +12,6 @@ import * as assert from "assert";
 import * as dotenv from "dotenv";
 import { RestError } from "@azure/core-http";
 import { DefaultAzureCredential, TokenCredential } from "@azure/identity";
-import { InternalAppConfigurationClientOptions } from "../src/appConfigurationClient";
 dotenv.config();
 
 let connectionStringNotPresentWarning = false;
@@ -70,8 +69,8 @@ export function getTokenAuthenticationCredential(): CredsAndEndpoint | undefined
   };
 }
 
-export function createAppConfigurationClientForTests(
-  options?: InternalAppConfigurationClientOptions
+export function createAppConfigurationClientForTests<Options extends AppConfigurationClientOptions>(
+  options?: Options
 ): AppConfigurationClient | undefined {
   const connectionString = env["APPCONFIG_CONNECTION_STRING"];
 

sdk/appconfiguration/app-configuration/tests.yml

@@ -7,6 +7,7 @@ extends:
     ResourceServiceDirectory: appconfiguration
     TestBrowser: true
     TestSamples: false
+    TestMinMax: true
     EnvVars:
       AZURE_CLIENT_ID: $(aad-azure-sdk-test-client-id)
       AZURE_TENANT_ID: $(aad-azure-sdk-test-tenant-id)