Skip to content

Commit 256bcc8

Browse files
szuendDevtools-frontend LUCI CQ
authored andcommitted
[e2e] Port 'show all levels' console filter test
R=alexrudenko@chromium.org Fixed: 416522177 Change-Id: Ia6eb90136d8f646e20b9fe02c85e63078541f006 Reviewed-on: https://chromium-review.googlesource.com/c/devtools/devtools-frontend/+/6613683 Reviewed-by: Alex Rudenko <alexrudenko@chromium.org> Commit-Queue: Simon Zünd <szuend@chromium.org>
1 parent a5eb929 commit 256bcc8

3 files changed

Lines changed: 44 additions & 42 deletions

File tree

test/e2e/console/console-filter_test.ts

Lines changed: 1 addition & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -5,14 +5,13 @@
55
import {assert} from 'chai';
66
import type * as puppeteer from 'puppeteer-core';
77

8-
import {$, getBrowserAndPages, step} from '../../shared/helper.js';
8+
import {getBrowserAndPages, step} from '../../shared/helper.js';
99
import {
1010
CONSOLE_MESSAGE_WRAPPER_SELECTOR,
1111
deleteConsoleMessagesFilter,
1212
filterConsoleMessages,
1313
getConsoleMessages,
1414
getCurrentConsoleMessages,
15-
showVerboseMessages,
1615
toggleShowCorsErrors,
1716
waitForConsoleMessagesToBeNonEmpty,
1817
} from '../helpers/console-helpers.js';
@@ -88,42 +87,6 @@ describe('The Console Tab', () => {
8887
});
8988
});
9089

91-
it('shows messages from all levels', async () => {
92-
let messages: string[];
93-
const withAnchor = true;
94-
await step('navigate to console-filter.html and get console messages', async () => {
95-
messages = await getConsoleMessages('console-filter', withAnchor, showVerboseMessages);
96-
});
97-
98-
await step('ensure that all levels are logged', async () => {
99-
const allLevelsDropdown = await $('[aria-label^="Log level: All levels"]');
100-
assert.isNotNull(allLevelsDropdown);
101-
});
102-
103-
await step('check that all console messages appear', async () => {
104-
assert.deepEqual(messages, [
105-
'console-filter.html:10 1topGroup: log1()',
106-
'log-source.js:6 2topGroup: log2()',
107-
'console-filter.html:10 3topGroup: log1()',
108-
'console-filter.html:17 enter outerGroup',
109-
'console-filter.html:10 1outerGroup: log1()',
110-
'log-source.js:6 2outerGroup: log2()',
111-
'console-filter.html:21 enter innerGroup1',
112-
'console-filter.html:10 1innerGroup1: log1()',
113-
'log-source.js:6 2innerGroup1: log2()',
114-
'console-filter.html:26 enter innerGroup2',
115-
'console-filter.html:10 1innerGroup2: log1()',
116-
'log-source.js:6 2innerGroup2: log2()',
117-
'console-filter.html:10 4topGroup: log1()',
118-
'log-source.js:6 5topGroup: log2()',
119-
'console-filter.html:38 Hello 1',
120-
'console-filter.html:39 Hello 2',
121-
'console-filter.html:41 verbose debug message',
122-
'console-filter.html:42 end',
123-
]);
124-
});
125-
});
126-
12790
it('can exclude messages from a source url', async () => {
12891
const {frontend} = getBrowserAndPages();
12992
let sourceUrls: string[];

test/e2e/helpers/console-helpers.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -230,9 +230,9 @@ export async function focusConsolePrompt() {
230230
await timeout(50);
231231
}
232232

233-
export async function showVerboseMessages() {
234-
await click(LOG_LEVELS_SELECTOR);
235-
await click(LOG_LEVELS_VERBOSE_OPTION_SELECTOR);
233+
export async function showVerboseMessages(devToolsPage = getBrowserAndPagesWrappers().devToolsPage) {
234+
await devToolsPage.click(LOG_LEVELS_SELECTOR);
235+
await devToolsPage.click(LOG_LEVELS_VERBOSE_OPTION_SELECTOR);
236236
await expectVeEvents(
237237
[
238238
veClick(''),
@@ -248,7 +248,7 @@ export async function showVerboseMessages() {
248248
veClick('Menu > Toggle: verbose'),
249249
veResize('Menu'),
250250
],
251-
`${await veRoot()} > Toolbar > DropDown: log-level`);
251+
`${await veRoot(devToolsPage)} > Toolbar > DropDown: log-level`, devToolsPage);
252252
}
253253

254254
export async function typeIntoConsole(message: string) {

test/e2e_non_hosted/console/filter_test.ts

Lines changed: 39 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -2,14 +2,53 @@
22
// Use of this source code is governed by a BSD-style license that can be
33
// found in the LICENSE file.
44

5+
import {assert} from 'chai';
6+
57
import {
8+
CONSOLE_MESSAGE_TEXT_AND_ANCHOR_SELECTOR,
69
deleteConsoleMessagesFilter,
710
filterConsoleMessages,
811
navigateToConsoleTab,
12+
showVerboseMessages,
913
waitForConsoleMessagesToBeNonEmpty
1014
} from '../../e2e/helpers/console-helpers.js';
1115

1216
describe('The Console Tab', () => {
17+
it('shows messages from all levels', async ({devToolsPage, inspectedPage}) => {
18+
await Promise.all([
19+
inspectedPage.goToResource('console/console-filter.html'),
20+
navigateToConsoleTab(devToolsPage),
21+
]);
22+
23+
await showVerboseMessages(devToolsPage);
24+
await waitForConsoleMessagesToBeNonEmpty(18, devToolsPage);
25+
26+
const actualMessages = await devToolsPage.evaluate(
27+
selector => Array.from(document.querySelectorAll(selector)).map(e => e.textContent ?? ''),
28+
CONSOLE_MESSAGE_TEXT_AND_ANCHOR_SELECTOR);
29+
30+
assert.deepEqual(actualMessages, [
31+
'console-filter.html:10 1topGroup: log1()',
32+
'log-source.js:6 2topGroup: log2()',
33+
'console-filter.html:10 3topGroup: log1()',
34+
'console-filter.html:17 enter outerGroup',
35+
'console-filter.html:10 1outerGroup: log1()',
36+
'log-source.js:6 2outerGroup: log2()',
37+
'console-filter.html:21 enter innerGroup1',
38+
'console-filter.html:10 1innerGroup1: log1()',
39+
'log-source.js:6 2innerGroup1: log2()',
40+
'console-filter.html:26 enter innerGroup2',
41+
'console-filter.html:10 1innerGroup2: log1()',
42+
'log-source.js:6 2innerGroup2: log2()',
43+
'console-filter.html:10 4topGroup: log1()',
44+
'log-source.js:6 5topGroup: log2()',
45+
'console-filter.html:38 Hello 1',
46+
'console-filter.html:39 Hello 2',
47+
'console-filter.html:41 verbose debug message',
48+
'console-filter.html:42 end',
49+
]);
50+
});
51+
1352
it('resets the filter', async ({devToolsPage, inspectedPage}) => {
1453
await Promise.all([
1554
inspectedPage.goToResource('console/console-filter.html'),

0 commit comments

Comments
 (0)