Skip to content

Commit 17d4417

Browse files
authored
V14 QA Fixed the failing smoke tests (#16953)
* Updated api tests for Data type due to test helper changes * Fixed Data Type tests due to test helper changes * Updated the test for edit password due to ui helper changes * Bumped version
1 parent 8eeac27 commit 17d4417

5 files changed

Lines changed: 21 additions & 21 deletions

File tree

tests/Umbraco.Tests.AcceptanceTest/package-lock.json

Lines changed: 4 additions & 4 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

tests/Umbraco.Tests.AcceptanceTest/package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
},
2323
"dependencies": {
2424
"@umbraco/json-models-builders": "^2.0.17",
25-
"@umbraco/playwright-testhelpers": "^2.0.0-beta.76",
25+
"@umbraco/playwright-testhelpers": "^2.0.0-beta.77",
2626
"camelize": "^1.0.0",
2727
"dotenv": "^16.3.1",
2828
"faker": "^4.1.0",

tests/Umbraco.Tests.AcceptanceTest/tests/ApiTesting/DataType/DataType.spec.ts

Lines changed: 6 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ test.describe('DataType tests', () => {
99
const dataTypeName = 'TestDataType';
1010
const folderName = 'TestDataTypeFolder';
1111
const editorAlias = 'Umbraco.DateTime';
12+
const editorUiAlias = 'Umb.PropertyEditorUi.DatePicker';
1213
const dataTypeData = [
1314
{
1415
"alias": "tester",
@@ -29,15 +30,15 @@ test.describe('DataType tests', () => {
2930

3031
test('can create dataType', async ({umbracoApi}) => {
3132
// Act
32-
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, dataTypeData);
33+
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, editorUiAlias, dataTypeData);
3334

3435
// Assert
3536
expect(umbracoApi.dataType.doesExist(dataTypeId)).toBeTruthy();
3637
});
3738

3839
test('can update dataType', async ({umbracoApi}) => {
3940
// Arrange
40-
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, []);
41+
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, editorUiAlias, []);
4142
const dataType = await umbracoApi.dataType.get(dataTypeId);
4243
dataType.values = dataTypeData;
4344

@@ -52,7 +53,7 @@ test.describe('DataType tests', () => {
5253

5354
test('can delete dataType', async ({umbracoApi}) => {
5455
// Arrange
55-
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, dataTypeData);
56+
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, editorUiAlias, dataTypeData);
5657
expect(await umbracoApi.dataType.doesExist(dataTypeId)).toBeTruthy();
5758

5859
// Act
@@ -65,7 +66,7 @@ test.describe('DataType tests', () => {
6566
test('can move a dataType to a folder', async ({umbracoApi}) => {
6667
// Arrange
6768
await umbracoApi.dataType.ensureNameNotExists(folderName);
68-
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, dataTypeData);
69+
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, editorUiAlias, dataTypeData);
6970
expect(await umbracoApi.dataType.doesNameExist(dataTypeName)).toBeTruthy();
7071
dataTypeFolderId = await umbracoApi.dataType.createFolder(folderName);
7172
expect(await umbracoApi.dataType.doesFolderExist(dataTypeFolderId)).toBeTruthy();
@@ -82,7 +83,7 @@ test.describe('DataType tests', () => {
8283
test('can copy a dataType to a folder', async ({umbracoApi}) => {
8384
// Arrange
8485
await umbracoApi.dataType.ensureNameNotExists(folderName);
85-
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, dataTypeData);
86+
dataTypeId = await umbracoApi.dataType.create(dataTypeName, editorAlias, editorUiAlias, dataTypeData);
8687
dataTypeFolderId = await umbracoApi.dataType.createFolder(folderName);
8788

8889
const dataType = await umbracoApi.dataType.get(dataTypeId);

tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/DataType/DataType.spec.ts

Lines changed: 9 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,6 @@
22
import {expect} from "@playwright/test";
33

44
const dataTypeName = 'TestDataType';
5-
const editorAlias = 'Umbraco.ColorPicker';
6-
const propertyEditorName = 'Color Picker';
75

86
test.beforeEach(async ({umbracoApi, umbracoUi}) => {
97
await umbracoApi.dataType.ensureNameNotExists(dataTypeName);
@@ -22,7 +20,7 @@ test('can create a data type', {tag: '@smoke'}, async ({umbracoApi, umbracoUi})
2220
await umbracoUi.dataType.clickNewDataTypeThreeDotsButton();
2321
await umbracoUi.dataType.enterDataTypeName(dataTypeName);
2422
await umbracoUi.dataType.clickSelectAPropertyEditorButton();
25-
await umbracoUi.dataType.selectAPropertyEditor(propertyEditorName);
23+
await umbracoUi.dataType.selectAPropertyEditor('Text Box');
2624
await umbracoUi.dataType.clickSaveButton();
2725

2826
// Assert
@@ -34,7 +32,7 @@ test('can rename a data type', {tag: '@smoke'}, async ({umbracoApi, umbracoUi})
3432
// Arrange
3533
const wrongDataTypeName = 'Wrong Data Type';
3634
await umbracoApi.dataType.ensureNameNotExists(wrongDataTypeName);
37-
await umbracoApi.dataType.create(wrongDataTypeName, editorAlias, []);
35+
await umbracoApi.dataType.createTextstringDataType(wrongDataTypeName);
3836
expect(await umbracoApi.dataType.doesNameExist(wrongDataTypeName)).toBeTruthy();
3937

4038
// Act
@@ -49,7 +47,7 @@ test('can rename a data type', {tag: '@smoke'}, async ({umbracoApi, umbracoUi})
4947

5048
test('can delete a data type', {tag: '@smoke'}, async ({umbracoApi, umbracoUi}) => {
5149
// Arrange
52-
await umbracoApi.dataType.create(dataTypeName, editorAlias, []);
50+
await umbracoApi.dataType.createTextstringDataType(dataTypeName);
5351
expect(await umbracoApi.dataType.doesNameExist(dataTypeName)).toBeTruthy();
5452

5553
// Act
@@ -67,7 +65,7 @@ test('can change property editor in a data type', {tag: '@smoke'}, async ({umbra
6765
const updatedEditorAlias = 'Umbraco.TextArea';
6866
const updatedEditorUiAlias = 'Umb.PropertyEditorUi.TextArea';
6967

70-
await umbracoApi.dataType.create(dataTypeName, editorAlias, []);
68+
await umbracoApi.dataType.createTextstringDataType(dataTypeName);
7169
expect(await umbracoApi.dataType.doesNameExist(dataTypeName)).toBeTruthy();
7270

7371
// Act
@@ -98,16 +96,17 @@ test('cannot create a data type without selecting the property editor', {tag: '@
9896

9997
test('can change settings', {tag: '@smoke'}, async ({umbracoApi, umbracoUi}) => {
10098
// Arrange
99+
const maxCharsValue = 126;
101100
const expectedDataTypeValues = {
102-
alias: "useLabel",
103-
value: true
101+
"alias": "maxChars",
102+
"value": maxCharsValue
104103
};
105-
await umbracoApi.dataType.create(dataTypeName, editorAlias, []);
104+
await umbracoApi.dataType.createTextstringDataType(dataTypeName);
106105
expect(await umbracoApi.dataType.doesNameExist(dataTypeName)).toBeTruthy();
107106

108107
// Act
109108
await umbracoUi.dataType.goToDataType(dataTypeName);
110-
await umbracoUi.dataType.clickIncludeLabelsSlider();
109+
await umbracoUi.dataType.enterMaximumAllowedCharactersValue(maxCharsValue.toString());
111110
await umbracoUi.dataType.clickSaveButton();
112111

113112
// Assert

tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Members/Members.spec.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ test('can edit password', {tag: '@smoke'}, async ({umbracoApi, umbracoUi}) => {
106106
// Act
107107
await umbracoUi.member.clickMemberLinkByName(memberName);
108108
await umbracoUi.member.clickChangePasswordButton();
109-
await umbracoUi.member.enterPassword(updatedPassword);
109+
await umbracoUi.member.enterNewPassword(updatedPassword);
110110
await umbracoUi.member.enterConfirmNewPassword(updatedPassword);
111111
await umbracoUi.member.clickSaveButton();
112112

0 commit comments

Comments
 (0)