Skip to content

Commit 8e02308

Browse files
karancs06karancs06
committed
Revert "adding logs"
This reverts commit b3c9f83.
1 parent f4156b1 commit 8e02308

File tree

5 files changed

+47
-421
lines changed

5 files changed

+47
-421
lines changed

‎src/visualBuilder/__test__/hover/fields/all-hover.test.ts‎

Lines changed: 0 additions & 90 deletions
Original file line numberDiff line numberDiff line change
@@ -107,40 +107,25 @@ describe("When an element is hovered in visual builder mode", () => {
107107
const fieldSchemaMap = getFieldSchemaMap().all_fields;
108108

109109
beforeAll(() => {
110-
const startTime = performance.now();
111110
// Pre-set all field schemas in cache to avoid async fetches during hover
112111
// This significantly speeds up tests, especially for html-rte, json-rte, link fields
113112
FieldSchemaMap.setFieldSchema("all_fields", fieldSchemaMap);
114113

115114
// Field schemas are already set above - no need for additional caching
116115
// The FieldSchemaMap.setFieldSchema call above sets all fields at once
117-
const endTime = performance.now();
118-
console.log(
119-
`[TIMING] beforeAll - setFieldSchema: ${(endTime - startTime).toFixed(2)}ms`
120-
);
121116
});
122117

123118
beforeEach(() => {
124-
const startTime = performance.now();
125119
Config.reset();
126120
Config.set("mode", 2);
127121
mousemoveEvent = new Event("mousemove", {
128122
bubbles: true,
129123
cancelable: true,
130124
});
131-
const endTime = performance.now();
132-
console.log(
133-
`[TIMING] beforeEach (outer) - Config setup: ${(endTime - startTime).toFixed(2)}ms`
134-
);
135125
});
136126

137127
afterEach(async () => {
138-
const startTime = performance.now();
139128
document.getElementsByTagName("html")[0].innerHTML = "";
140-
const endTime = performance.now();
141-
console.log(
142-
`[TIMING] afterEach (outer) - cleanup: ${(endTime - startTime).toFixed(2)}ms`
143-
);
144129
});
145130

146131
afterAll(() => {
@@ -154,37 +139,18 @@ describe("When an element is hovered in visual builder mode", () => {
154139
let visualBuilder: VisualBuilder;
155140

156141
beforeEach(() => {
157-
const startTime = performance.now();
158142
fieldElement = document.createElement("p");
159143
fieldElement.setAttribute("data-cslp", SINGLE_FIELD.cslp);
160144
fieldElement.getBoundingClientRect = vi
161145
.fn()
162146
.mockReturnValue(mockDomRect.singleLeft());
163147

164148
document.body.appendChild(fieldElement);
165-
const domSetupTime = performance.now();
166-
console.log(
167-
`[TIMING] beforeEach (${SINGLE_FIELD.name} field) - DOM setup: ${(domSetupTime - startTime).toFixed(2)}ms`
168-
);
169-
170-
const vbStartTime = performance.now();
171149
visualBuilder = new VisualBuilder();
172-
const vbEndTime = performance.now();
173-
console.log(
174-
`[TIMING] beforeEach (${SINGLE_FIELD.name} field) - VisualBuilder init: ${(vbEndTime - vbStartTime).toFixed(2)}ms`
175-
);
176-
console.log(
177-
`[TIMING] beforeEach (${SINGLE_FIELD.name} field) - TOTAL: ${(vbEndTime - startTime).toFixed(2)}ms`
178-
);
179150
});
180151

181152
afterEach(() => {
182-
const startTime = performance.now();
183153
visualBuilder.destroy();
184-
const endTime = performance.now();
185-
console.log(
186-
`[TIMING] afterEach (${SINGLE_FIELD.name} field) - VisualBuilder.destroy: ${(endTime - startTime).toFixed(2)}ms`
187-
);
188154
});
189155

190156
test("should have outline and custom cursor", async () => {
@@ -217,37 +183,18 @@ describe("When an element is hovered in visual builder mode", () => {
217183
let visualBuilder: VisualBuilder;
218184

219185
beforeEach(() => {
220-
const startTime = performance.now();
221186
fieldElement = document.createElement("p");
222187
fieldElement.setAttribute("data-cslp", MULTIPLE_FIELD.cslp);
223188
fieldElement.getBoundingClientRect = vi
224189
.fn()
225190
.mockReturnValue(mockDomRect.singleLeft());
226191

227192
document.body.appendChild(fieldElement);
228-
const domSetupTime = performance.now();
229-
console.log(
230-
`[TIMING] beforeEach (${MULTIPLE_FIELD.name} field) - DOM setup: ${(domSetupTime - startTime).toFixed(2)}ms`
231-
);
232-
233-
const vbStartTime = performance.now();
234193
visualBuilder = new VisualBuilder();
235-
const vbEndTime = performance.now();
236-
console.log(
237-
`[TIMING] beforeEach (${MULTIPLE_FIELD.name} field) - VisualBuilder init: ${(vbEndTime - vbStartTime).toFixed(2)}ms`
238-
);
239-
console.log(
240-
`[TIMING] beforeEach (${MULTIPLE_FIELD.name} field) - TOTAL: ${(vbEndTime - startTime).toFixed(2)}ms`
241-
);
242194
});
243195

244196
afterEach(() => {
245-
const startTime = performance.now();
246197
visualBuilder.destroy();
247-
const endTime = performance.now();
248-
console.log(
249-
`[TIMING] afterEach (${MULTIPLE_FIELD.name} field) - VisualBuilder.destroy: ${(endTime - startTime).toFixed(2)}ms`
250-
);
251198
});
252199

253200
test("should have outline and custom cursor", async () => {
@@ -281,7 +228,6 @@ describe("When an element is hovered in visual builder mode", () => {
281228
let visualBuilder: VisualBuilder;
282229

283230
beforeEach(() => {
284-
const startTime = performance.now();
285231
container = document.createElement("div");
286232
container.setAttribute("data-cslp", MULTIPLE_FIELD.multipleCslp);
287233
container.getBoundingClientRect = vi
@@ -309,29 +255,12 @@ describe("When an element is hovered in visual builder mode", () => {
309255
container.appendChild(firstField);
310256
container.appendChild(secondField);
311257
document.body.appendChild(container);
312-
const domSetupTime = performance.now();
313-
console.log(
314-
`[TIMING] beforeEach (${MULTIPLE_FIELD.name} field multiple) - DOM setup: ${(domSetupTime - startTime).toFixed(2)}ms`
315-
);
316258

317-
const vbStartTime = performance.now();
318259
visualBuilder = new VisualBuilder();
319-
const vbEndTime = performance.now();
320-
console.log(
321-
`[TIMING] beforeEach (${MULTIPLE_FIELD.name} field multiple) - VisualBuilder init: ${(vbEndTime - vbStartTime).toFixed(2)}ms`
322-
);
323-
console.log(
324-
`[TIMING] beforeEach (${MULTIPLE_FIELD.name} field multiple) - TOTAL: ${(vbEndTime - startTime).toFixed(2)}ms`
325-
);
326260
});
327261

328262
afterEach(() => {
329-
const startTime = performance.now();
330263
visualBuilder.destroy();
331-
const endTime = performance.now();
332-
console.log(
333-
`[TIMING] afterEach (${MULTIPLE_FIELD.name} field multiple) - VisualBuilder.destroy: ${(endTime - startTime).toFixed(2)}ms`
334-
);
335264
});
336265

337266
test("should have outline and custom cursor on container", async () => {
@@ -360,30 +289,15 @@ describe("When an element is hovered in visual builder mode", () => {
360289
const testStartTime = performance.now();
361290
const dispatchStartTime = performance.now();
362291
firstField.dispatchEvent(mousemoveEvent);
363-
const dispatchEndTime = performance.now();
364-
console.log(
365-
`[TIMING] test - dispatchEvent: ${(dispatchEndTime - dispatchStartTime).toFixed(2)}ms`
366-
);
367-
368-
const hoverOutlineStartTime = performance.now();
369292
await waitForHoverOutline();
370-
const hoverOutlineEndTime = performance.now();
371-
console.log(
372-
`[TIMING] test - waitForHoverOutline: ${(hoverOutlineEndTime - hoverOutlineStartTime).toFixed(2)}ms`
373-
);
374293

375294
const hoverOutline = document.querySelector(
376295
"[data-testid='visual-builder__hover-outline']"
377296
);
378297
expect(hoverOutline).toHaveAttribute("style");
379298

380299
// Wait for cursor icon to be set (not "loading")
381-
const cursorIconStartTime = performance.now();
382300
await waitForCursorIcon(MULTIPLE_FIELD.icon, { timeout: 5000 });
383-
const cursorIconEndTime = performance.now();
384-
console.log(
385-
`[TIMING] test - waitForCursorIcon: ${(cursorIconEndTime - cursorIconStartTime).toFixed(2)}ms`
386-
);
387301

388302
const customCursor = document.querySelector(
389303
`[data-testid="visual-builder__cursor"]`
@@ -393,10 +307,6 @@ describe("When an element is hovered in visual builder mode", () => {
393307
MULTIPLE_FIELD.icon
394308
);
395309
expect(customCursor?.classList.contains("visible")).toBeTruthy();
396-
const testEndTime = performance.now();
397-
console.log(
398-
`[TIMING] test - TOTAL: ${(testEndTime - testStartTime).toFixed(2)}ms`
399-
);
400310
});
401311
});
402312
});

‎src/visualBuilder/__test__/hover/fields/file.test.ts‎

Lines changed: 0 additions & 85 deletions
Original file line numberDiff line numberDiff line change
@@ -83,7 +83,6 @@ describe("When an element is hovered in visual builder mode", () => {
8383
let mousemoveEvent: Event;
8484

8585
beforeAll(() => {
86-
const startTime = performance.now();
8786
FieldSchemaMap.setFieldSchema(
8887
"all_fields",
8988
getFieldSchemaMap().all_fields
@@ -93,36 +92,22 @@ describe("When an element is hovered in visual builder mode", () => {
9392
observe: vi.fn(),
9493
disconnect: vi.fn(),
9594
}));
96-
const endTime = performance.now();
97-
console.log(
98-
`[TIMING] beforeAll - setFieldSchema: ${(endTime - startTime).toFixed(2)}ms`
99-
);
10095
});
10196

10297
beforeEach(() => {
103-
const startTime = performance.now();
10498
Config.reset();
10599
Config.set("mode", 2);
106100
mousemoveEvent = new Event("mousemove", {
107101
bubbles: true,
108102
cancelable: true,
109103
});
110104
document.getElementsByTagName("html")[0].innerHTML = "";
111-
const endTime = performance.now();
112-
console.log(
113-
`[TIMING] beforeEach (outer) - Config setup: ${(endTime - startTime).toFixed(2)}ms`
114-
);
115105
});
116106

117107
afterEach(async () => {
118-
const startTime = performance.now();
119108
// Wait longer for any pending async operations (like fetchEntryPermissionsAndStageDetails) to complete
120109
// await new Promise((resolve) => setTimeout(resolve, 500));
121110
document.getElementsByTagName("html")[0].innerHTML = "";
122-
const endTime = performance.now();
123-
console.log(
124-
`[TIMING] afterEach (outer) - cleanup: ${(endTime - startTime).toFixed(2)}ms`
125-
);
126111
});
127112

128113
afterAll(() => {
@@ -135,7 +120,6 @@ describe("When an element is hovered in visual builder mode", () => {
135120
let visualBuilder: VisualBuilder;
136121

137122
beforeEach(() => {
138-
const startTime = performance.now();
139123
fileField = document.createElement("p");
140124
fileField.setAttribute(
141125
"data-cslp",
@@ -154,29 +138,12 @@ describe("When an element is hovered in visual builder mode", () => {
154138

155139
document.body.appendChild(fileField);
156140
document.body.appendChild(imageField);
157-
const domSetupTime = performance.now();
158-
console.log(
159-
`[TIMING] beforeEach (file field) - DOM setup: ${(domSetupTime - startTime).toFixed(2)}ms`
160-
);
161141

162-
const vbStartTime = performance.now();
163142
visualBuilder = new VisualBuilder();
164-
const vbEndTime = performance.now();
165-
console.log(
166-
`[TIMING] beforeEach (file field) - VisualBuilder init: ${(vbEndTime - vbStartTime).toFixed(2)}ms`
167-
);
168-
console.log(
169-
`[TIMING] beforeEach (file field) - TOTAL: ${(vbEndTime - startTime).toFixed(2)}ms`
170-
);
171143
});
172144

173145
afterEach(() => {
174-
const startTime = performance.now();
175146
visualBuilder.destroy();
176-
const endTime = performance.now();
177-
console.log(
178-
`[TIMING] afterEach (file field) - VisualBuilder.destroy: ${(endTime - startTime).toFixed(2)}ms`
179-
);
180147
});
181148

182149
test("should have outline and custom cursor", async () => {
@@ -198,25 +165,14 @@ describe("When an element is hovered in visual builder mode", () => {
198165
const testStartTime = performance.now();
199166
const dispatchStartTime = performance.now();
200167
imageField.dispatchEvent(mousemoveEvent);
201-
const dispatchEndTime = performance.now();
202-
console.log(
203-
`[TIMING] test - dispatchEvent: ${(dispatchEndTime - dispatchStartTime).toFixed(2)}ms`
204-
);
205-
206-
const hoverOutlineStartTime = performance.now();
207168
await waitForHoverOutline();
208-
const hoverOutlineEndTime = performance.now();
209-
console.log(
210-
`[TIMING] test - waitForHoverOutline: ${(hoverOutlineEndTime - hoverOutlineStartTime).toFixed(2)}ms`
211-
);
212169

213170
const hoverOutline = document.querySelector(
214171
"[data-testid='visual-builder__hover-outline']"
215172
);
216173
expect(hoverOutline).toHaveAttribute("style");
217174

218175
// Wait for cursor icon to be set (not "loading") - optimized timeout
219-
const waitForStartTime = performance.now();
220176
await waitFor(
221177
() => {
222178
const customCursor = document.querySelector(
@@ -226,20 +182,12 @@ describe("When an element is hovered in visual builder mode", () => {
226182
},
227183
{ timeout: 2000, interval: 10 } // Optimized: reduced timeout and faster polling
228184
);
229-
const waitForEndTime = performance.now();
230-
console.log(
231-
`[TIMING] test - waitFor cursor: ${(waitForEndTime - waitForStartTime).toFixed(2)}ms`
232-
);
233185

234186
const customCursor = document.querySelector(
235187
`[data-testid="visual-builder__cursor"]`
236188
);
237189
expect(customCursor).toHaveAttribute("data-icon", "file");
238190
expect(customCursor?.classList.contains("visible")).toBeTruthy();
239-
const testEndTime = performance.now();
240-
console.log(
241-
`[TIMING] test - TOTAL: ${(testEndTime - testStartTime).toFixed(2)}ms`
242-
);
243191
});
244192
});
245193

@@ -252,7 +200,6 @@ describe("When an element is hovered in visual builder mode", () => {
252200
let visualBuilder: VisualBuilder;
253201

254202
beforeEach(() => {
255-
const startTime = performance.now();
256203
container = document.createElement("div");
257204
container.setAttribute(
258205
"data-cslp",
@@ -306,29 +253,12 @@ describe("When an element is hovered in visual builder mode", () => {
306253
container.appendChild(firstImageField);
307254
container.appendChild(secondImageField);
308255
document.body.appendChild(container);
309-
const domSetupTime = performance.now();
310-
console.log(
311-
`[TIMING] beforeEach (file field multiple) - DOM setup: ${(domSetupTime - startTime).toFixed(2)}ms`
312-
);
313256

314-
const vbStartTime = performance.now();
315257
visualBuilder = new VisualBuilder();
316-
const vbEndTime = performance.now();
317-
console.log(
318-
`[TIMING] beforeEach (file field multiple) - VisualBuilder init: ${(vbEndTime - vbStartTime).toFixed(2)}ms`
319-
);
320-
console.log(
321-
`[TIMING] beforeEach (file field multiple) - TOTAL: ${(vbEndTime - startTime).toFixed(2)}ms`
322-
);
323258
});
324259

325260
afterEach(() => {
326-
const startTime = performance.now();
327261
visualBuilder.destroy();
328-
const endTime = performance.now();
329-
console.log(
330-
`[TIMING] afterEach (file field multiple) - VisualBuilder.destroy: ${(endTime - startTime).toFixed(2)}ms`
331-
);
332262
});
333263

334264
test("should have outline and custom cursor", async () => {
@@ -353,18 +283,7 @@ describe("When an element is hovered in visual builder mode", () => {
353283
const testStartTime = performance.now();
354284
const dispatchStartTime = performance.now();
355285
firstFileField.dispatchEvent(mousemoveEvent);
356-
const dispatchEndTime = performance.now();
357-
console.log(
358-
`[TIMING] test - dispatchEvent: ${(dispatchEndTime - dispatchStartTime).toFixed(2)}ms`
359-
);
360-
361-
const hoverOutlineStartTime = performance.now();
362286
await waitForHoverOutline();
363-
const hoverOutlineEndTime = performance.now();
364-
console.log(
365-
`[TIMING] test - waitForHoverOutline: ${(hoverOutlineEndTime - hoverOutlineStartTime).toFixed(2)}ms`
366-
);
367-
368287
const hoverOutline = document.querySelector(
369288
"[data-testid='visual-builder__hover-outline']"
370289
) as HTMLElement;
@@ -377,10 +296,6 @@ describe("When an element is hovered in visual builder mode", () => {
377296
);
378297
expect(customCursor?.getAttribute("data-icon")).toBe("file");
379298
expect(customCursor?.classList.contains("visible")).toBeTruthy();
380-
const testEndTime = performance.now();
381-
console.log(
382-
`[TIMING] test - TOTAL: ${(testEndTime - testStartTime).toFixed(2)}ms`
383-
);
384299
});
385300

386301
test("should have outline and custom cursor on the url", async () => {

0 commit comments

Comments
 (0)