jomarko commented on code in PR #2239:
URL:
https://github.com/apache/incubator-kie-tools/pull/2239#discussion_r1571209430
##########
packages/dmn-editor/tests/e2e/__fixtures__/nodes.ts:
##########
@@ -91,7 +91,7 @@ export class Nodes {
targetPosition: { x: number; y: number };
thenRenameTo?: string;
}) {
- await this.hover({ name: args.from, position: NodePosition.TOP });
+ await this.select({ name: args.from, position: NodePosition.TOP });
Review Comment:
@ljmotta hi, this is needed because of the **Decision Service** nodes. For
those, `hover` is not enough to display their contextual menus. The fixture
`dragNewConnectedNode` worked for **Decision Service** nodes tests until now,
just because we have a pseudocode
```
addDecisionServiceNode() // this caused Decision service node is selected
and its contextual menu is visible
dragNewConnectedNode() // worked , however its internal `hover` had no effect
```
--
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.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
