-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Acceptance test for group names with emoji #35247
Conversation
@@ -44,6 +44,8 @@ Feature: delete groups | |||
| 50%2Eagle | %2E literal looks like an escaped "." | | |||
| 50%2Fix | %2F literal looks like an escaped slash | | |||
| staff?group | Question mark | | |||
| 😁 😂 | emoji | | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
extra line
@@ -22,7 +22,7 @@ Feature: remove a user from a group | |||
| España | special European characters | | |||
| नेपाली | Unicode group name | | |||
|
|||
Scenario Outline: admin removes a user from a group | |||
Scenario Outline: admin removes a user from a group |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
incorrectly indented
Codecov Report
@@ Coverage Diff @@
## master #35247 +/- ##
=========================================
Coverage 65.54% 65.54%
Complexity 18647 18647
=========================================
Files 1218 1218
Lines 70546 70546
Branches 1288 1288
=========================================
Hits 46236 46236
Misses 23933 23933
Partials 377 377
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #35247 +/- ##
=========================================
Coverage 65.54% 65.54%
Complexity 18647 18647
=========================================
Files 1218 1218
Lines 70546 70546
Branches 1288 1288
=========================================
Hits 46236 46236
Misses 23933 23933
Partials 377 377
Continue to review full report at Codecov.
|
restarted build |
@dpakach @paurakhsharma @skshetry please review again |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 👍
@bhawanaprasain please backport to stable10 branch |
Backport |
Using these tests we can check if we can use group names with emojis .
Part of issue #34437