Email change test improvements#411
Conversation
| # And the email address of user "user1" should be "ldapuser@oc.com" | ||
| And the email address of user "user1" should be "apiuser@example.com" | ||
| And the LDAP users are resynced | ||
| And the email address of user "user1" should be "ldapuser@oc.com" |
There was a problem hiding this comment.
was it broken?
the admin still can overwrite the setting owncloud/core#33186 but when synced the correct LDAP setting will come back
There was a problem hiding this comment.
No, just asked as it was commented before.
skshetry
left a comment
skshetry
left a comment
There was a problem hiding this comment.
LGTM, just wanted a clarification.
|
drone apiAuth job was cancelled after running for 3 hours. |
Codecov Report
@@ Coverage Diff @@
## master #411 +/- ##
=========================================
Coverage 34.94% 34.94%
Complexity 1327 1327
=========================================
Files 31 31
Lines 3780 3780
=========================================
Hits 1321 1321
Misses 2459 2459Continue to review full report at Codecov.
|
1 similar comment
Codecov Report
@@ Coverage Diff @@
## master #411 +/- ##
=========================================
Coverage 34.94% 34.94%
Complexity 1327 1327
=========================================
Files 31 31
Lines 3780 3780
=========================================
Hits 1321 1321
Misses 2459 2459Continue to review full report at Codecov.
|
4 participants
small improvements to the testing code