Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Also include external share results in recommendations #24864

Merged
merged 1 commit into from
Dec 30, 2020

Conversation

juliushaertl
Copy link
Member

@juliushaertl juliushaertl commented Dec 28, 2020

Include external share results in recommendations if condition matches

The use case for this is that deck can add a default result which will always be shown in order to not mess the share result list with too many entries.

Signed-off-by: Julius Härtl <jus@bitgrid.net>
@juliushaertl juliushaertl force-pushed the bugfix/noid/always-show-external-share branch from 50044f0 to 9f5ada5 Compare December 30, 2020 09:34
@juliushaertl juliushaertl marked this pull request as ready for review December 30, 2020 09:34
@rullzer rullzer mentioned this pull request Dec 30, 2020
39 tasks
Copy link
Member

@rullzer rullzer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure...

@faily-bot
Copy link

faily-bot bot commented Dec 30, 2020

🤖 beep boop beep 🤖

Here are the logs for the failed build:

Status of 845: failure

nodb

Show full log
There were 5 errors:

1) OCA\DAV\Tests\Command\MoveCalendarTest::testMove
Error: Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found

/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:214

2) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithDestinationNotPartOfGroup with data set #1 (false)
Error: Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found

/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:269

3) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithDestinationPartOfGroup
Error: Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found

/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:312

4) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithDestinationNotPartOfGroupAndForce
Error: Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found

/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:358

5) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithCalendarAlreadySharedToDestination with data set #0 (true)
Error: Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found

/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:421

--

There were 11 warnings:

1) Test\AppFramework\Controller\AuthPublicShareControllerTest::testAuthenticateAuthenticated
Trying to configure method "isAuthenticated" which cannot be configured because it does not exist, has not been specified, is final, or is static

2) Test\Support\Subscription\RegistryTest::testDelegateIsHardUserLimitReachedWithoutSupportAppAndUserCount with data set #0 (35, 15, 2, false)
Cannot stub or mock class or interface "Test\Support\Subscription\UserInterface" which does not exist

3) Test\Support\Subscription\RegistryTest::testDelegateIsHardUserLimitReachedWithoutSupportAppAndUserCount with data set #1 (35, 45, 15, false)
Cannot stub or mock class or interface "Test\Support\Subscription\UserInterface" which does not exist

4) Test\Support\Subscription\RegistryTest::testDelegateIsHardUserLimitReachedWithoutSupportAppAndUserCount with data set #2 (35, 45, 5, true)
Cannot stub or mock class or interface "Test\Support\Subscription\UserInterface" which does not exist

5) Test\Support\Subscription\RegistryTest::testDelegateIsHardUserLimitReachedWithoutSupportAppAndUserCount with data set #3 (35, 45, 55, false)
Cannot stub or mock class or interface "Test\Support\Subscription\UserInterface" which does not exist

6) OCA\DAV\Tests\unit\CalDAV\CalendarTest::testConfidentialClassification with data set #0 (3, false)
No method rule is set

7) Warning
The data provider specified for ExpirationTest::testGetMaxAgeAsTimestamp is invalid.
PHPUnit\Util\Exception: Method timestampTestData does not exist

8) OCA\Settings\Tests\Mailer\NewUserMailHelperTest::testGenerateTemplateWithPasswordResetToken
Method getTime may not return value of type string, its return declaration is ": int"

9) OCA\TwoFactorBackupCodes\Tests\Unit\Listener\ActivityPublisherTest::testHandleCodesGeneratedEvent
Method publish may not return value of type Mock_IEvent_6ce04dd0, its return declaration is ": void"

10) OCA\UpdateNotification\Tests\Notification\BackgroundJobTest::testCreateNotifications with data set #1 ('app2', '1.0.1', '1.0.0', '1.0.0', true, array('user1'), array(array('user1')))
Method notify may not return value of type Mock_INotification_8f6ee36b, its return declaration is ": void"

11) OCA\UpdateNotification\Tests\Notification\BackgroundJobTest::testCreateNotifications with data set #2 ('app3', '1.0.1', false, false, true, array('user2', 'user3'), array(array('user2'), array('user3')))
Method notify may not return value of type Mock_INotification_8f6ee36b, its return declaration is ": void"

--

There were 6 failures:

1) OCA\DAV\Tests\Command\MoveCalendarTest::testWithBadUserOrigin with data set #0 (false, true)
Failed asserting that exception of type "Error" matches expected exception "InvalidArgumentException". Message was: "Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found" at
/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:121
.

2) OCA\DAV\Tests\Command\MoveCalendarTest::testWithBadUserOrigin with data set #1 (true, false)
Failed asserting that exception of type "Error" matches expected exception "InvalidArgumentException". Message was: "Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found" at
/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:121
.

3) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithInexistantCalendar
Failed asserting that exception of type "Error" matches expected exception "InvalidArgumentException". Message was: "Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found" at
/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:146
.

4) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithExistingDestinationCalendar
Failed asserting that exception of type "Error" matches expected exception "InvalidArgumentException". Message was: "Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found" at
/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:181
.

5) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithDestinationNotPartOfGroup with data set #0 (true)
Failed asserting that exception of type "Error" matches expected exception "InvalidArgumentException". Message was: "Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found" at
/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:269
.

6) OCA\DAV\Tests\Command\MoveCalendarTest::testMoveWithCalendarAlreadySharedToDestination with data set #1 (false)
Failed asserting that exception of type "Error" matches expected exception "InvalidArgumentException". Message was: "Class 'Symfony\Component\Console\Output\TrimmedBufferOutput' not found" at
/drone/src/3rdparty/symfony/console/Style/SymfonyStyle.php:49
/drone/src/apps/dav/lib/Command/MoveCalendar.php:114
/drone/src/3rdparty/symfony/console/Command/Command.php:255
/drone/src/3rdparty/symfony/console/Tester/CommandTester.php:76
/drone/src/apps/dav/tests/unit/Command/MoveCalendarTest.php:421
.

mariadb10.4-php7.4

Show full log
There were 2 warnings:

1) 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

2) 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

--

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

@juliushaertl juliushaertl merged commit 5e3fe1c into master Dec 30, 2020
@juliushaertl juliushaertl deleted the bugfix/noid/always-show-external-share branch December 30, 2020 16:41
@rullzer rullzer mentioned this pull request Jan 4, 2021
5 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3. to review Waiting for reviews enhancement
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants