Skip to content

Commit 656b605

Browse files
committed
chore: updated beforeScreenshot
- fixed and added UT's
1 parent 64a70fd commit 656b605

9 files changed

Lines changed: 939 additions & 394 deletions

File tree

packages/image-comparison-core/src/commands/saveFullPageScreen.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -60,7 +60,7 @@ export default async function saveFullPageScreen(
6060
toolBarShadowPadding,
6161
waitForFontsLoaded,
6262
}
63-
const enrichedInstanceData: BeforeScreenshotResult = await beforeScreenshot(methods.executor, beforeOptions, true)
63+
const enrichedInstanceData: BeforeScreenshotResult = await beforeScreenshot(beforeOptions, true)
6464
const {
6565
browserName,
6666
browserVersion,

packages/image-comparison-core/src/commands/saveWebElement.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@ export default async function saveWebElement(
5353
toolBarShadowPadding,
5454
waitForFontsLoaded,
5555
}
56-
const enrichedInstanceData: BeforeScreenshotResult = await beforeScreenshot(executor, beforeOptions, true)
56+
const enrichedInstanceData: BeforeScreenshotResult = await beforeScreenshot(beforeOptions, true)
5757
const {
5858
browserName,
5959
browserVersion,

packages/image-comparison-core/src/commands/saveWebScreen.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -49,7 +49,7 @@ export default async function saveWebScreen(
4949
toolBarShadowPadding,
5050
waitForFontsLoaded,
5151
}
52-
const enrichedInstanceData: BeforeScreenshotResult = await beforeScreenshot(methods.executor, beforeOptions)
52+
const enrichedInstanceData: BeforeScreenshotResult = await beforeScreenshot(beforeOptions)
5353
const {
5454
browserName,
5555
browserVersion,

packages/image-comparison-core/src/helpers/__snapshots__/beforeScreenshot.test.ts.snap

Lines changed: 6 additions & 128 deletions
Original file line numberDiff line numberDiff line change
@@ -2,146 +2,24 @@
22

33
exports[`beforeScreenshot > should be able to return the enriched instance data with \`addShadowPadding: true\` 1`] = `
44
{
5-
"addressBarShadowPadding": 0,
6-
"appName": "appName",
7-
"browserName": "browserName",
8-
"browserVersion": "browserVersion",
9-
"deviceName": "deviceName",
5+
"appName": "mocked-app",
6+
"browserName": "mocked-browser",
107
"devicePixelRatio": 1,
11-
"deviceRectangles": {
12-
"bottomBar": {
13-
"height": 0,
14-
"width": 0,
15-
"x": 0,
16-
"y": 0,
17-
},
18-
"homeBar": {
19-
"height": 0,
20-
"width": 0,
21-
"x": 0,
22-
"y": 0,
23-
},
24-
"leftSidePadding": {
25-
"height": 0,
26-
"width": 0,
27-
"x": 0,
28-
"y": 0,
29-
},
30-
"rightSidePadding": {
31-
"height": 0,
32-
"width": 0,
33-
"x": 0,
34-
"y": 0,
35-
},
36-
"screenSize": {
37-
"height": 1,
38-
"width": 1,
39-
},
40-
"statusBar": {
41-
"height": 0,
42-
"width": 0,
43-
"x": 0,
44-
"y": 0,
45-
},
46-
"statusBarAndAddressBar": {
47-
"height": 0,
48-
"width": 0,
49-
"x": 0,
50-
"y": 0,
51-
},
52-
"viewport": {
53-
"height": 0,
54-
"width": 0,
55-
"x": 0,
56-
"y": 0,
57-
},
58-
},
59-
"initialDevicePixelRatio": 1,
608
"isAndroid": false,
61-
"isAndroidChromeDriverScreenshot": false,
62-
"isAndroidNativeWebScreenshot": false,
639
"isIOS": false,
6410
"isMobile": false,
65-
"isTestInBrowser": true,
66-
"isTestInMobileBrowser": false,
67-
"logName": "logName",
68-
"name": "name",
69-
"nativeWebScreenshot": false,
70-
"platformName": "platformName",
71-
"platformVersion": "platformVersion",
72-
"toolBarShadowPadding": 0,
11+
"platformName": "mocked-platform",
7312
}
7413
`;
7514

7615
exports[`beforeScreenshot > should be able to return the enriched instance data with default options 1`] = `
7716
{
78-
"addressBarShadowPadding": 0,
79-
"appName": "appName",
80-
"browserName": "browserName",
81-
"browserVersion": "browserVersion",
82-
"deviceName": "deviceName",
17+
"appName": "mocked-app",
18+
"browserName": "mocked-browser",
8319
"devicePixelRatio": 1,
84-
"deviceRectangles": {
85-
"bottomBar": {
86-
"height": 0,
87-
"width": 0,
88-
"x": 0,
89-
"y": 0,
90-
},
91-
"homeBar": {
92-
"height": 0,
93-
"width": 0,
94-
"x": 0,
95-
"y": 0,
96-
},
97-
"leftSidePadding": {
98-
"height": 0,
99-
"width": 0,
100-
"x": 0,
101-
"y": 0,
102-
},
103-
"rightSidePadding": {
104-
"height": 0,
105-
"width": 0,
106-
"x": 0,
107-
"y": 0,
108-
},
109-
"screenSize": {
110-
"height": 1,
111-
"width": 1,
112-
},
113-
"statusBar": {
114-
"height": 0,
115-
"width": 0,
116-
"x": 0,
117-
"y": 0,
118-
},
119-
"statusBarAndAddressBar": {
120-
"height": 0,
121-
"width": 0,
122-
"x": 0,
123-
"y": 0,
124-
},
125-
"viewport": {
126-
"height": 0,
127-
"width": 0,
128-
"x": 0,
129-
"y": 0,
130-
},
131-
},
132-
"initialDevicePixelRatio": 1,
13320
"isAndroid": false,
134-
"isAndroidChromeDriverScreenshot": false,
135-
"isAndroidNativeWebScreenshot": false,
13621
"isIOS": false,
13722
"isMobile": false,
138-
"isTestInBrowser": true,
139-
"isTestInMobileBrowser": false,
140-
"logName": "logName",
141-
"name": "name",
142-
"nativeWebScreenshot": false,
143-
"platformName": "platformName",
144-
"platformVersion": "platformVersion",
145-
"toolBarShadowPadding": 0,
23+
"platformName": "mocked-platform",
14624
}
14725
`;

0 commit comments

Comments
 (0)