Skip to content

Conversation

nmammadli
Copy link
Contributor

No description provided.

@CLAassistant
Copy link

CLAassistant commented Dec 8, 2022

CLA assistant check
All committers have signed the CLA.

Selector(`[data-testid="node-item_${folders[0]}:${folders[1]}:keys:keys:"]`).visible)
.ok("Folder is not selected");

});
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't see these tests:

  1. "The folder without any patterns selected and the list of keys is displayed when there is a folder without any patterns"
    image
  2. The folder without any namespaces is selected (if exists) when folder does not exist after search/filter
    image
  3. The first folder with namespaces is expanded and selected when folder and folder without any namespaces does not exist after search/filter
    (it is when one folder is opened and you search for key from other folder - other folder should be opened after such search)
  4. The previously selected folder is preselected when key does not exist after keys refresh (you need to select one folder, then delete any key from this folder by CLI and click refresh - verify that the same folder still selected)

@nmammadli nmammadli changed the base branch from main to feature/RI-3355_Tree_view_improvments December 13, 2022 11:36
@nmammadli nmammadli changed the base branch from feature/RI-3355_Tree_view_improvments to main December 13, 2022 11:37
@nmammadli nmammadli changed the base branch from main to feature/RI-3355_Tree_view_improvments December 13, 2022 12:12
@vlad-dargel vlad-dargel changed the base branch from feature/RI-3355_Tree_view_improvments to main December 13, 2022 12:16
@vlad-dargel vlad-dargel changed the base branch from main to feature/RI-3355_Tree_view_improvments December 13, 2022 12:17
Copy link
Contributor

@vlad-dargel vlad-dargel left a comment

Choose a reason for hiding this comment

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

Looks good
just need to map all verifications (.expect and verifyKeysDisplayedInTheList) with tests from checklist.
you can use names both from Scenario and Checklist column
image

@nmammadli nmammadli force-pushed the e2e/feature/RI-3892_treeViewImprovements branch from b496361 to dbe0380 Compare December 14, 2022 07:45
@nmammadli nmammadli merged commit 7041463 into feature/RI-3355_Tree_view_improvments Dec 14, 2022
@nmammadli nmammadli deleted the e2e/feature/RI-3892_treeViewImprovements branch December 14, 2022 08:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants