vogievetsky commented on a change in pull request #10469:
URL: https://github.com/apache/druid/pull/10469#discussion_r499157234
##########
File path: web-console/e2e-tests/component/datasources/overview.ts
##########
@@ -116,6 +124,23 @@ export class DatasourcesOverview {
const editActions = await this.page.$$('span[icon=wrench]');
editActions[index].click();
+ await this.waitForPopupMenu();
+ }
+
+ private async waitForPopupMenu(): Promise<void> {
+ await this.page.waitFor(1000); //
https://blueprintjs.com/docs/#core/components/popover.testing
Review comment:
Do you think that we should make an allowance in the UI and have a
'testing' mode that zeroes out all the delays in all the popovers in the UI
`hoverCloseDelay={0} hoverOpenDelay={0}`. I am imagining that you could
evaluate some sort of `enterTestingMode()` function in the app context and it
will do this and other things. Not for this PR but wanted to get your thoughts
on it.
----------------------------------------------------------------
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
