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

Added test cases of Zoom settings slash command and edit meeting message. #125

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

AayushChaudhary0001
Copy link
Contributor

This PR consists of the test cases for the Zoom settings slash command and editing the message that is presented to join the meeting.

PR for which test cases are added:

* [MI-3689]: Test cases of Zoom PR #284 and #287

* Review Fixes

Co-authored-by: Raghav Aggarwal <[email protected]>

* Review fixes

Co-authored-by: Raghav Aggarwal <[email protected]>

* [MI-3689]: Review fixes of changing the title

* [MI-3689]: Review fixes of slash command

* Review Fixes

Co-authored-by: Raghav Aggarwal <[email protected]>

---------

Co-authored-by: Raghav Aggarwal <[email protected]>
@lindalumitchell lindalumitchell added the 2: QA Review Requires review by a QA tester label Apr 30, 2024
@DHaussermann
Copy link
Contributor

Thanks @arush-vashishtha this looks accurate to me.
Are there steps still needed to resolve with your other Zoom PR that runs through the various settings?

The script is fine. I will approve for merge once I confirm if we need any further validation of the Zephyr test data.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
2: QA Review Requires review by a QA tester
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants