Skip to content

kie-issues#1155: Add end-to-end tests for keyboard shortcuts for the new DMN Editor #3128

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 17 commits into
base: main
Choose a base branch
from

Conversation

ramkumar-hub
Copy link

Closes apache/incubator-kie-issues#1155

Setup Storybook
Added DevWebapp Story
Added Loan Pre Qualification Story
Setup Playwright
Added required fixtures for the End-To-End tests
Added End-To-End tests for the keyboard shortcuts
Setup containerization for the End-To-End tests

@ramkumar-hub ramkumar-hub requested a review from tiagobento as a code owner May 9, 2025 06:34
@yesamer yesamer changed the title Kie issues#1155 e2e test keyboard shortcuts kie-issues#1155: Add end-to-end tests for keyboard shortcuts for the new DMN Editor May 9, 2025
Copy link
Contributor

@jomarko jomarko left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for the PR @ramkumar-hub . I put some comments inline regarding tests. Furthermore I am not sure about the strategy we chose for:

  • copying the existing __fixtures__ like creating a copy of nodes.ts
  • using same port as in dmn-editor so in dmn-editor-envelope - 9901

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Do we test node deletion or node name deletion?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Also here, same question, do we test node deletion or node name deletion?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

the screenshot name seems to be different than the screenshot content, is it expected?

  • name says all
  • while content has single node selected

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think I do not see properties in the screenshot content, could you please clarify if the screenshot is correct?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

could you please clarify if the screenshot is correct?

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test is to toggle between select and deselect all nodes using shortcut key- A

Select all nodes using shortcut - A
selected-all-nodes
Deselect all nodes using shortcut - A
unselected-all-nodes

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

comparing other screenshots, it sounds zoom level is same, could you please clarify?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

cold you please clarify if the screenshot is correct? it seems like 100% zoom level

Copy link
Contributor

@ljmotta ljmotta left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I didn't finish my review, but I believe you have changes other than the tests on this PR. Like on the dmn-editor and online-editor in the accelerators. Please check the commit history. I'll review again after the fix.

@ramkumar-hub ramkumar-hub force-pushed the kie-issues#1155-e2e-test-keyboard-shortcuts branch from 93d72ee to a8e5600 Compare May 14, 2025 19:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add end-to-end tests for keyboard shortcuts for the new DMN Editor
4 participants