Skip to content

Conversation

@juliusknorr
Copy link
Member

Signed-off-by: Julius Härtl [email protected]

Signed-off-by: Julius Härtl <[email protected]>
@skjnldsv
Copy link
Member

........PHP Fatal error:  Class OCA\AnnouncementCenter\Notification\Notifier contains 2 abstract methods and must therefore be declared abstract or implement the remaining methods (OCP\Notification\INotifier::getID, OCP\Notification\INotifier::getName) in /home/travis/build/nextcloud/server/apps/announcementcenter/lib/Notification/Notifier.php on line 38
PHP Stack trace:
PHP   1. {main}() /home/travis/build/nextcloud/vendor/phpunit/phpunit/phpunit:0
PHP   2. PHPUnit\TextUI\Command::main() /home/travis/build/nextcloud/vendor/phpunit/phpunit/phpunit:53
PHP   3. PHPUnit\TextUI\Command->run() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/TextUI/Command.php:148
PHP   4. PHPUnit\TextUI\TestRunner->doRun() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/TextUI/Command.php:195
PHP   5. PHPUnit\Framework\TestSuite->run() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/TextUI/TestRunner.php:545
PHP   6. PHPUnit\Framework\TestSuite->run() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/Framework/TestSuite.php:755
PHP   7. OCA\AnnouncementCenter\Tests\AppTest->run() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/Framework/TestSuite.php:755
PHP   8. PHPUnit\Framework\TestResult->run() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/Framework/TestCase.php:894
PHP   9. OCA\AnnouncementCenter\Tests\AppTest->runBare() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/Framework/TestResult.php:698
PHP  10. OCA\AnnouncementCenter\Tests\AppTest->runTest() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/Framework/TestCase.php:939
PHP  11. ReflectionMethod->invokeArgs() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/Framework/TestCase.php:1071
PHP  12. OCA\AnnouncementCenter\Tests\AppTest->testAppNotification() /home/travis/build/nextcloud/vendor/phpunit/phpunit/src/Framework/TestCase.php:1071
PHP  13. include() /home/travis/build/nextcloud/server/apps/announcementcenter/tests/AppInfo/AppTest.php:95
PHP  14. OCA\AnnouncementCenter\AppInfo\Application->register() /home/travis/build/nextcloud/server/apps/announcementcenter/appinfo/app.php:25
PHP  15. OCA\AnnouncementCenter\AppInfo\Application->registerNotificationNotifier() /home/travis/build/nextcloud/server/apps/announcementcenter/lib/AppInfo/Application.php:44
PHP  16. Mock_IManager_42edf976->registerNotifier() /home/travis/build/nextcloud/server/apps/announcementcenter/lib/AppInfo/Application.php:72
PHP  17. PHPUnit\Framework\MockObject\InvocationMocker->invoke() /home/travis/build/nextcloud/vendor/phpunit/phpunit-mock-objects/src/Generator.php(264) : eval()'d code:49
PHP  18. PHPUnit\Framework\MockObject\Matcher->invoked() /home/travis/build/nextcloud/vendor/phpunit/phpunit-mock-objects/src/InvocationMocker.php:131
PHP  19. PHPUnit\Framework\MockObject\Stub\ReturnCallback->invoke() /home/travis/build/nextcloud/vendor/phpunit/phpunit-mock-objects/src/Matcher.php:177
PHP  20. OCA\AnnouncementCenter\Tests\AppTest->OCA\AnnouncementCenter\Tests\{closure:/home/travis/build/nextcloud/server/apps/announcementcenter/tests/AppInfo/AppTest.php:82-93}() /home/travis/build/nextcloud/vendor/phpunit/phpunit-mock-objects/src/Stub/ReturnCallback.php:26
PHP  21. OCA\AnnouncementCenter\AppInfo\Application->OCA\AnnouncementCenter\AppInfo\{closure:/home/travis/build/nextcloud/server/apps/announcementcenter/lib/AppInfo/Application.php:64-66}() /home/travis/build/nextcloud/server/apps/announcementcenter/tests/AppInfo/AppTest.php:84
PHP  22. OC\AppFramework\DependencyInjection\DIContainer->query() /home/travis/build/nextcloud/server/apps/announcementcenter/lib/AppInfo/Application.php:65
PHP  23. OC\AppFramework\DependencyInjection\DIContainer->queryNoFallback() /home/travis/build/nextcloud/server/lib/private/AppFramework/DependencyInjection/DIContainer.php:386
PHP  24. OC\AppFramework\DependencyInjection\DIContainer->query() /home/travis/build/nextcloud/server/lib/private/AppFramework/DependencyInjection/DIContainer.php:415
PHP  25. OC\AppFramework\DependencyInjection\DIContainer->resolve() /home/travis/build/nextcloud/server/lib/private/AppFramework/Utility/SimpleContainer.php:114
PHP  26. ReflectionClass->__construct() /home/travis/build/nextcloud/server/lib/private/AppFramework/Utility/SimpleContainer.php:97
PHP  27. spl_autoload_call() /home/travis/build/nextcloud/server/lib/private/AppFramework/Utility/SimpleContainer.php:97
PHP  28. Composer\Autoload\ClassLoader->loadClass() /home/travis/build/nextcloud/server/lib/private/AppFramework/Utility/SimpleContainer.php:97
PHP  29. Composer\Autoload\includeFile() /home/travis/build/nextcloud/vendor/composer/ClassLoader.php:322
PHP  30. include() /home/travis/build/nextcloud/vendor/composer/ClassLoader.php:444

@rullzer
Copy link
Member

rullzer commented Aug 14, 2019

See #156

@skjnldsv
Copy link
Member

@rullzer so let's merge anyway?

@rullzer rullzer merged commit 24066c1 into master Aug 14, 2019
@delete-merged-branch delete-merged-branch bot deleted the tests/travis-db branch August 14, 2019 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants