Skip to content

Commit b6bb191

Browse files
authored
Merge pull request #690 from gemini-testing/users/rocketraccoon/TESTPLANE-642.fix-screenshots
Hide unnecessary screenshots
2 parents 167796f + d191b56 commit b6bb191

File tree

1 file changed

+28
-16
lines changed
  • lib/static/new-ui/components/TreeViewItemSubtitle

1 file changed

+28
-16
lines changed

lib/static/new-ui/components/TreeViewItemSubtitle/index.tsx

Lines changed: 28 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ import stripAnsi from 'strip-ansi';
44

55
import {TreeViewItemData} from '@/static/new-ui/features/suites/components/SuitesPage/types';
66
import {ImageWithMagnifier} from '@/static/new-ui/components/ImageWithMagnifier';
7-
import {ImageEntityFail} from '@/static/new-ui/types/store';
87
import styles from './index.module.css';
98
import {getAssertViewStatusMessage} from '@/static/new-ui/utils/assert-view-status';
109
import {makeLinksClickable} from '@/static/new-ui/utils';
@@ -30,28 +29,41 @@ export function TreeViewItemSubtitle(props: TreeViewItemSubtitleProps): ReactNod
3029
return (
3130
<div>
3231
{props.item.images.map((imageEntity) => {
33-
const imageItem: ImageEntityFail = imageEntity as ImageEntityFail;
32+
const images = [];
3433

35-
const images = [
36-
{
34+
if (
35+
imageEntity.status === TestStatus.FAIL ||
36+
imageEntity.status === TestStatus.UPDATED ||
37+
imageEntity.status === TestStatus.SUCCESS
38+
) {
39+
images.push({
3740
title: 'Expected',
38-
image: imageItem.expectedImg
39-
},
40-
{
41+
image: imageEntity.expectedImg
42+
});
43+
}
44+
45+
if (
46+
imageEntity.status !== TestStatus.UPDATED &&
47+
imageEntity.status !== TestStatus.SUCCESS
48+
) {
49+
images.push({
4150
title: 'Actual',
42-
image: imageItem.actualImg
43-
},
44-
{
51+
image: imageEntity.actualImg
52+
});
53+
}
54+
55+
if (imageEntity.status === TestStatus.FAIL) {
56+
images.push({
4557
title: 'Diff',
46-
image: imageItem.diffImg
47-
}
48-
].filter(({image}) => Boolean(image));
58+
image: imageEntity.diffImg
59+
});
60+
}
4961

5062
return (
51-
<div key={imageItem.id}>
52-
<span className={styles.imageStatus}>{imageItem.stateName}{getAssertViewStatusMessage(imageEntity)}</span>
63+
<div key={imageEntity.id}>
64+
<span className={styles.imageStatus}>{imageEntity.stateName}{getAssertViewStatusMessage(imageEntity)}</span>
5365
<div className={styles.imageDiff}>
54-
{images.map((item) => (
66+
{images.filter(({image}) => Boolean(image)).map((item) => (
5567
<div
5668
className={
5769
classNames(styles.imageDiffItem, images.length === 3 && item.title !== 'Diff' && styles.canHide)

0 commit comments

Comments
 (0)