diff options
| author | Côme Chilliet <91878298+come-nc@users.noreply.github.com> | 2025-02-13 11:33:52 +0100 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2025-02-13 11:33:52 +0100 |
| commit | ed9b47462f25937831471ba1aabcc03757e5dee5 (patch) | |
| tree | 80e07fb06f64b52a2c3c3df396953c065be76431 /tests/lib | |
| parent | ee48cafd200233203a1444dba797ef3eb89a35ca (diff) | |
| parent | f758f565d4da004ebef97350365a066eee5267fe (diff) | |
| download | nextcloud-server-ed9b47462f25937831471ba1aabcc03757e5dee5.tar.gz nextcloud-server-ed9b47462f25937831471ba1aabcc03757e5dee5.zip | |
Merge pull request #49648 from nextcloud/fix/clarify-app-manager-methods
Clarify app manager method names
Diffstat (limited to 'tests/lib')
| -rw-r--r-- | tests/lib/App/AppManagerTest.php | 10 | ||||
| -rw-r--r-- | tests/lib/NavigationManagerTest.php | 4 |
2 files changed, 7 insertions, 7 deletions
diff --git a/tests/lib/App/AppManagerTest.php b/tests/lib/App/AppManagerTest.php index fd7a065d5b3..19039366aa8 100644 --- a/tests/lib/App/AppManagerTest.php +++ b/tests/lib/App/AppManagerTest.php @@ -426,17 +426,17 @@ class AppManagerTest extends TestCase { public function testIsInstalledEnabled(): void { $this->appConfig->setValue('test', 'enabled', 'yes'); - $this->assertTrue($this->manager->isInstalled('test')); + $this->assertTrue($this->manager->isEnabledForAnyone('test')); } public function testIsInstalledDisabled(): void { $this->appConfig->setValue('test', 'enabled', 'no'); - $this->assertFalse($this->manager->isInstalled('test')); + $this->assertFalse($this->manager->isEnabledForAnyone('test')); } public function testIsInstalledEnabledForGroups(): void { $this->appConfig->setValue('test', 'enabled', '["foo"]'); - $this->assertTrue($this->manager->isInstalled('test')); + $this->assertTrue($this->manager->isEnabledForAnyone('test')); } private function newUser($uid) { @@ -539,7 +539,7 @@ class AppManagerTest extends TestCase { $this->assertTrue($this->manager->isEnabledForUser('test')); } - public function testGetInstalledApps(): void { + public function testGetEnabledApps(): void { $this->appConfig->setValue('test1', 'enabled', 'yes'); $this->appConfig->setValue('test2', 'enabled', 'no'); $this->appConfig->setValue('test3', 'enabled', '["foo"]'); @@ -560,7 +560,7 @@ class AppManagerTest extends TestCase { 'viewer', 'workflowengine', ]; - $this->assertEquals($apps, $this->manager->getInstalledApps()); + $this->assertEquals($apps, $this->manager->getEnabledApps()); } public function testGetAppsForUser(): void { diff --git a/tests/lib/NavigationManagerTest.php b/tests/lib/NavigationManagerTest.php index d9db928e8a1..91da970f3b9 100644 --- a/tests/lib/NavigationManagerTest.php +++ b/tests/lib/NavigationManagerTest.php @@ -718,7 +718,7 @@ class NavigationManagerTest extends TestCase { 'id' => 'settings', ]); - $this->appManager->method('getInstalledApps')->willReturn([]); + $this->appManager->method('getEnabledApps')->willReturn([]); $user = $this->createMock(IUser::class); $user->method('getUID')->willReturn('user1'); @@ -743,7 +743,7 @@ class NavigationManagerTest extends TestCase { } public function testDefaultEntryUpdated(): void { - $this->appManager->method('getInstalledApps')->willReturn([]); + $this->appManager->method('getEnabledApps')->willReturn([]); $user = $this->createMock(IUser::class); $user->method('getUID')->willReturn('user1'); |