[stable10] UI tests for reset password

[phil-davis]

Backport #31168

[phil-davis]

This is a rebased version of PR #31192
I had trouble messing with that backport PR from @paurakhsharma so I made a new PR.

This backport had been stuck waiting for #31144 to be merged.

And other backports are waiting for this to get merged - e.g. backport for #31220 which will quickly remove all this EmailContext stuff again.

It is all easier if the backports happen in order - no conflict mess.

[codecov]

Codecov Report

Merging #31225 into stable10 will not change coverage.
The diff coverage is n/a.

@@             Coverage Diff              @@
##             stable10   #31225    +/-   ##
============================================
  Coverage       60.24%   60.24%            
+ Complexity      18352    18189   -163     
============================================
  Files            1194     1194            
  Lines           71781    71781            
  Branches         1248     1248            
============================================
  Hits            43241    43241            
  Misses          28170    28170            
  Partials          370      370

Flag	Coverage Δ	Complexity Δ
#javascript	52.64% <ø> (ø)	0 <ø> (ø)	⬇️
#phpunit	61.07% <ø> (ø)	18189 <ø> (-163)	⬇️

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 37991c3...c2c6802. Read the comment docs.

[phil-davis]

I had to add USE_EMAIL: true to webUIManageUsersGroups because it has resetPassword.feature. And webUIManageUsersGroups is no longer in core master - so it is always a bit of a learning experience backporting webUI infrastructure changes from core to stable10

[patrickjahns]

I had to add USE_EMAIL: true to webUIManageUsersGroups because it has resetPassword.feature (and webUIManageUsersGroups is no longer in core master - so it is always a bit of a learning experience backporting webUI infrastructure changes from core to stable10

@DeepDiver1975
that is what i mean by not diverging from the contents of the branches too much

[phil-davis]

At some point it is life - core master has to get new features.
In this case features were taken out of core master. But again, that is life. At some time we have to make progress reducing core.

[phil-davis]

This works. Now we are stuck because of the Redis problem, and that Travis has these huge cron jobs running. So PR #31229 is waiting... for a few short Travis jobs to get run.

[phil-davis]

Rebased again - the Redis fixes are merged in stable10. I expect CI to pass here now.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.