-
Notifications
You must be signed in to change notification settings - Fork 180
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][full-ci] tests: add api tests for changing system language using graph api #7817
Conversation
05c7b6a
to
c3bdd1a
Compare
| language | subject | message | api | | ||
| de | Neue Freigabe | Alice Hansen hat textfile1.txt mit Ihnen geteilt | Graph | | ||
| de | Neue Freigabe | Alice Hansen hat textfile1.txt mit Ihnen geteilt | settings | | ||
| es | Recurso compartido | Alice Hansen compartió textfile1.txt contigo | Graph | | ||
| es | Recurso compartido | Alice Hansen compartió textfile1.txt contigo | settings | |
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.
@ScharfViktor I've refactored this like so ^ to include a few scenarios with settings API, if we don't need coverage for settings API I can remove the related code.
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.
fine, thanks. Since we have settings endpoint it's good to have a couple of tests for it
c3bdd1a
to
40b185e
Compare
| language | subject | message | api | | ||
| de | Neue Freigabe | Alice Hansen hat textfile1.txt mit Ihnen geteilt | Graph | | ||
| de | Neue Freigabe | Alice Hansen hat textfile1.txt mit Ihnen geteilt | settings | | ||
| es | Recurso compartido | Alice Hansen compartió textfile1.txt contigo | Graph | | ||
| es | Recurso compartido | Alice Hansen compartió textfile1.txt contigo | settings | |
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.
fine, thanks. Since we have settings endpoint it's good to have a couple of tests for it
40b185e
to
01f78ce
Compare
Kudos, SonarCloud Quality Gate passed! |
Related Issue
Fixes: #7764