Testing md5home

[PVince81]

Description

Allow clearing default user backends in config.php.
Testing app provides test user backend for alternative homes.

Related Issue

Alternative to #26846 and #26851

Motivation and Context

Need tests for alternative getHome

How Has This Been Tested?

1. Edit config.php and add:

'user_backends' => 
  array (
    'default' => false,   // this is the new setting that kills the default DB user backend
    0 => 
    array (
      'class' => '\\OCA\\Testing\\AlternativeHomeUserBackend',
      'arguments' => 
      array (
      ),
    ),
  ),

Screenshots (if appropriate):

Types of changes

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• My code follows the code style of this project.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• All new and existing tests passed.

@DeepDiver1975 as discussed
@SergioBertolinSG

If approve, I'll add an automated test suite that uses this in #26846

[mention-bot]

@PVince81, thanks for your PR! By analyzing the history of the files in this pull request, we identified @DeepDiver1975, @bantu and @xoen to be potential reviewers.

[SergioBertolinSG]

Admin user account stops working with this set up.

This is my data folder after having admin, user1, user2 and user3 logged in at least once:

21232f297a57a5a743894a0e4a801fc3  7e58d63b60197ceb55a1c487989a3720  admin    files_external  owncloud.log
24c9e15e52afc47c225b757e7bee1f9d  92877af70a45fd6a2ed7fe81e1236b78  avatars  index.html

Admin user cannot upload files or access to his files.

[PVince81]

Admin user is not relevant for integration tests, at least not the filesystem ones. FS tests run as newly created users which would get the correct home.

If causing side effects I can add an exception for users called "admin" and return the normal home for that one.

[butonic]

ohhh nice!

[PVince81]

@butonic hold your horses, this is only for the testing app 😉
But nice indeed.

[PVince81]

@SergioBertolinSG please retry, I excluded "admin" here 79d1ed0

[PVince81]

All tests passed now, merge ?

[SergioBertolinSG]

Works fine 👍

[PVince81]

We need to backport this if we want to benefit from automated testing... Agreed ? @DeepDiver1975 @SergioBertolinSG

[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.