fix: Rename config option to skipAuthPickerApplications to match what it does

Signed-off-by: Côme Chilliet <come.chilliet@nextcloud.com>

Author: come-nc

apps/oauth2/lib/Controller/LoginRedirectorController.php

@@ -81,8 +81,8 @@ public function authorize($client_id,
 
 		$this->session->set('oauth.state', $state);
 
-		if (in_array($client->getName(), $this->appConfig->getValueArray('oauth2', 'autoGrantApplications', []))) {
-			/* See ClientFlowLoginController::showAuthPickerPage */
+		if (in_array($client->getName(), $this->appConfig->getValueArray('oauth2', 'skipAuthPickerApplications', []))) {
+			/** @see ClientFlowLoginController::showAuthPickerPage **/
 			$stateToken = $this->random->generate(
 				64,
 				ISecureRandom::CHAR_LOWER . ISecureRandom::CHAR_UPPER . ISecureRandom::CHAR_DIGITS

apps/oauth2/tests/Controller/LoginRedirectorControllerTest.php

@@ -85,7 +85,7 @@ public function testAuthorize(): void {
 		$this->assertEquals($expected, $this->loginRedirectorController->authorize('MyClientId', 'MyState', 'code'));
 	}
 
-	public function testAuthorizeSkipGrant(): void {
+	public function testAuthorizeSkipPicker(): void {
 		$client = new Client();
 		$client->setName('MyClientName');
 		$client->setClientIdentifier('MyClientIdentifier');
@@ -110,7 +110,7 @@ public function testAuthorizeSkipGrant(): void {
 		$this->appConfig
 			->expects(static::once())
 			->method('getValueArray')
-			->with('oauth2', 'autoGrantApplications', [])
+			->with('oauth2', 'skipAuthPickerApplications', [])
 			->willReturn(['MyClientName']);
 		$this->random
 			->expects(static::once())