Skip to content

Commit 52aa18c

Browse files
authored
V15 QA Added acceptance tests to verify the SVG thumbnail (#17966)
* Added tests to verify the media thumbnail * Added tests to verify the media thumbnail * Bumped version * Make the SVG tests run in the pipeline * Cleaned code * Updated smokeTest command * Reverted * Updated the thumbnail file for audio * Reverted
1 parent 1c68e3d commit 52aa18c

File tree

2 files changed

+10
-8
lines changed

2 files changed

+10
-8
lines changed

tests/Umbraco.Tests.AcceptanceTest/tests/DefaultConfig/Content/ContentWithUploadVectorGraphics.spec.ts

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -80,10 +80,10 @@ test(`can upload a file with the svg extension in the content`, async ({umbracoA
8080
const contentData = await umbracoApi.document.getByName(contentName);
8181
expect(contentData.values[0].alias).toEqual(AliasHelper.toAlias(dataTypeName));
8282
expect(contentData.values[0].value.src).toContain(AliasHelper.toAlias(vectorGraphicsName));
83+
await umbracoUi.content.doesUploadedSvgThumbnailHaveSrc(contentData.values[0].value.src);
8384
});
8485

85-
// TODO: Remove skip when the front-end is ready. Currently the uploaded vector graphics file still displays after removing.
86-
test.skip('can remove an svg file in the content', async ({umbracoApi, umbracoUi}) => {
86+
test('can remove an svg file in the content', async ({umbracoApi, umbracoUi}) => {
8787
// Arrange
8888
const uploadVectorGraphicsName = 'VectorGraphics.svg';
8989
const mineType = 'image/svg+xml';

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

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -50,12 +50,12 @@ test('can rename a media file', async ({umbracoApi, umbracoUi}) => {
5050
});
5151

5252
const mediaFileTypes = [
53-
{fileName: 'Article', filePath: 'Article.pdf'},
54-
{fileName: 'Audio', filePath: 'Audio.mp3'},
55-
{fileName: 'File', filePath: 'File.txt'},
56-
{fileName: 'Image', filePath: 'Umbraco.png'},
57-
{fileName: 'Vector Graphics (SVG)', filePath: 'VectorGraphics.svg'},
58-
{fileName: 'Video', filePath: 'Video.mp4'}
53+
{fileName: 'Article', filePath: 'Article.pdf', thumbnail: 'icon-article'},
54+
{fileName: 'Audio', filePath: 'Audio.mp3', thumbnail: 'icon-audio-lines'},
55+
{fileName: 'File', filePath: 'File.txt', thumbnail: 'icon-document'},
56+
{fileName: 'Image', filePath: 'Umbraco.png', thumbnail: 'image'},
57+
{fileName: 'Vector Graphics (SVG)', filePath: 'VectorGraphics.svg', thumbnail: 'image'},
58+
{fileName: 'Video', filePath: 'Video.mp4', thumbnail: 'icon-video'}
5959
];
6060

6161
for (const mediaFileType of mediaFileTypes) {
@@ -72,6 +72,8 @@ for (const mediaFileType of mediaFileTypes) {
7272

7373
// Assert
7474
await umbracoUi.media.doesSuccessNotificationHaveText(NotificationConstantHelper.success.created);
75+
const mediaData = await umbracoApi.media.getByName(mediaFileType.fileName);
76+
await umbracoUi.media.doesMediaHaveThumbnail(mediaData.id, mediaFileType.thumbnail, mediaData.urls[0].url);
7577
await umbracoUi.media.reloadMediaTree();
7678
await umbracoUi.media.isMediaTreeItemVisible(mediaFileType.fileName);
7779
expect(await umbracoApi.media.doesNameExist(mediaFileType.fileName)).toBeTruthy();

0 commit comments

Comments
 (0)