Skip to content

V14 QA Added Content tests with various of data types#16824

Merged
nhudinh0309 merged 50 commits intov14/devfrom
v14/QA/content-with-image-cropper-tests
Aug 27, 2024
Merged

V14 QA Added Content tests with various of data types#16824
nhudinh0309 merged 50 commits intov14/devfrom
v14/QA/content-with-image-cropper-tests

Conversation

@nhudinh0309
Copy link
Copy Markdown
Contributor

@nhudinh0309 nhudinh0309 commented Jul 26, 2024

Added Content tests with various of data types:

  • Image Cropper
  • Member Picker
  • Multiple Image Media Picker
  • Numeric

@nhudinh0309 nhudinh0309 changed the title V14 QA Added Content tests with Image Cropper data type V14 QA Added Content tests with various of data types Aug 5, 2024
Copy link
Copy Markdown
Contributor

@andr317c andr317c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had comments

Copy link
Copy Markdown
Contributor

@andr317c andr317c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had a comment

Copy link
Copy Markdown
Contributor

@andr317c andr317c left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Had a few comments

await umbracoApi.documentType.ensureNameNotExists(documentTypeName);
await umbracoApi.document.ensureNameNotExists(contentName);
contentPickerDocumentTypeId = await umbracoApi.documentType.createDefaultDocumentTypeWithAllowAsRoot(contentPickerDocumentTypeName);
await umbracoUi.goToBackOffice();
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Why was this removed from the beforeEach?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Because we need to go to the back office after creating the content in some tests, this step should be defined in each test rather than in the beforeEach

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants