@@ -22,99 +22,99 @@ |
||
22 | 22 | use Test\TestCase; |
23 | 23 | |
24 | 24 | class LoadAdditionalListenerTest extends TestCase { |
25 | - protected LoggerInterface&MockObject $logger; |
|
26 | - protected LoadAdditionalScriptsEvent&MockObject $event; |
|
27 | - protected IManager&MockObject $shareManager; |
|
28 | - protected IFactory&MockObject $factory; |
|
29 | - protected InitialStateService&MockObject $initialStateService; |
|
30 | - protected IConfig&MockObject $config; |
|
31 | - |
|
32 | - protected function setUp(): void { |
|
33 | - parent::setUp(); |
|
34 | - |
|
35 | - $this->logger = $this->createMock(LoggerInterface::class); |
|
36 | - $this->event = $this->createMock(LoadAdditionalScriptsEvent::class); |
|
37 | - $this->shareManager = $this->createMock(IManager::class); |
|
38 | - $this->factory = $this->createMock(IFactory::class); |
|
39 | - $this->initialStateService = $this->createMock(InitialStateService::class); |
|
40 | - $this->config = $this->createMock(IConfig::class); |
|
41 | - |
|
42 | - /* Empty static array to avoid inter-test conflicts */ |
|
43 | - \OC_Util::$styles = []; |
|
44 | - self::invokePrivate(Util::class, 'scripts', [[]]); |
|
45 | - self::invokePrivate(Util::class, 'scriptDeps', [[]]); |
|
46 | - self::invokePrivate(Util::class, 'scriptsInit', [[]]); |
|
47 | - } |
|
48 | - |
|
49 | - protected function tearDown(): void { |
|
50 | - parent::tearDown(); |
|
51 | - |
|
52 | - \OC_Util::$styles = []; |
|
53 | - self::invokePrivate(Util::class, 'scripts', [[]]); |
|
54 | - self::invokePrivate(Util::class, 'scriptDeps', [[]]); |
|
55 | - self::invokePrivate(Util::class, 'scriptsInit', [[]]); |
|
56 | - } |
|
57 | - |
|
58 | - public function testHandleIgnoresNonMatchingEvent(): void { |
|
59 | - $listener = new LoadAdditionalListener(); |
|
60 | - $event = $this->createMock(Event::class); |
|
61 | - |
|
62 | - // Should not throw or call anything |
|
63 | - $listener->handle($event); |
|
64 | - |
|
65 | - $this->assertTrue(true); // No exception means pass |
|
66 | - } |
|
67 | - |
|
68 | - public function testHandleWithLoadAdditionalScriptsEvent(): void { |
|
69 | - $listener = new LoadAdditionalListener(); |
|
70 | - |
|
71 | - $this->shareManager->method('shareApiEnabled')->willReturn(false); |
|
72 | - $this->factory->method('findLanguage')->willReturn('language_mock'); |
|
73 | - $this->config->method('getSystemValueBool')->willReturn(true); |
|
74 | - |
|
75 | - $this->overwriteService(IManager::class, $this->shareManager); |
|
76 | - $this->overwriteService(IFactory::class, $this->factory); |
|
77 | - $this->overwriteService(InitialStateService::class, $this->initialStateService); |
|
78 | - $this->overwriteService(IConfig::class, $this->config); |
|
79 | - |
|
80 | - $scriptsBefore = Util::getScripts(); |
|
81 | - $this->assertNotContains('files_sharing/l10n/language_mock', $scriptsBefore); |
|
82 | - $this->assertNotContains('files_sharing/js/additionalScripts', $scriptsBefore); |
|
83 | - $this->assertNotContains('files_sharing/js/init', $scriptsBefore); |
|
84 | - $this->assertNotContains('files_sharing/css/icons', \OC_Util::$styles); |
|
85 | - |
|
86 | - // Util static methods can't be easily mocked, so just ensure no exceptions |
|
87 | - $listener->handle($this->event); |
|
88 | - |
|
89 | - // assert array $scripts contains the expected scripts |
|
90 | - $scriptsAfter = Util::getScripts(); |
|
91 | - $this->assertContains('files_sharing/l10n/language_mock', $scriptsAfter); |
|
92 | - $this->assertContains('files_sharing/js/additionalScripts', $scriptsAfter); |
|
93 | - $this->assertNotContains('files_sharing/js/init', $scriptsAfter); |
|
94 | - |
|
95 | - $this->assertContains('files_sharing/css/icons', \OC_Util::$styles); |
|
96 | - } |
|
97 | - |
|
98 | - public function testHandleWithLoadAdditionalScriptsEventWithShareApiEnabled(): void { |
|
99 | - $listener = new LoadAdditionalListener(); |
|
100 | - |
|
101 | - $this->shareManager->method('shareApiEnabled')->willReturn(true); |
|
102 | - $this->config->method('getSystemValueBool')->willReturn(true); |
|
103 | - |
|
104 | - $this->overwriteService(IManager::class, $this->shareManager); |
|
105 | - $this->overwriteService(InitialStateService::class, $this->initialStateService); |
|
106 | - $this->overwriteService(IConfig::class, $this->config); |
|
107 | - $this->overwriteService(IFactory::class, $this->factory); |
|
108 | - |
|
109 | - $scriptsBefore = Util::getScripts(); |
|
110 | - $this->assertNotContains('files_sharing/js/init', $scriptsBefore); |
|
111 | - |
|
112 | - // Util static methods can't be easily mocked, so just ensure no exceptions |
|
113 | - $listener->handle($this->event); |
|
114 | - |
|
115 | - $scriptsAfter = Util::getScripts(); |
|
116 | - |
|
117 | - // assert array $scripts contains the expected scripts |
|
118 | - $this->assertContains('files_sharing/js/init', $scriptsAfter); |
|
119 | - } |
|
25 | + protected LoggerInterface&MockObject $logger; |
|
26 | + protected LoadAdditionalScriptsEvent&MockObject $event; |
|
27 | + protected IManager&MockObject $shareManager; |
|
28 | + protected IFactory&MockObject $factory; |
|
29 | + protected InitialStateService&MockObject $initialStateService; |
|
30 | + protected IConfig&MockObject $config; |
|
31 | + |
|
32 | + protected function setUp(): void { |
|
33 | + parent::setUp(); |
|
34 | + |
|
35 | + $this->logger = $this->createMock(LoggerInterface::class); |
|
36 | + $this->event = $this->createMock(LoadAdditionalScriptsEvent::class); |
|
37 | + $this->shareManager = $this->createMock(IManager::class); |
|
38 | + $this->factory = $this->createMock(IFactory::class); |
|
39 | + $this->initialStateService = $this->createMock(InitialStateService::class); |
|
40 | + $this->config = $this->createMock(IConfig::class); |
|
41 | + |
|
42 | + /* Empty static array to avoid inter-test conflicts */ |
|
43 | + \OC_Util::$styles = []; |
|
44 | + self::invokePrivate(Util::class, 'scripts', [[]]); |
|
45 | + self::invokePrivate(Util::class, 'scriptDeps', [[]]); |
|
46 | + self::invokePrivate(Util::class, 'scriptsInit', [[]]); |
|
47 | + } |
|
48 | + |
|
49 | + protected function tearDown(): void { |
|
50 | + parent::tearDown(); |
|
51 | + |
|
52 | + \OC_Util::$styles = []; |
|
53 | + self::invokePrivate(Util::class, 'scripts', [[]]); |
|
54 | + self::invokePrivate(Util::class, 'scriptDeps', [[]]); |
|
55 | + self::invokePrivate(Util::class, 'scriptsInit', [[]]); |
|
56 | + } |
|
57 | + |
|
58 | + public function testHandleIgnoresNonMatchingEvent(): void { |
|
59 | + $listener = new LoadAdditionalListener(); |
|
60 | + $event = $this->createMock(Event::class); |
|
61 | + |
|
62 | + // Should not throw or call anything |
|
63 | + $listener->handle($event); |
|
64 | + |
|
65 | + $this->assertTrue(true); // No exception means pass |
|
66 | + } |
|
67 | + |
|
68 | + public function testHandleWithLoadAdditionalScriptsEvent(): void { |
|
69 | + $listener = new LoadAdditionalListener(); |
|
70 | + |
|
71 | + $this->shareManager->method('shareApiEnabled')->willReturn(false); |
|
72 | + $this->factory->method('findLanguage')->willReturn('language_mock'); |
|
73 | + $this->config->method('getSystemValueBool')->willReturn(true); |
|
74 | + |
|
75 | + $this->overwriteService(IManager::class, $this->shareManager); |
|
76 | + $this->overwriteService(IFactory::class, $this->factory); |
|
77 | + $this->overwriteService(InitialStateService::class, $this->initialStateService); |
|
78 | + $this->overwriteService(IConfig::class, $this->config); |
|
79 | + |
|
80 | + $scriptsBefore = Util::getScripts(); |
|
81 | + $this->assertNotContains('files_sharing/l10n/language_mock', $scriptsBefore); |
|
82 | + $this->assertNotContains('files_sharing/js/additionalScripts', $scriptsBefore); |
|
83 | + $this->assertNotContains('files_sharing/js/init', $scriptsBefore); |
|
84 | + $this->assertNotContains('files_sharing/css/icons', \OC_Util::$styles); |
|
85 | + |
|
86 | + // Util static methods can't be easily mocked, so just ensure no exceptions |
|
87 | + $listener->handle($this->event); |
|
88 | + |
|
89 | + // assert array $scripts contains the expected scripts |
|
90 | + $scriptsAfter = Util::getScripts(); |
|
91 | + $this->assertContains('files_sharing/l10n/language_mock', $scriptsAfter); |
|
92 | + $this->assertContains('files_sharing/js/additionalScripts', $scriptsAfter); |
|
93 | + $this->assertNotContains('files_sharing/js/init', $scriptsAfter); |
|
94 | + |
|
95 | + $this->assertContains('files_sharing/css/icons', \OC_Util::$styles); |
|
96 | + } |
|
97 | + |
|
98 | + public function testHandleWithLoadAdditionalScriptsEventWithShareApiEnabled(): void { |
|
99 | + $listener = new LoadAdditionalListener(); |
|
100 | + |
|
101 | + $this->shareManager->method('shareApiEnabled')->willReturn(true); |
|
102 | + $this->config->method('getSystemValueBool')->willReturn(true); |
|
103 | + |
|
104 | + $this->overwriteService(IManager::class, $this->shareManager); |
|
105 | + $this->overwriteService(InitialStateService::class, $this->initialStateService); |
|
106 | + $this->overwriteService(IConfig::class, $this->config); |
|
107 | + $this->overwriteService(IFactory::class, $this->factory); |
|
108 | + |
|
109 | + $scriptsBefore = Util::getScripts(); |
|
110 | + $this->assertNotContains('files_sharing/js/init', $scriptsBefore); |
|
111 | + |
|
112 | + // Util static methods can't be easily mocked, so just ensure no exceptions |
|
113 | + $listener->handle($this->event); |
|
114 | + |
|
115 | + $scriptsAfter = Util::getScripts(); |
|
116 | + |
|
117 | + // assert array $scripts contains the expected scripts |
|
118 | + $this->assertContains('files_sharing/js/init', $scriptsAfter); |
|
119 | + } |
|
120 | 120 | } |
@@ -8,15 +8,15 @@ |
||
8 | 8 | require_once __DIR__ . '/../lib/base.php'; |
9 | 9 | |
10 | 10 | function enableApp($app) { |
11 | - (new \OC_App())->enable($app); |
|
12 | - echo "Enabled application {$app}\n"; |
|
11 | + (new \OC_App())->enable($app); |
|
12 | + echo "Enabled application {$app}\n"; |
|
13 | 13 | } |
14 | 14 | |
15 | 15 | foreach (new \DirectoryIterator(__DIR__ . '/../apps/') as $file) { |
16 | - if ($file->isDot()) { |
|
17 | - continue; |
|
18 | - } |
|
19 | - if (!file_exists($file->getPathname() . '/.git')) { |
|
20 | - enableApp($file->getFilename()); |
|
21 | - } |
|
16 | + if ($file->isDot()) { |
|
17 | + continue; |
|
18 | + } |
|
19 | + if (!file_exists($file->getPathname() . '/.git')) { |
|
20 | + enableApp($file->getFilename()); |
|
21 | + } |
|
22 | 22 | } |
@@ -5,18 +5,18 @@ |
||
5 | 5 | * SPDX-License-Identifier: AGPL-3.0-or-later |
6 | 6 | */ |
7 | 7 | |
8 | -require_once __DIR__ . '/../lib/base.php'; |
|
8 | +require_once __DIR__.'/../lib/base.php'; |
|
9 | 9 | |
10 | 10 | function enableApp($app) { |
11 | 11 | (new \OC_App())->enable($app); |
12 | 12 | echo "Enabled application {$app}\n"; |
13 | 13 | } |
14 | 14 | |
15 | -foreach (new \DirectoryIterator(__DIR__ . '/../apps/') as $file) { |
|
15 | +foreach (new \DirectoryIterator(__DIR__.'/../apps/') as $file) { |
|
16 | 16 | if ($file->isDot()) { |
17 | 17 | continue; |
18 | 18 | } |
19 | - if (!file_exists($file->getPathname() . '/.git')) { |
|
19 | + if (!file_exists($file->getPathname().'/.git')) { |
|
20 | 20 | enableApp($file->getFilename()); |
21 | 21 | } |
22 | 22 | } |