From ee02165005f3fd24e1a17fdb3faeedda036d93c7 Mon Sep 17 00:00:00 2001 From: Joas Schilling Date: Thu, 14 Jan 2016 14:35:24 +0100 Subject: [PATCH] Move the notification API to public namespace --- apps/files_sharing/lib/external/manager.php | 2 +- apps/files_sharing/lib/notifier.php | 4 +- lib/private/notification/action.php | 2 + lib/private/notification/manager.php | 5 + lib/private/notification/notification.php | 3 + lib/private/server.php | 2 +- lib/public/iservercontainer.php | 2 +- .../notification/iaction.php | 8 +- lib/{private => public}/notification/iapp.php | 8 +- .../notification/imanager.php | 8 +- .../notification/inotification.php | 8 +- .../notification/inotifier.php | 8 +- tests/lib/notification/actiontest.php | 2 +- tests/lib/notification/managertest.php | 92 +++++++++---------- tests/lib/notification/notificationtest.php | 34 +++---- tests/lib/server.php | 2 +- 16 files changed, 90 insertions(+), 100 deletions(-) rename lib/{private => public}/notification/iaction.php (90%) rename lib/{private => public}/notification/iapp.php (85%) rename lib/{private => public}/notification/imanager.php (88%) rename lib/{private => public}/notification/inotification.php (95%) rename lib/{private => public}/notification/inotifier.php (84%) diff --git a/apps/files_sharing/lib/external/manager.php b/apps/files_sharing/lib/external/manager.php index 7112aa63d1..ea49348737 100644 --- a/apps/files_sharing/lib/external/manager.php +++ b/apps/files_sharing/lib/external/manager.php @@ -28,7 +28,7 @@ namespace OCA\Files_Sharing\External; use OC\Files\Filesystem; use OCP\Files; -use OC\Notification\IManager; +use OCP\Notification\IManager; class Manager { const STORAGE = '\OCA\Files_Sharing\External\Storage'; diff --git a/apps/files_sharing/lib/notifier.php b/apps/files_sharing/lib/notifier.php index c5c5615145..802f22e44c 100644 --- a/apps/files_sharing/lib/notifier.php +++ b/apps/files_sharing/lib/notifier.php @@ -22,8 +22,8 @@ namespace OCA\Files_Sharing; -use OC\Notification\INotification; -use OC\Notification\INotifier; +use OCP\Notification\INotification; +use OCP\Notification\INotifier; class Notifier implements INotifier { /** @var \OCP\L10N\IFactory */ diff --git a/lib/private/notification/action.php b/lib/private/notification/action.php index 1730756c1d..deac680765 100644 --- a/lib/private/notification/action.php +++ b/lib/private/notification/action.php @@ -22,6 +22,8 @@ namespace OC\Notification; +use OCP\Notification\IAction; + class Action implements IAction { /** @var string */ diff --git a/lib/private/notification/manager.php b/lib/private/notification/manager.php index 239b5bfe5c..a14a1deec0 100644 --- a/lib/private/notification/manager.php +++ b/lib/private/notification/manager.php @@ -22,6 +22,11 @@ namespace OC\Notification; +use OCP\Notification\IApp; +use OCP\Notification\IManager; +use OCP\Notification\INotification; +use OCP\Notification\INotifier; + class Manager implements IManager { /** @var IApp */ protected $apps; diff --git a/lib/private/notification/notification.php b/lib/private/notification/notification.php index 677658eb3e..6568f53f25 100644 --- a/lib/private/notification/notification.php +++ b/lib/private/notification/notification.php @@ -22,6 +22,9 @@ namespace OC\Notification; +use OCP\Notification\IAction; +use OCP\Notification\INotification; + class Notification implements INotification { /** @var string */ protected $app; diff --git a/lib/private/server.php b/lib/private/server.php index a16e6b9839..0cb52b7918 100644 --- a/lib/private/server.php +++ b/lib/private/server.php @@ -1152,7 +1152,7 @@ class Server extends ServerContainer implements IServerContainer { /** * Get the Notification Manager * - * @return \OC\Notification\IManager + * @return \OCP\Notification\IManager * @since 8.2.0 */ public function getNotificationManager() { diff --git a/lib/public/iservercontainer.php b/lib/public/iservercontainer.php index 1976f59f61..259f991414 100644 --- a/lib/public/iservercontainer.php +++ b/lib/public/iservercontainer.php @@ -468,7 +468,7 @@ interface IServerContainer { /** * Get the Notification Manager * - * @return \OC\Notification\IManager + * @return \OCP\Notification\IManager * @since 8.2.0 */ public function getNotificationManager(); diff --git a/lib/private/notification/iaction.php b/lib/public/notification/iaction.php similarity index 90% rename from lib/private/notification/iaction.php rename to lib/public/notification/iaction.php index 79e28ecef8..81b842a985 100644 --- a/lib/private/notification/iaction.php +++ b/lib/public/notification/iaction.php @@ -19,17 +19,13 @@ * */ -namespace OC\Notification; +namespace OCP\Notification; /** * Interface IAction * - * @package OC\Notification + * @package OCP\Notification * @since 8.2.0 - * - * DEVELOPER NOTE: - * The notification api is experimental only in 8.2.0! Do not start using it, - * if you can not prepare an update for the next version afterwards. */ interface IAction { /** diff --git a/lib/private/notification/iapp.php b/lib/public/notification/iapp.php similarity index 85% rename from lib/private/notification/iapp.php rename to lib/public/notification/iapp.php index 8ae4f6a543..eae3fe9271 100644 --- a/lib/private/notification/iapp.php +++ b/lib/public/notification/iapp.php @@ -19,17 +19,13 @@ * */ -namespace OC\Notification; +namespace OCP\Notification; /** * Interface IApp * - * @package OC\Notification + * @package OCP\Notification * @since 8.2.0 - * - * DEVELOPER NOTE: - * The notification api is experimental only in 8.2.0! Do not start using it, - * if you can not prepare an update for the next version afterwards. */ interface IApp { /** diff --git a/lib/private/notification/imanager.php b/lib/public/notification/imanager.php similarity index 88% rename from lib/private/notification/imanager.php rename to lib/public/notification/imanager.php index 2f8e3ba39f..b891385c79 100644 --- a/lib/private/notification/imanager.php +++ b/lib/public/notification/imanager.php @@ -19,17 +19,13 @@ * */ -namespace OC\Notification; +namespace OCP\Notification; /** * Interface IManager * - * @package OC\Notification + * @package OCP\Notification * @since 8.2.0 - * - * DEVELOPER NOTE: - * The notification api is experimental only in 8.2.0! Do not start using it, - * if you can not prepare an update for the next version afterwards. */ interface IManager extends IApp, INotifier { /** diff --git a/lib/private/notification/inotification.php b/lib/public/notification/inotification.php similarity index 95% rename from lib/private/notification/inotification.php rename to lib/public/notification/inotification.php index 921f0779b9..058e9721d8 100644 --- a/lib/private/notification/inotification.php +++ b/lib/public/notification/inotification.php @@ -19,17 +19,13 @@ * */ -namespace OC\Notification; +namespace OCP\Notification; /** * Interface INotification * - * @package OC\Notification + * @package OCP\Notification * @since 8.2.0 - * - * DEVELOPER NOTE: - * The notification api is experimental only in 8.2.0! Do not start using it, - * if you can not prepare an update for the next version afterwards. */ interface INotification { /** diff --git a/lib/private/notification/inotifier.php b/lib/public/notification/inotifier.php similarity index 84% rename from lib/private/notification/inotifier.php rename to lib/public/notification/inotifier.php index c4312308fc..1740029664 100644 --- a/lib/private/notification/inotifier.php +++ b/lib/public/notification/inotifier.php @@ -19,17 +19,13 @@ * */ -namespace OC\Notification; +namespace OCP\Notification; /** * Interface INotifier * - * @package OC\Notification + * @package OCP\Notification * @since 8.2.0 - * - * DEVELOPER NOTE: - * The notification api is experimental only in 8.2.0! Do not start using it, - * if you can not prepare an update for the next version afterwards. */ interface INotifier { /** diff --git a/tests/lib/notification/actiontest.php b/tests/lib/notification/actiontest.php index 0044df82cd..74c995280c 100644 --- a/tests/lib/notification/actiontest.php +++ b/tests/lib/notification/actiontest.php @@ -23,7 +23,7 @@ namespace Test\Notification; use OC\Notification\Action; -use OC\Notification\IAction; +use OCP\Notification\IAction; use Test\TestCase; class ActionTest extends TestCase { diff --git a/tests/lib/notification/managertest.php b/tests/lib/notification/managertest.php index 1f411bc74e..4410781bc3 100644 --- a/tests/lib/notification/managertest.php +++ b/tests/lib/notification/managertest.php @@ -22,7 +22,7 @@ namespace Test\Notification; use OC\Notification\Manager; -use OC\Notification\IManager; +use OCP\Notification\IManager; use Test\TestCase; class ManagerTest extends TestCase { @@ -35,7 +35,7 @@ class ManagerTest extends TestCase { } public function testRegisterApp() { - $app = $this->getMockBuilder('OC\Notification\IApp') + $app = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); @@ -59,7 +59,7 @@ class ManagerTest extends TestCase { * @expectedException \InvalidArgumentException */ public function testRegisterAppInvalid() { - $notifier = $this->getMockBuilder('OC\Notification\INotifier') + $notifier = $this->getMockBuilder('OCP\Notification\INotifier') ->disableOriginalConstructor() ->getMock(); @@ -73,7 +73,7 @@ class ManagerTest extends TestCase { } public function testRegisterNotifier() { - $notifier = $this->getMockBuilder('OC\Notification\INotifier') + $notifier = $this->getMockBuilder('OCP\Notification\INotifier') ->disableOriginalConstructor() ->getMock(); @@ -105,7 +105,7 @@ class ManagerTest extends TestCase { * @expectedException \InvalidArgumentException */ public function testRegisterNotifierInvalid() { - $app = $this->getMockBuilder('OC\Notification\IApp') + $app = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); @@ -136,7 +136,7 @@ class ManagerTest extends TestCase { * @param mixed $data */ public function testRegisterNotifierInfoInvalid($data) { - $app = $this->getMockBuilder('OC\Notification\IApp') + $app = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); @@ -156,7 +156,7 @@ class ManagerTest extends TestCase { * @expectedExceptionMessage The given notifier ID test1 is already in use */ public function testRegisterNotifierInfoDuplicate() { - $app = $this->getMockBuilder('OC\Notification\IApp') + $app = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); @@ -177,28 +177,28 @@ class ManagerTest extends TestCase { public function testCreateNotification() { $action = $this->manager->createNotification(); - $this->assertInstanceOf('OC\Notification\INotification', $action); + $this->assertInstanceOf('OCP\Notification\INotification', $action); } public function testNotify() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); $notification->expects($this->once()) ->method('isValid') ->willReturn(true); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app */ - $app = $this->getMockBuilder('OC\Notification\IApp') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app */ + $app = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); $app->expects($this->once()) ->method('notify') ->with($notification); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app2 */ - $app2 = $this->getMockBuilder('OC\Notification\IApp') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app2 */ + $app2 = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); $app2->expects($this->once()) @@ -219,8 +219,8 @@ class ManagerTest extends TestCase { * @expectedException \InvalidArgumentException */ public function testNotifyInvalid() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); $notification->expects($this->once()) @@ -231,23 +231,23 @@ class ManagerTest extends TestCase { } public function testPrepare() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); $notification->expects($this->once()) ->method('isValidParsed') ->willReturn(true); - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification2 */ - $notification2 = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification2 */ + $notification2 = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); $notification2->expects($this->exactly(2)) ->method('isValidParsed') ->willReturn(true); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier */ - $notifier = $this->getMockBuilder('OC\Notification\INotifier') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier */ + $notifier = $this->getMockBuilder('OCP\Notification\INotifier') ->disableOriginalConstructor() ->getMock(); $notifier->expects($this->once()) @@ -255,8 +255,8 @@ class ManagerTest extends TestCase { ->with($notification, 'en') ->willReturnArgument(0); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier2 */ - $notifier2 = $this->getMockBuilder('OC\Notification\INotifier') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier2 */ + $notifier2 = $this->getMockBuilder('OCP\Notification\INotifier') ->disableOriginalConstructor() ->getMock(); $notifier2->expects($this->once()) @@ -282,16 +282,16 @@ class ManagerTest extends TestCase { * @expectedException \InvalidArgumentException */ public function testPrepareInvalid() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); $notification->expects($this->once()) ->method('isValidParsed') ->willReturn(false); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier */ - $notifier = $this->getMockBuilder('OC\Notification\INotifier') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier */ + $notifier = $this->getMockBuilder('OCP\Notification\INotifier') ->disableOriginalConstructor() ->getMock(); $notifier->expects($this->once()) @@ -309,16 +309,16 @@ class ManagerTest extends TestCase { } public function testPrepareNotifierThrows() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); $notification->expects($this->once()) ->method('isValidParsed') ->willReturn(true); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier */ - $notifier = $this->getMockBuilder('OC\Notification\INotifier') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $notifier */ + $notifier = $this->getMockBuilder('OCP\Notification\INotifier') ->disableOriginalConstructor() ->getMock(); $notifier->expects($this->once()) @@ -339,8 +339,8 @@ class ManagerTest extends TestCase { * @expectedException \InvalidArgumentException */ public function testPrepareNoNotifier() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); $notification->expects($this->once()) @@ -351,21 +351,21 @@ class ManagerTest extends TestCase { } public function testMarkProcessed() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app */ - $app = $this->getMockBuilder('OC\Notification\IApp') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app */ + $app = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); $app->expects($this->once()) ->method('markProcessed') ->with($notification); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app2 */ - $app2 = $this->getMockBuilder('OC\Notification\IApp') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app2 */ + $app2 = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); $app2->expects($this->once()) @@ -383,13 +383,13 @@ class ManagerTest extends TestCase { } public function testGetCount() { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ - $notification = $this->getMockBuilder('OC\Notification\INotification') + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + $notification = $this->getMockBuilder('OCP\Notification\INotification') ->disableOriginalConstructor() ->getMock(); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app */ - $app = $this->getMockBuilder('OC\Notification\IApp') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app */ + $app = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); $app->expects($this->once()) @@ -397,8 +397,8 @@ class ManagerTest extends TestCase { ->with($notification) ->willReturn(21); - /** @var \OC\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app2 */ - $app2 = $this->getMockBuilder('OC\Notification\IApp') + /** @var \OCP\Notification\IApp|\PHPUnit_Framework_MockObject_MockObject $app2 */ + $app2 = $this->getMockBuilder('OCP\Notification\IApp') ->disableOriginalConstructor() ->getMock(); $app2->expects($this->once()) diff --git a/tests/lib/notification/notificationtest.php b/tests/lib/notification/notificationtest.php index 91fa1de8b4..ee9454c77a 100644 --- a/tests/lib/notification/notificationtest.php +++ b/tests/lib/notification/notificationtest.php @@ -23,7 +23,7 @@ namespace Test\Notification; use OC\Notification\Notification; -use OC\Notification\INotification; +use OCP\Notification\INotification; use Test\TestCase; class NotificationTest extends TestCase { @@ -388,12 +388,12 @@ class NotificationTest extends TestCase { public function testCreateAction() { $action = $this->notification->createAction(); - $this->assertInstanceOf('OC\Notification\IAction', $action); + $this->assertInstanceOf('OCP\Notification\IAction', $action); } public function testAddAction() { - /** @var \OC\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ - $action = $this->getMockBuilder('OC\Notification\IAction') + /** @var \OCP\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ + $action = $this->getMockBuilder('OCP\Notification\IAction') ->disableOriginalConstructor() ->getMock(); $action->expects($this->once()) @@ -412,8 +412,8 @@ class NotificationTest extends TestCase { * @expectedException \InvalidArgumentException */ public function testAddActionInvalid() { - /** @var \OC\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ - $action = $this->getMockBuilder('OC\Notification\IAction') + /** @var \OCP\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ + $action = $this->getMockBuilder('OCP\Notification\IAction') ->disableOriginalConstructor() ->getMock(); $action->expects($this->once()) @@ -426,8 +426,8 @@ class NotificationTest extends TestCase { } public function testAddActionSecondPrimary() { - /** @var \OC\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ - $action = $this->getMockBuilder('OC\Notification\IAction') + /** @var \OCP\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ + $action = $this->getMockBuilder('OCP\Notification\IAction') ->disableOriginalConstructor() ->getMock(); $action->expects($this->exactly(2)) @@ -444,8 +444,8 @@ class NotificationTest extends TestCase { } public function testAddParsedAction() { - /** @var \OC\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ - $action = $this->getMockBuilder('OC\Notification\IAction') + /** @var \OCP\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ + $action = $this->getMockBuilder('OCP\Notification\IAction') ->disableOriginalConstructor() ->getMock(); $action->expects($this->once()) @@ -464,8 +464,8 @@ class NotificationTest extends TestCase { * @expectedException \InvalidArgumentException */ public function testAddParsedActionInvalid() { - /** @var \OC\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ - $action = $this->getMockBuilder('OC\Notification\IAction') + /** @var \OCP\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ + $action = $this->getMockBuilder('OCP\Notification\IAction') ->disableOriginalConstructor() ->getMock(); $action->expects($this->once()) @@ -478,8 +478,8 @@ class NotificationTest extends TestCase { } public function testAddActionSecondParsedPrimary() { - /** @var \OC\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ - $action = $this->getMockBuilder('OC\Notification\IAction') + /** @var \OCP\Notification\IAction|\PHPUnit_Framework_MockObject_MockObject $action */ + $action = $this->getMockBuilder('OCP\Notification\IAction') ->disableOriginalConstructor() ->getMock(); $action->expects($this->exactly(2)) @@ -512,7 +512,7 @@ class NotificationTest extends TestCase { * @param bool $expected */ public function testIsValid($isValidCommon, $subject, $expected) { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ $notification = $this->getMockBuilder('\OC\Notification\Notification') ->setMethods([ 'isValidCommon', @@ -543,7 +543,7 @@ class NotificationTest extends TestCase { * @param bool $expected */ public function testIsParsedValid($isValidCommon, $subject, $expected) { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ $notification = $this->getMockBuilder('\OC\Notification\Notification') ->setMethods([ 'isValidCommon', @@ -588,7 +588,7 @@ class NotificationTest extends TestCase { * @param bool $expected */ public function testIsValidCommon($app, $user, $timestamp, $objectType, $objectId, $expected) { - /** @var \OC\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ + /** @var \OCP\Notification\INotification|\PHPUnit_Framework_MockObject_MockObject $notification */ $notification = $this->getMockBuilder('\OC\Notification\Notification') ->setMethods([ 'getApp', diff --git a/tests/lib/server.php b/tests/lib/server.php index 684e6a9736..91beea7c2f 100644 --- a/tests/lib/server.php +++ b/tests/lib/server.php @@ -116,7 +116,7 @@ class Server extends \Test\TestCase { ['NavigationManager', '\OC\NavigationManager'], ['NavigationManager', '\OCP\INavigationManager'], ['NotificationManager', '\OC\Notification\Manager'], - ['NotificationManager', '\OC\Notification\IManager'], + ['NotificationManager', '\OCP\Notification\IManager'], ['UserCache', '\OC\Cache\File'], ['UserCache', '\OCP\ICache'],