diff --git a/apps/dav/lib/AppInfo/Application.php b/apps/dav/lib/AppInfo/Application.php index a90e42d56f..41570ee744 100644 --- a/apps/dav/lib/AppInfo/Application.php +++ b/apps/dav/lib/AppInfo/Application.php @@ -77,7 +77,7 @@ class Application extends App { } /** - * @param IManager $contactsManager + * @param IContactsManager $contactsManager */ public function setupSystemContactsProvider(IContactsManager $contactsManager) { /** @var ContactsManager $cm */ diff --git a/apps/dav/tests/unit/AppInfo/ApplicationTest.php b/apps/dav/tests/unit/AppInfo/ApplicationTest.php index d4f53d360a..d34749ec92 100644 --- a/apps/dav/tests/unit/AppInfo/ApplicationTest.php +++ b/apps/dav/tests/unit/AppInfo/ApplicationTest.php @@ -26,7 +26,6 @@ namespace OCA\DAV\Tests\unit\AppInfo; use OCA\DAV\AppInfo\Application; use OCA\DAV\CardDAV\CardDavBackend; use OCA\DAV\CardDAV\ContactsManager; -use OCP\Contacts\IManager; use Test\TestCase; /** @@ -43,24 +42,8 @@ class ApplicationTest extends TestCase { // assert service instances in the container are properly setup $s = $c->query(ContactsManager::class); - $this->assertInstanceOf('OCA\DAV\CardDAV\ContactsManager', $s); + $this->assertInstanceOf(ContactsManager::class, $s); $s = $c->query(CardDavBackend::class); - $this->assertInstanceOf('OCA\DAV\CardDAV\CardDavBackend', $s); - } - - public function testContactsManagerSetup() { - $app = new Application(); - $c = $app->getContainer(); - $c->registerService(CardDavBackend::class, function() { - $service = $this->createMock(CardDavBackend::class); - $service->method('getAddressBooksForUser')->willReturn([]); - return $service; - }); - - // assert setupContactsProvider() is proper - /** @var IManager|\PHPUnit_Framework_MockObject_MockObject $cm */ - $cm = $this->createMock(IManager::class); - $app->setupContactsProvider($cm, 'xxx'); - $this->addToAssertionCount(1); + $this->assertInstanceOf(CardDavBackend::class, $s); } }