Fix testing app for oracle#23
Conversation
…ke the query is anyhow no longer needed
| $userManager->clearBackends(); | ||
| $userManager->registerBackend($c->query(AlternativeHomeUserBackend::class)); | ||
|
|
||
| $userManager->listen('\OC\User', 'postCreateUser', function ($user, $password) use ($c) { |
There was a problem hiding this comment.
if this is only for fixing Oracle, I don't see why the code has been moved outside of the "postCreateUser" hook instead of just changing the literal
There was a problem hiding this comment.
That was from TM patch
What's the intention of this postCreateUser here?
There was a problem hiding this comment.
I think I added this back then to delay updating the accounts table to after the admin user got created, because in some cases it would happen too early. Not sure if this time we got lucky or other core changes would remove the need for this.
If this doesn't break acceptance tests on core then I'm fine keeping it this way.
There was a problem hiding this comment.
https://drone.owncloud.com/owncloud/core/10273 - acceptance tests are all green
|
I've updated "latest" |
3 participants
Description
Fix running acceptance+ui tests on drone
Taken from owncloud/core#31145
Checklist: