Skip to content

Commit 74355a4

Browse files
astandrikAnton Standrik
and
Anton Standrik
authored
fix: stream test in safari (#2059)
Co-authored-by: Anton Standrik <[email protected]>
1 parent 21de77f commit 74355a4

File tree

1 file changed

+7
-4
lines changed

1 file changed

+7
-4
lines changed

tests/suites/tenant/queryEditor/queryEditor.test.ts

+7-4
Original file line numberDiff line numberDiff line change
@@ -145,11 +145,14 @@ test.describe('Test Query Editor', async () => {
145145

146146
test('Streaming query shows some results and banner when stop button is clicked', async ({
147147
page,
148-
browserName,
149148
}) => {
150-
// For some reason Safari handles large numbers list bad in Safari
151-
// Will be investigated here https://github.com/ydb-platform/ydb-embedded-ui/issues/1989
152-
test.skip(browserName === 'webkit', 'This test is skipped in Safari');
149+
// Safari in playwright has problem with painting an array
150+
// of million values for frequently appearing rows.
151+
// But still need them for heavy responses to simulate
152+
// long running queries. Setting their display to none resolves the issue.
153+
await page.addStyleTag({
154+
content: '.ydb-query-result-sets-viewer__result tr td:nth-child(3n) { display: none; }',
155+
});
153156
const queryEditor = new QueryEditor(page);
154157
await toggleExperiment(page, 'on', 'Query Streaming');
155158

0 commit comments

Comments
 (0)