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 the test cases for the context menu features and message access request in to-do list. #158

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

Conversation

arush-vashishtha
Copy link
Contributor

Summary

This PR consists the test cases for the following scenarios,

  • The feature to create Gitlab issue from context menu.
  • The feature to attach message to Gitlab issue.
  • The format message access request notification in the to-do list.

arush-vashishtha and others added 2 commits January 16, 2024 18:57
…ge access request in to-do list. (#32)

* [MM-70]:Added the test cases for PR 404.

* added the test cases for create gitlab issue

* [MM-70]: added the test cases for the message attach to gitlab issue.

* Fixed the comments by Aayush.

* Review fixes by Raghav

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

* Review fixes by Raghav

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

---------

Co-authored-by: Raghav Aggarwal <[email protected]>
@arush-vashishtha
Copy link
Contributor Author

@DHaussermann @saturninoabril Can you please have a look at this?

@lindalumitchell lindalumitchell added the 2: QA Review Requires review by a QA tester label Apr 30, 2024
@DHaussermann DHaussermann self-requested a review May 3, 2024 16:52
@DHaussermann
Copy link
Contributor

This looks good @arush-vashishtha. The script looks accurate to me.

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

Copy link
Member

@saturninoabril saturninoabril left a comment

Choose a reason for hiding this comment

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

Thanks @arush-vashishtha, overall looks good except for few comments.

name: "Add comment to a gitlab issue from the `Attach to GitLab Issue` option in the context menu on MM."
status: Active
priority: Normal
folder: Context-menu-feature
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
folder: Context-menu-feature
folder: Context menu feature

@@ -0,0 +1,43 @@
---
# (Required) Ensure all values are filled up
name: "Add comment to a gitlab issue from the `Attach to GitLab Issue` option in the context menu on MM."
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
name: "Add comment to a gitlab issue from the `Attach to GitLab Issue` option in the context menu on MM."
name: "Add comment to a GitLab issue from the `Attach to GitLab Issue` option in the context menu"

@@ -0,0 +1,49 @@
---
# (Required) Ensure all values are filled up
name: "Create a Gitlab issue from the `Create Gitlab Issue` in the context menu on MM."
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
name: "Create a Gitlab issue from the `Create Gitlab Issue` in the context menu on MM."
name: "Create a Gitlab issue from the `Create Gitlab Issue` in the context menu"

name: "Create a Gitlab issue from the `Create Gitlab Issue` in the context menu on MM."
status: Active
priority: Normal
folder: Context-menu-feature
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
folder: Context-menu-feature
folder: Context menu feature

Comment on lines +46 to +49
**Expected**

The issue should be created for the selected project on Gitlab.
After step 2, the dropdown menu for additional options should get hidden upon removing the project.
Copy link
Member

Choose a reason for hiding this comment

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

Please arrange the format to follow steps/expected as commented here - #161 (review)

name: "Member access request in the todo list after entering the `/gitlab todo` slash command."
status: Active
priority: Normal
folder: Subscriptions-and-notifications
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
folder: Subscriptions-and-notifications
folder: Subscriptions and Notifications


**Step 1**

1. Create a `member access request` from a different user to any one of your desired public project/group on gitlab.
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
1. Create a `member access request` from a different user to any one of your desired public project/group on gitlab.
1. Create a `member access request` from a different user to any one of your desired public project/group on GitLab.

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.

4 participants