Skip to content

Conversation

vlad-dargel
Copy link
Contributor

@vlad-dargel vlad-dargel commented Sep 6, 2022

fixes for tests:

  • Verify that user can clone Sentinel
  • Verify that when manager publishes new notification, it appears in the app
  • Verify that user can open notification center by clicking on icon and see all received messages
  • Verify that when user see the toggle is OFF - Profiler logs are not being saved
  • Disabled Editor Cleanup toggle behavior
  • Enabled Editor Cleanup toggle behavior
  • Editor Cleanup settings
  • Verify that user can see the text in settings for pipeline with link
  • Verify that user can enter only numbers >0 in "Commands in pipeline" input
  • Verify that user can interact with the Editor while command(s) in progress
  • Verify that command results are added to history in order most recent - on top
  • Verify that user can run multiple commands written in multiple lines in Workbench page
  • Verify that user can use double slashes (//) wrapped in double quotes and these slashes will not comment out any characters

returned to skipped/desktop:

  • Verify that user can edit and run automatically added "Search" script in Workbench and see the results
  • Verify that user can switches between Table and Text for FT.INFO and see results corresponding to their views
  • Verify that user can switches between Table and Text for FT.SEARCH and see results corresponding to their views
  • Verify that user can switches between Table and Text for FT.AGGREGATE and see results corresponding to their views
  • Verify that user can open notification center by clicking on icon and see all received messages

@vlad-dargel vlad-dargel requested a review from AlenaSY September 6, 2022 11:12
@vlad-dargel vlad-dargel self-assigned this Sep 6, 2022
@vlad-dargel vlad-dargel changed the title fixes for profiler skipped test regression fixes and fix for profiler skipped test Sep 6, 2022
});
// Skipped due to testCafe issue https://github.com/DevExpress/testcafe/issues/5574
test.skip
test.only
Copy link
Contributor

Choose a reason for hiding this comment

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

Do you need .only here?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fixed, thanks

@vlad-dargel vlad-dargel requested a review from AlenaSY September 7, 2022 11:59
@AlenaSY AlenaSY merged commit 6b8ce15 into main Sep 7, 2022
@AlenaSY AlenaSY deleted the e2e/bugfix/downloads-fix-profiler branch September 7, 2022 13:10
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.

2 participants