-
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
[Tests-Only]Used json file to store translated messages #38412
Conversation
d930cbd
to
12d0eea
Compare
12d0eea
to
aee9ad4
Compare
| dav_version | language | | ||
| old | de-DE | | ||
| new | de-DE | | ||
| old | zh-CN | | ||
| new | zh-CN | | ||
| old | fr-FR | | ||
| new | fr-FR | | ||
| old | es-ES | | ||
| new | es-ES | |
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.
can we just have a data table rather than senario outline, that will make tests a lot faster
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.
can we? 🤔 Also, I wanted to reuse the logic from the step Then the OCS status message should be "<message>"
. So, I am not sure if it is a great idea. What do you think @phil-davis
0495378
to
84a2ba2
Compare
Kudos, SonarCloud Quality Gate passed! |
Description
generalizing API tests related to translations.
Related Issue
part of owncloud/QA#608
Motivation and Context
How Has This Been Tested?
Types of changes
Checklist: