From dd93b70a6158e67772ed0f607dee3f35d82656a4 Mon Sep 17 00:00:00 2001 From: Asier Isayas Date: Wed, 21 Jan 2026 06:51:26 -0800 Subject: [PATCH] dont delete database --- test/sql/scaleAndSettings/changePartitionKey.spec.ts | 9 ++++++--- test/sql/scaleAndSettings/computedProperties.spec.ts | 9 ++++++--- test/sql/scaleAndSettings/scale.spec.ts | 4 +++- test/sql/scaleAndSettings/settings.spec.ts | 9 ++++++--- test/sql/scripts/storedProcedure.spec.ts | 11 +++++++---- test/sql/scripts/trigger.spec.ts | 11 +++++++---- test/sql/scripts/userDefinedFunction.spec.ts | 11 +++++++---- test/testData.ts | 1 - 8 files changed, 42 insertions(+), 23 deletions(-) diff --git a/test/sql/scaleAndSettings/changePartitionKey.spec.ts b/test/sql/scaleAndSettings/changePartitionKey.spec.ts index e8c9b1aab..02858766c 100644 --- a/test/sql/scaleAndSettings/changePartitionKey.spec.ts +++ b/test/sql/scaleAndSettings/changePartitionKey.spec.ts @@ -25,9 +25,12 @@ test.describe("Change Partition Key", () => { await PartitionKeyTab.click(); }); - test.afterEach("Delete Test Database", async () => { - await context?.dispose(); - }); + // Delete database only if not running in CI + if (!process.env.CI) { + test.afterEach("Delete Test Database", async () => { + await context?.dispose(); + }); + } test("Change partition key path", async ({ page }) => { await expect(explorer.frame.getByText("/partitionKey")).toBeVisible(); diff --git a/test/sql/scaleAndSettings/computedProperties.spec.ts b/test/sql/scaleAndSettings/computedProperties.spec.ts index 2ee1576fd..ec61ad29d 100644 --- a/test/sql/scaleAndSettings/computedProperties.spec.ts +++ b/test/sql/scaleAndSettings/computedProperties.spec.ts @@ -24,9 +24,12 @@ test.describe("Computed Properties", () => { await computedPropertiesTab.click(); }); - test.afterAll("Delete Test Database", async () => { - await context?.dispose(); - }); + // Delete database only if not running in CI + if (!process.env.CI) { + test.afterEach("Delete Test Database", async () => { + await context?.dispose(); + }); + } test("Add valid computed property", async ({ page }) => { await clearComputedPropertiesTextBoxContent({ page }); diff --git a/test/sql/scaleAndSettings/scale.spec.ts b/test/sql/scaleAndSettings/scale.spec.ts index d886b2def..d12db999c 100644 --- a/test/sql/scaleAndSettings/scale.spec.ts +++ b/test/sql/scaleAndSettings/scale.spec.ts @@ -118,5 +118,7 @@ async function openScaleTab(browser: Browser): Promise { } async function cleanup({ context }: Partial) { - await context?.dispose(); + if (!process.env.CI) { + await context?.dispose(); + } } diff --git a/test/sql/scaleAndSettings/settings.spec.ts b/test/sql/scaleAndSettings/settings.spec.ts index f60889574..18cd32efb 100644 --- a/test/sql/scaleAndSettings/settings.spec.ts +++ b/test/sql/scaleAndSettings/settings.spec.ts @@ -17,9 +17,12 @@ test.describe("Settings under Scale & Settings", () => { await settingsTab.click(); }); - test.afterAll("Delete Test Database", async () => { - await context?.dispose(); - }); + // Delete database only if not running in CI + if (!process.env.CI) { + test.afterEach("Delete Test Database", async () => { + await context?.dispose(); + }); + } test("Update TTL to On (no default)", async () => { const ttlOnNoDefaultRadioButton = explorer.frame.getByRole("radio", { name: "ttl-on-no-default-option" }); diff --git a/test/sql/scripts/storedProcedure.spec.ts b/test/sql/scripts/storedProcedure.spec.ts index c4084423a..eb8bef4e8 100644 --- a/test/sql/scripts/storedProcedure.spec.ts +++ b/test/sql/scripts/storedProcedure.spec.ts @@ -13,12 +13,15 @@ test.describe("Stored Procedures", () => { }); test.beforeEach("Open container", async ({ page }) => { - explorer = await DataExplorer.open(page, TestAccount.SQL); + explorer = await DataExplorer.open(page, TestAccount.SQL2); }); - test.afterAll("Delete Test Database", async () => { - await context?.dispose(); - }); + // Delete database only if not running in CI + if (!process.env.CI) { + test.afterEach("Delete Test Database", async () => { + await context?.dispose(); + }); + } test("Add, execute, and delete stored procedure", async ({ page }, testInfo) => { void page; diff --git a/test/sql/scripts/trigger.spec.ts b/test/sql/scripts/trigger.spec.ts index f233f54ea..f1aa39975 100644 --- a/test/sql/scripts/trigger.spec.ts +++ b/test/sql/scripts/trigger.spec.ts @@ -25,12 +25,15 @@ test.describe("Triggers", () => { }); test.beforeEach("Open container", async ({ page }) => { - explorer = await DataExplorer.open(page, TestAccount.SQL); + explorer = await DataExplorer.open(page, TestAccount.SQL2); }); - test.afterAll("Delete Test Database", async () => { - await context?.dispose(); - }); + // Delete database only if not running in CI + if (!process.env.CI) { + test.afterEach("Delete Test Database", async () => { + await context?.dispose(); + }); + } test("Add and delete trigger", async ({ page }, testInfo) => { // Open container context menu and click New Trigger diff --git a/test/sql/scripts/userDefinedFunction.spec.ts b/test/sql/scripts/userDefinedFunction.spec.ts index 4dc4c3c7c..5f5f0fe93 100644 --- a/test/sql/scripts/userDefinedFunction.spec.ts +++ b/test/sql/scripts/userDefinedFunction.spec.ts @@ -18,12 +18,15 @@ test.describe("User Defined Functions", () => { }); test.beforeEach("Open container", async ({ page }) => { - explorer = await DataExplorer.open(page, TestAccount.SQL); + explorer = await DataExplorer.open(page, TestAccount.SQL2); }); - test.afterAll("Delete Test Database", async () => { - await context?.dispose(); - }); + // Delete database only if not running in CI + if (!process.env.CI) { + test.afterEach("Delete Test Database", async () => { + await context?.dispose(); + }); + } test("Add, execute, and delete user defined function", async ({ page }, testInfo) => { // Open container context menu and click New UDF diff --git a/test/testData.ts b/test/testData.ts index 47165786b..fff851992 100644 --- a/test/testData.ts +++ b/test/testData.ts @@ -159,7 +159,6 @@ export async function createTestSQLContainer({ const adaptedCredentials = new AzureIdentityCredentialAdapter(credentials); const armClient = new CosmosDBManagementClient(adaptedCredentials, subscriptionId); const accountName = getAccountName(testAccount); - console.log(accountName); const account = await armClient.databaseAccounts.get(resourceGroupName, accountName); const clientOptions: CosmosClientOptions = {