Skip to content

Commit 42a9d6b

Browse files
OrKoNDevtools-frontend LUCI CQ
authored andcommitted
Migrate test/e2e/elements/sidebar-event-listeners_test.ts
Fixed: 416405248 Change-Id: I6488dea06f2829c543c7c051b5248b2ba86207f5 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6837818 Auto-Submit: Alex Rudenko <[email protected]> Commit-Queue: Alex Rudenko <[email protected]> Reviewed-by: Philip Pfaffe <[email protected]>
1 parent b19242e commit 42a9d6b

File tree

4 files changed

+161
-91
lines changed

4 files changed

+161
-91
lines changed

test/e2e/elements/BUILD.gn

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,6 @@ node_ts_library("elements") {
1313
"selection-after-delete_test.ts",
1414
"shadowroot-styles_test.ts",
1515
"sidebar-event-listeners-remove_test.ts",
16-
"sidebar-event-listeners_test.ts",
1716
"style-pane-properties_test.ts",
1817
"styles-disable-inherited_test.ts",
1918
]

test/e2e/elements/sidebar-event-listeners_test.ts

Lines changed: 0 additions & 90 deletions
This file was deleted.

test/e2e_non_hosted/elements/BUILD.gn

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,7 @@ ts_e2e_library("elements") {
2323
"pseudo-states_test.ts",
2424
"reveal-correct-node_test.ts",
2525
"search-elements_test.ts",
26+
"sidebar-event-listeners_test.ts",
2627
"style-pane-properties_test.ts",
2728
"switch-panels-while-editing-as-html_test.ts",
2829
"violating-element-and-attributes_test.ts",
Lines changed: 160 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,160 @@
1+
// Copyright 2024 The Chromium Authors. All rights reserved.
2+
// Use of this source code is governed by a BSD-style license that can be
3+
// found in the LICENSE file.
4+
5+
import {assert} from 'chai';
6+
import type * as puppeteer from 'puppeteer-core';
7+
8+
import {
9+
waitForContentOfSelectedElementsNode,
10+
waitForElementsStyleSection,
11+
waitForSelectedNodeToBeExpanded,
12+
} from '../../e2e/helpers/elements-helpers.js';
13+
import {clickMoreTabsButton} from '../../shared/helper.js';
14+
import type {DevToolsPage} from '../shared/frontend-helper.js';
15+
import type {InspectedPage} from '../shared/target-helper.js';
16+
17+
describe('Event listeners in the elements sidebar', () => {
18+
const loadEventListenersAndSelectButtonNode = async (devToolsPage: DevToolsPage, inspectedPage: InspectedPage) => {
19+
await inspectedPage.goToResource('elements/sidebar-event-listeners.html');
20+
await waitForElementsStyleSection(undefined, devToolsPage);
21+
22+
// Wait for element to be expanded
23+
await waitForSelectedNodeToBeExpanded(devToolsPage);
24+
25+
// Select the button that has the events and make sure it's selected
26+
await devToolsPage.page.keyboard.press('ArrowRight');
27+
await waitForContentOfSelectedElementsNode(
28+
'<button id=\u200B"test-button">\u200Bhello world\u200B</button>\u200B', devToolsPage);
29+
};
30+
31+
const EVENT_LISTENERS_PANEL_LINK = '[aria-label="Event Listeners"]';
32+
/* We add :not(.hidden) here as if you create an event listener + remove it via the UI
33+
* it gets the class of .hidden rather than being removed
34+
*/
35+
const EVENT_LISTENERS_SELECTOR = '[aria-label$="event listener"]:not(.hidden)';
36+
37+
const getDisplayedEventListenerNames = async (devToolsPage: DevToolsPage) => {
38+
const eventListeners = await devToolsPage.$$(EVENT_LISTENERS_SELECTOR);
39+
const eventListenerNames = await Promise.all(
40+
eventListeners.map((listener: puppeteer.JSHandle) => listener.evaluate(l => (l as Element).textContent)));
41+
return eventListenerNames as string[];
42+
};
43+
44+
const getEventListenerProperties = async (devToolsPage: DevToolsPage, selector: string) => {
45+
const clickEventProperties = await devToolsPage.$$(selector);
46+
47+
const propertiesOutput = await Promise.all(clickEventProperties.map(n => n.evaluate((node: Element) => {
48+
const nameNode = node.querySelector('.name');
49+
const valueNode = node.querySelector('.value');
50+
51+
if (!nameNode || !valueNode) {
52+
throw new Error('Could not find a name and value node for event listener properties.');
53+
}
54+
55+
const key = nameNode.textContent;
56+
const value = valueNode.textContent;
57+
return [key, value];
58+
})));
59+
60+
return propertiesOutput as string[][];
61+
};
62+
63+
const getFirstNodeForEventListener = async (devToolsPage: DevToolsPage, listenerTypeSelector: string) => {
64+
await devToolsPage.click(listenerTypeSelector);
65+
66+
const listenerNodesSelector = `${listenerTypeSelector} + ol>li`;
67+
const firstListenerNode = await devToolsPage.waitFor(listenerNodesSelector);
68+
if (!firstListenerNode) {
69+
throw new Error(`Could not find listener node for selector ${listenerNodesSelector}`);
70+
}
71+
const firstListenerText = await firstListenerNode.evaluate((node: Element) => {
72+
return node.textContent || '';
73+
});
74+
75+
return {
76+
firstListenerText,
77+
listenerSelector: listenerNodesSelector,
78+
};
79+
};
80+
81+
const openEventListenersPaneAndWaitForListeners = async (devToolsPage: DevToolsPage) => {
82+
let eventListenersPanel = await devToolsPage.$('Event Listeners', undefined, 'aria');
83+
if (!eventListenersPanel) {
84+
await clickMoreTabsButton(undefined, devToolsPage);
85+
eventListenersPanel = await devToolsPage.waitFor(EVENT_LISTENERS_PANEL_LINK);
86+
}
87+
await devToolsPage.clickElement(eventListenersPanel);
88+
await devToolsPage.waitFor(EVENT_LISTENERS_SELECTOR);
89+
};
90+
91+
it('lists the active event listeners on the page', async ({devToolsPage, inspectedPage}) => {
92+
await loadEventListenersAndSelectButtonNode(devToolsPage, inspectedPage);
93+
await openEventListenersPaneAndWaitForListeners(devToolsPage);
94+
95+
const eventListenerNames = await getDisplayedEventListenerNames(devToolsPage);
96+
assert.deepEqual(eventListenerNames, ['click', 'custom event', 'hover']);
97+
});
98+
99+
it('shows the event listener properties when expanding it', async ({devToolsPage, inspectedPage}) => {
100+
await loadEventListenersAndSelectButtonNode(devToolsPage, inspectedPage);
101+
await openEventListenersPaneAndWaitForListeners(devToolsPage);
102+
const {
103+
firstListenerText,
104+
listenerSelector,
105+
} = await getFirstNodeForEventListener(devToolsPage, '[aria-label="click, event listener"]');
106+
107+
// check that we have the right event for the right element
108+
// we can't use assert.strictEqual() as the text also includes the "Remove" button
109+
assert.include(firstListenerText, 'button#test-button');
110+
111+
// we have to use keyboard navigation here to expand
112+
// the event, as single click reveals it in the elements
113+
// tree and double click triggers the "Remove" button on
114+
// some platforms.
115+
await devToolsPage.page.keyboard.press('ArrowRight'); // select
116+
await devToolsPage.page.keyboard.press('ArrowRight'); // expand
117+
await devToolsPage.waitFor(`${listenerSelector}[aria-expanded="true"]`);
118+
119+
const clickEventPropertiesSelector = `${listenerSelector} + ol .name-and-value`;
120+
const propertiesOutput = await getEventListenerProperties(devToolsPage, clickEventPropertiesSelector);
121+
122+
assert.deepEqual(propertiesOutput, [
123+
['useCapture', 'false'],
124+
['passive', 'false'],
125+
['once', 'false'],
126+
['handler', '() => {}'],
127+
]);
128+
});
129+
130+
it('shows custom event listeners and their properties correctly', async ({devToolsPage, inspectedPage}) => {
131+
await loadEventListenersAndSelectButtonNode(devToolsPage, inspectedPage);
132+
await openEventListenersPaneAndWaitForListeners(devToolsPage);
133+
const {
134+
firstListenerText,
135+
listenerSelector,
136+
} = await getFirstNodeForEventListener(devToolsPage, '[aria-label="custom event, event listener"]');
137+
138+
// check that we have the right event for the right element
139+
// we can't use assert.strictEqual() as the text also includes the "Remove" button
140+
assert.include(firstListenerText, 'body');
141+
142+
// we have to use keyboard navigation here to expand
143+
// the event, as single click reveals it in the elements
144+
// tree and double click triggers the "Remove" button on
145+
// some platforms.
146+
await devToolsPage.page.keyboard.press('ArrowRight'); // select
147+
await devToolsPage.page.keyboard.press('ArrowRight'); // expand
148+
await devToolsPage.waitFor(`${listenerSelector}[aria-expanded="true"]`);
149+
150+
const customEventProperties = `${listenerSelector} + ol .name-and-value`;
151+
const propertiesOutput = await getEventListenerProperties(devToolsPage, customEventProperties);
152+
153+
assert.deepEqual(propertiesOutput, [
154+
['useCapture', 'true'],
155+
['passive', 'false'],
156+
['once', 'true'],
157+
['handler', '() => console.log(\'test\')'],
158+
]);
159+
});
160+
});

0 commit comments

Comments
 (0)