Skip to content
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

[Engineering] Lower time for botbuilder-dialogs tests #2613

Conversation

denscollo
Copy link
Contributor

Addresses #2338

Description

This PR optimizes the time for the botbuilder-dialogs tests having no impact in the code coverage numbers and reducing the execution time approximately from 4s to 3s.

Specific Changes

choices_channel.test.js

  • Removed duplicated tests should return true for supportsSuggestedActions() with skype and 10 and should return false for supportsSuggestedActions() with skype and 11.
  • Fixed eslint warnings.

choices_choiceFactory.test.js

  • Created a variable called expectedActivity for Teams and Cortana tests.
  • Fixed eslint warnings.

dialogSet.test.js

  • Moved some ConversationState instances at the top of the describe to reuse the same instance.
  • Fixed eslint warnings.

memory_dialogStateManager.test.js

  • Moved some createConfiguredTestDc function calls at the top of the describe to reuse the same DialogContext instance.
  • Removed duplicated test Should raise an error if getValue() called with an invalid scope..
  • Fixed eslint warnings.

Testing

image

@denscollo denscollo requested a review from a team as a code owner July 29, 2020 22:24
@denscollo denscollo requested a review from stevengum July 29, 2020 22:24
@stevengum stevengum requested a review from EricDahlvang July 30, 2020 07:39
@stevengum stevengum merged commit 91f80d2 into microsoft:master Jul 31, 2020
@denscollo denscollo deleted the southworks/update/botbuilder-dialogs-tests branch July 31, 2020 20:22
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