add new test to revoke the oauth token. it should be the last one to run

Signed-off-by: Julien Veyssier <julien-nc@posteo.net>

Author: julien-nc

tests/integration/GitHubIssuePrReferenceIntegrationTest.php

@@ -12,7 +12,6 @@
 use OCA\Github\Reference\GithubIssuePrReferenceProvider;
 use OCA\Github\Service\SecretService;
 use OCP\Collaboration\Reference\IReference;
-use OCP\IConfig;
 use OCP\Server;
 use PHPUnit\Framework\Attributes\DependsExternal;
 use PHPUnit\Framework\Attributes\Group;
@@ -22,14 +21,12 @@
 class GitHubIssuePrReferenceIntegrationTest extends TestCase {
 	private GithubIssuePrReferenceProvider $referenceProvider;
 	private SecretService $secretService;
-	private IConfig $config;
 
 	protected function setUp(): void {
 		parent::setUp();
 
 		$this->referenceProvider = Server::get(GithubIssuePrReferenceProvider::class);
 		$this->secretService = Server::get(SecretService::class);
-		$this->config = Server::get(IConfig::class);
 	}
 
 	#[DependsExternal(GithubOauthIntegrationTest::class, 'testOAuthLogin')]

tests/integration/GitHubNotificationsIntegrationTest.php

@@ -11,7 +11,6 @@
 
 use OCA\Github\Service\GithubAPIService;
 use OCA\Github\Service\SecretService;
-use OCP\IConfig;
 use OCP\Server;
 use PHPUnit\Framework\Attributes\DependsExternal;
 use PHPUnit\Framework\Attributes\Group;
@@ -21,14 +20,12 @@
 class GitHubNotificationsIntegrationTest extends TestCase {
 	private GithubAPIService $githubAPIService;
 	private SecretService $secretService;
-	private IConfig $config;
 
 	protected function setUp(): void {
 		parent::setUp();
 
 		$this->githubAPIService = Server::get(GithubAPIService::class);
 		$this->secretService = Server::get(SecretService::class);
-		$this->config = Server::get(IConfig::class);
 	}
 
 	#[DependsExternal(GithubOauthIntegrationTest::class, 'testOAuthLogin')]

tests/integration/GitHubSearchIntegrationTest.php

@@ -11,7 +11,6 @@
 
 use OCA\Github\Service\GithubAPIService;
 use OCA\Github\Service\SecretService;
-use OCP\IConfig;
 use OCP\Server;
 use PHPUnit\Framework\Attributes\DependsExternal;
 use PHPUnit\Framework\Attributes\Group;
@@ -21,14 +20,12 @@
 class GitHubSearchIntegrationTest extends TestCase {
 	private GithubAPIService $githubAPIService;
 	private SecretService $secretService;
-	private IConfig $config;
 
 	protected function setUp(): void {
 		parent::setUp();
 
 		$this->githubAPIService = Server::get(GithubAPIService::class);
 		$this->secretService = Server::get(SecretService::class);
-		$this->config = Server::get(IConfig::class);
 	}
 
 	#[DependsExternal(GithubOauthIntegrationTest::class, 'testOAuthLogin')]

tests/integration/GithubZTokenRevocationIntegrationTest.php

@@ -0,0 +1,52 @@
+<?php
+
+declare(strict_types=1);
+
+/**
+ * SPDX-FileCopyrightText: 2026 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
+ */
+
+namespace OCA\Github\Tests\Integration;
+
+use OCA\Github\Controller\ConfigController;
+use OCA\Github\Service\SecretService;
+use OCP\Server;
+use PHPUnit\Framework\Attributes\DependsExternal;
+use PHPUnit\Framework\Attributes\Group;
+use Test\TestCase;
+
+#[Group('DB')]
+class GithubZTokenRevocationIntegrationTest extends TestCase {
+	private SecretService $secretService;
+	private ConfigController $configController;
+
+	protected function setUp(): void {
+		parent::setUp();
+
+		$this->secretService = Server::get(SecretService::class);
+		$this->configController = Server::get(ConfigController::class);
+	}
+
+	#[DependsExternal(GithubOauthIntegrationTest::class, 'testOAuthLogin')]
+	public function testRevokeToken(array $oauthData): void {
+		$this->assertIsArray($oauthData, 'oauthData should be an array from OAuth test');
+		$this->assertArrayHasKey('userId', $oauthData, 'oauthData must contain userId');
+		$userId = $oauthData['userId'];
+
+		self::loginAsUser($userId);
+
+		$token = $this->secretService->getEncryptedUserValue($userId, 'token');
+		$this->assertNotSame('', $token, 'Token should exist before revocation');
+		echo 'TOKEN is not empty ' .strlen($token);
+
+		$response = $this->configController->setConfig(['token' => '']);
+
+		$this->assertArrayHasKey('user_name', $response->getData(), 'Response should contain user_name');
+		$this->assertSame('', $response->getData()['user_name'], 'user_name should be empty after token revocation');
+
+		$tokenAfter = $this->secretService->getEncryptedUserValue($userId, 'token');
+		$this->assertSame('', $tokenAfter, 'Token should be empty after revocation');
+		echo 'after revoke, TOKEN is empty ' .strlen($tokenAfter);
+	}
+}