diff --git a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ApprovedColor.spec.ts b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ApprovedColor.spec.ts index e97f1b8e9a..54dc602a32 100644 --- a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ApprovedColor.spec.ts +++ b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ApprovedColor.spec.ts @@ -4,7 +4,7 @@ import {expect} from "@playwright/test"; const dataTypeName = 'Approved Color'; test.describe(`${dataTypeName} tests`, () => { let dataTypeDefaultData = null; - let dataTypeData = null; + let dataTypeData = null; const colorValue = '#ffffff'; const colorLabel = 'TestColor'; @@ -16,11 +16,11 @@ test.describe(`${dataTypeName} tests`, () => { test.afterEach(async ({umbracoApi}) => { if (dataTypeDefaultData !== null) { - await umbracoApi.dataType.update(dataTypeDefaultData.id, dataTypeDefaultData); - } + await umbracoApi.dataType.update(dataTypeDefaultData.id, dataTypeDefaultData); + } }); - test('can include label', async ({umbracoApi, umbracoUi}) => { + test('can include label', async ({umbracoApi, umbracoUi}) => { // Arrange const expectedDataTypeValues = [ { @@ -31,7 +31,7 @@ test.describe(`${dataTypeName} tests`, () => { // Remove all existing values dataTypeData = await umbracoApi.dataType.getByName(dataTypeName); dataTypeData.values = []; - await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); + await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); await umbracoUi.dataType.goToDataType(dataTypeName); // Act @@ -43,7 +43,8 @@ test.describe(`${dataTypeName} tests`, () => { expect(dataTypeData.values).toEqual(expectedDataTypeValues); }); - test('can add color', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can add color', async ({umbracoApi, umbracoUi}) => { // Arrange const expectedDataTypeValues = [ { @@ -59,7 +60,7 @@ test.describe(`${dataTypeName} tests`, () => { // Remove all existing values dataTypeData = await umbracoApi.dataType.getByName(dataTypeName); dataTypeData.values = []; - await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); + await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); await umbracoUi.dataType.goToDataType(dataTypeName); // Act @@ -72,7 +73,7 @@ test.describe(`${dataTypeName} tests`, () => { }); // TODO: remove .skip when the frontend is able to display the added color. Currently the added colors are not displayed after reloading page - test.skip('can remove color', async ({umbracoApi, umbracoUi}) => { + test.skip('can remove color', async ({umbracoApi, umbracoUi}) => { // Arrange const removedDataTypeValues = [ { @@ -88,7 +89,7 @@ test.describe(`${dataTypeName} tests`, () => { // Remove all existing values and add a color to remove dataTypeData = await umbracoApi.dataType.getByName(dataTypeName); dataTypeData.values = removedDataTypeValues; - await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); + await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); await umbracoUi.dataType.goToDataType(dataTypeName); // Act diff --git a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/DataTypeFolder.spec.ts b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/DataTypeFolder.spec.ts index 6c31a0e699..7e4c2ed590 100644 --- a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/DataTypeFolder.spec.ts +++ b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/DataTypeFolder.spec.ts @@ -34,7 +34,7 @@ test.describe('Data Types Folder tests', () => { await umbracoApi.dataType.ensureNameNotExists(wrongDataTypeFolderName); await umbracoApi.dataType.createFolder(wrongDataTypeFolderName); expect(await umbracoApi.dataType.doesNameExist(wrongDataTypeFolderName)).toBeTruthy(); - + // Act await umbracoUi.dataType.clickRootFolderCaretButton(); await umbracoUi.dataType.clickActionsMenuForDataType(wrongDataTypeFolderName); @@ -51,7 +51,7 @@ test.describe('Data Types Folder tests', () => { // Arrange await umbracoApi.dataType.createFolder(dataTypeFolderName); expect(await umbracoApi.dataType.doesNameExist(dataTypeFolderName)).toBeTruthy(); - + // Act await umbracoUi.dataType.clickRootFolderCaretButton(); await umbracoUi.dataType.deleteDataTypeFolder(dataTypeFolderName); @@ -64,7 +64,7 @@ test.describe('Data Types Folder tests', () => { // Arrange let dataTypeFolderId = await umbracoApi.dataType.createFolder(dataTypeFolderName); expect(await umbracoApi.dataType.doesNameExist(dataTypeFolderName)).toBeTruthy(); - + // Act await umbracoUi.dataType.clickRootFolderCaretButton(); await umbracoUi.dataType.clickActionsMenuForDataType(dataTypeFolderName); @@ -76,16 +76,17 @@ test.describe('Data Types Folder tests', () => { // Assert expect(await umbracoApi.dataType.doesNameExist(dataTypeName)).toBeTruthy(); const dataTypeChildren = await umbracoApi.dataType.getChildren(dataTypeFolderId); - expect(dataTypeChildren[0].name).toBe(dataTypeName); - expect(dataTypeChildren[0].isFolder).toBeFalsy(); + expect(dataTypeChildren[0].name).toBe(dataTypeName); + expect(dataTypeChildren[0].isFolder).toBeFalsy(); }); - test('can create a folder in a folder', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can create a folder in a folder', async ({umbracoApi, umbracoUi}) => { // Arrange const childFolderName = 'Child Folder'; let dataTypeFolderId = await umbracoApi.dataType.createFolder(dataTypeFolderName); expect(await umbracoApi.dataType.doesNameExist(dataTypeFolderName)).toBeTruthy(); - + // Act await umbracoUi.dataType.clickRootFolderCaretButton(); await umbracoUi.dataType.clickActionsMenuForDataType(dataTypeFolderName); @@ -97,17 +98,18 @@ test.describe('Data Types Folder tests', () => { // Assert expect(await umbracoApi.dataType.doesNameExist(childFolderName)).toBeTruthy(); const dataTypeChildren = await umbracoApi.dataType.getChildren(dataTypeFolderId); - expect(dataTypeChildren[0].name).toBe(childFolderName); - expect(dataTypeChildren[0].isFolder).toBeTruthy(); + expect(dataTypeChildren[0].name).toBe(childFolderName); + expect(dataTypeChildren[0].isFolder).toBeTruthy(); }); - test('cannot delete a non-empty data type folder', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('cannot delete a non-empty data type folder', async ({umbracoApi, umbracoUi}) => { // Arrange let dataTypeFolderId = await umbracoApi.dataType.createFolder(dataTypeFolderName); expect(await umbracoApi.dataType.doesNameExist(dataTypeFolderName)).toBeTruthy(); await umbracoApi.dataType.create(dataTypeName, editorAlias, [], dataTypeFolderId); expect(await umbracoApi.dataType.doesNameExist(dataTypeName)).toBeTruthy(); - + // Act await umbracoUi.dataType.clickRootFolderCaretButton(); await umbracoUi.dataType.deleteDataTypeFolder(dataTypeFolderName); @@ -117,8 +119,8 @@ test.describe('Data Types Folder tests', () => { expect(await umbracoApi.dataType.doesNameExist(dataTypeName)).toBeTruthy(); expect(await umbracoApi.dataType.doesNameExist(dataTypeFolderName)).toBeTruthy(); const dataTypeChildren = await umbracoApi.dataType.getChildren(dataTypeFolderId); - expect(dataTypeChildren[0].name).toBe(dataTypeName); - expect(dataTypeChildren[0].isFolder).toBeFalsy(); + expect(dataTypeChildren[0].name).toBe(dataTypeName); + expect(dataTypeChildren[0].isFolder).toBeFalsy(); }); test.skip('can move a data type to a data type folder', async ({}) => { diff --git a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ImageCropper.spec.ts b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ImageCropper.spec.ts index 130f418234..12fb60e1be 100644 --- a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ImageCropper.spec.ts +++ b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/ImageCropper.spec.ts @@ -14,11 +14,12 @@ test.describe(`${dataTypeName} tests`, () => { test.afterEach(async ({umbracoApi}) => { if (dataTypeDefaultData !== null) { - await umbracoApi.dataType.update(dataTypeDefaultData.id, dataTypeDefaultData); - } + await umbracoApi.dataType.update(dataTypeDefaultData.id, dataTypeDefaultData); + } }); - test('can add crop', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can add crop', async ({umbracoApi, umbracoUi}) => { // Arrange const cropData = ['Test Alias', 100, 50]; const expectedDataTypeValues = [{ @@ -52,7 +53,8 @@ test.describe(`${dataTypeName} tests`, () => { expect(dataTypeData.values).toEqual(expectedDataTypeValues); }); - test('can edit crop', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can edit crop', async ({umbracoApi, umbracoUi}) => { // Arrange const wrongCropData = ['Wrong Alias', 50, 100]; const wrongDataTypeValues = [{ diff --git a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Radiobox.spec.ts b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Radiobox.spec.ts index e9129cb414..d2afbb700f 100644 --- a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Radiobox.spec.ts +++ b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Radiobox.spec.ts @@ -3,7 +3,7 @@ import {expect} from "@playwright/test"; const dataTypeName = 'Radiobox'; test.describe(`${dataTypeName} tests`, () => { - let dataTypeDefaultData = null; + let dataTypeDefaultData = null; let dataTypeData = null; test.beforeEach(async ({umbracoUi, umbracoApi}) => { @@ -14,11 +14,12 @@ test.describe(`${dataTypeName} tests`, () => { test.afterEach(async ({umbracoApi}) => { if (dataTypeDefaultData !== null) { - await umbracoApi.dataType.update(dataTypeDefaultData.id, dataTypeDefaultData); - } + await umbracoApi.dataType.update(dataTypeDefaultData.id, dataTypeDefaultData); + } }); - test('can add option', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can add option', async ({umbracoApi, umbracoUi}) => { // Arrange const optionName = 'Test option'; const expectedDataTypeValues = [{ @@ -32,7 +33,7 @@ test.describe(`${dataTypeName} tests`, () => { // Remove all existing options dataTypeData = await umbracoApi.dataType.getByName(dataTypeName); dataTypeData.values = []; - await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); + await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); await umbracoUi.dataType.goToDataType(dataTypeName); // Act @@ -45,7 +46,8 @@ test.describe(`${dataTypeName} tests`, () => { expect(dataTypeData.values).toEqual(expectedDataTypeValues); }); - test('can remove option', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can remove option', async ({umbracoApi, umbracoUi}) => { // Arrange const removedOptionName = 'Removed Option'; const removedOptionValues = [{ @@ -59,7 +61,7 @@ test.describe(`${dataTypeName} tests`, () => { // Remove all existing options and add an option to remove dataTypeData = await umbracoApi.dataType.getByName(dataTypeName); dataTypeData.values = removedOptionValues; - await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); + await umbracoApi.dataType.update(dataTypeData.id, dataTypeData); await umbracoUi.dataType.goToDataType(dataTypeName); // Act diff --git a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Upload.spec.ts b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Upload.spec.ts index 2b7821abe6..152b019cef 100644 --- a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Upload.spec.ts +++ b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/Upload.spec.ts @@ -19,7 +19,8 @@ for (const uploadType of uploadTypes) { } }); - test("can add accepted file extension", async ({ umbracoApi, umbracoUi }) => { + //TODO: Remove skip when the frontend is ready + test.skip("can add accepted file extension", async ({ umbracoApi, umbracoUi }) => { // Arrange const fileExtensionValue = 'zip'; const expectedDataTypeValues = [ @@ -48,7 +49,8 @@ for (const uploadType of uploadTypes) { expect(dataTypeData.values).toEqual(expectedDataTypeValues); }); - test("can remove accepted file extension", async ({ umbracoApi, umbracoUi }) => { + //TODO: Remove skip when the frontend is ready + test.skip("can remove accepted file extension", async ({ umbracoApi, umbracoUi }) => { // Arrange const removedFileExtensionValue = "bat"; const removedFileExtensionsValues = [ diff --git a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/CreatedPackages.spec.ts b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/CreatedPackages.spec.ts index a14bfc64d6..2a9f0034ee 100644 --- a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/CreatedPackages.spec.ts +++ b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/CreatedPackages.spec.ts @@ -355,7 +355,8 @@ test.describe('Created packages tests', () => { }); // Currently you are not able to download a package - test('can download a package', async ({umbracoApi, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can download a package', async ({umbracoApi, umbracoUi}) => { // Arrange const packageId = await umbracoApi.package.createEmptyPackage(packageName); await umbracoUi.reloadPage(); diff --git a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/InstalledPackages.spec.ts b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/InstalledPackages.spec.ts index 8e5d455a89..835932b39f 100644 --- a/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/InstalledPackages.spec.ts +++ b/tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Packages/InstalledPackages.spec.ts @@ -2,7 +2,8 @@ import {ConstantHelper, test} from '@umbraco/playwright-testhelpers'; test.describe('Installed packages tests', () => { // We can't install any packages so we do not have any installed. - test('can see no package have been installed', async ({page, umbracoUi}) => { + //TODO: Remove skip when the frontend is ready + test.skip('can see no package have been installed', async ({page, umbracoUi}) => { // Arrange await umbracoUi.goToBackOffice(); await umbracoUi.package.goToSection(ConstantHelper.sections.packages);