Viewer should be always enabled

Signed-off-by: Roeland Jago Douma <roeland@famdouma.nl>

core/shipped.json

@@ -52,6 +52,7 @@
     "oauth2",
     "settings",
     "twofactor_backupcodes",
+    "viewer",
     "workflowengine"
   ]
 }

tests/lib/App/AppManagerTest.php

@@ -390,6 +390,7 @@ class AppManagerTest extends TestCase {
 			'test1',
 			'test3',
 			'twofactor_backupcodes',
+			'viewer',
 			'workflowengine',
 		];
 		$this->assertEquals($apps, $this->manager->getInstalledApps());
@@ -418,6 +419,7 @@ class AppManagerTest extends TestCase {
 			'test1',
 			'test3',
 			'twofactor_backupcodes',
+			'viewer',
 			'workflowengine',
 		];
 		$this->assertEquals($enabled, $this->manager->getEnabledAppsForUser($user));
@@ -444,6 +446,7 @@ class AppManagerTest extends TestCase {
 			'testnoversion' => ['id' => 'testnoversion', 'requiremin' => '8.2.0'],
 			'settings' => ['id' => 'settings'],
 			'twofactor_backupcodes' => ['id' => 'twofactor_backupcodes'],
+			'viewer' => ['id' => 'viewer'],
 			'workflowengine' => ['id' => 'workflowengine'],
 			'oauth2' => ['id' => 'oauth2'],
 		];
@@ -494,6 +497,7 @@ class AppManagerTest extends TestCase {
 			'twofactor_backupcodes' => ['id' => 'twofactor_backupcodes'],
 			'workflowengine' => ['id' => 'workflowengine'],
 			'oauth2' => ['id' => 'oauth2'],
+			'viewer' => ['id' => 'viewer'],
 		];
 
 		$manager->expects($this->any())
@@ -537,6 +541,7 @@ class AppManagerTest extends TestCase {
 			'test1',
 			'test3',
 			'twofactor_backupcodes',
+			'viewer',
 			'workflowengine',
 		];
 		$this->assertEquals($enabled, $this->manager->getEnabledAppsForGroup($group));