[stable19] do not remove valid group shares

[backportbot-nextcloud]

backport of #25218

[faily-bot]

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 1804: failure

mariadb10.4-php7.3

Show full log

There were 5 warnings:

1) Test\DB\QueryBuilder\QueryBuilderTest::testExecuteWithParameterTooLarge
Cannot stub or mock class or interface "Doctrine\DBAL\Result" which does not exist

2) Test\DB\QueryBuilder\QueryBuilderTest::testExecuteWithParametersTooMany
Cannot stub or mock class or interface "Doctrine\DBAL\Result" which does not exist

3) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

4) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

5) OCA\Files_Sharing\Tests\External\ManagerTest::testAddShare
Cannot stub or mock class or interface "OCA\Files_Sharing\Tests\External\IManager" which does not exist

--

There was 1 failure:

1) Test\User\DatabaseTest::testSearch
Failed asserting that actual size 3 matches expected size 2.

/drone/src/tests/lib/User/Backend.php:114
/drone/src/tests/lib/User/DatabaseTest.php:125

mysql5.6-php7.2

Show full log

There were 5 warnings:

1) Test\DB\QueryBuilder\QueryBuilderTest::testExecuteWithParameterTooLarge
Cannot stub or mock class or interface "Doctrine\DBAL\Result" which does not exist

2) Test\DB\QueryBuilder\QueryBuilderTest::testExecuteWithParametersTooMany
Cannot stub or mock class or interface "Doctrine\DBAL\Result" which does not exist

3) Test\Files\ViewTest::testRenameFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

4) Test\Files\ViewTest::testCopyFailDeleteTargetKeepSource
Trying to configure method "writeStream" which cannot be configured because it does not exist, has not been specified, is final, or is static

5) OCA\Files_Sharing\Tests\External\ManagerTest::testAddShare
Cannot stub or mock class or interface "OCA\Files_Sharing\Tests\External\IManager" which does not exist

--

There was 1 failure:

1) Test\Share20\DefaultShareProviderTest::testDeleteUser with data set #1 (0, 'a', 'b', 'c', 'b', false)
Failed asserting that actual size 0 matches expected size 1.

/drone/src/tests/lib/Share20/DefaultShareProviderTest.php:2272