fix tests

LycheeOrg · Feb 17, 2025 · 9afb01f · 9afb01f
1 parent 4dedc4a
commit 9afb01f
Show file tree

Hide file tree

Showing 4 changed files with 5 additions and 4 deletions.
diff --git a/app/Metadata/Cache/RouteCacheManager.php b/app/Metadata/Cache/RouteCacheManager.php
@@ -75,6 +75,7 @@ public function __construct()
 			'api/v2/Map' => new RouteCacheConfig(tag: CacheTag::GALLERY, user_dependant: true, extra: [RequestAttribute::ALBUM_ID_ATTRIBUTE]),
 			'api/v2/Map::provider' => new RouteCacheConfig(tag: CacheTag::SETTINGS),
 			'api/v2/Oauth' => new RouteCacheConfig(tag: CacheTag::USER, user_dependant: true),
+			'api/v2/Oauth::providers' => new RouteCacheConfig(tag: CacheTag::SETTINGS),
 			'api/v2/WebAuthn' => new RouteCacheConfig(tag: CacheTag::USER, user_dependant: true),
 
 			// Response must be different for each call.

diff --git a/resources/js/services/oauth-service.ts b/resources/js/services/oauth-service.ts
@@ -3,7 +3,7 @@ import Constants from "./constants";
 
 const OauthService = {
 	listProviders(): Promise<AxiosResponse<App.Enum.OauthProvidersType[]>> {
-		return axios.get(`${Constants.getApiUrl()}OauthProviders`, { data: {} });
+		return axios.get(`${Constants.getApiUrl()}Oauth::providers`, { data: {} });
 	},
 
 	list(): Promise<AxiosResponse<App.Http.Resources.Oauth.OauthRegistrationData[]>> {

diff --git a/routes/api_v2.php b/routes/api_v2.php
@@ -172,7 +172,7 @@
 // This route returns different results depending whether we are authenticated or not:
 // If Authenticated: list of the registrated Oauth providers
 // If not Authenticated: list of the available Oauth providers
-Route::get('/OauthProviders', [OauthController::class, 'listProviders']);
+Route::get('/Oauth::providers', [OauthController::class, 'listProviders'])->middleware(['cache_control']);
 Route::get('/Oauth', [OauthController::class, 'listForUser']);
 Route::delete('/Oauth', [OauthController::class, 'clear']);
 

diff --git a/tests/Feature_v2/Oauth/OauthTest.php b/tests/Feature_v2/Oauth/OauthTest.php
@@ -29,7 +29,7 @@ public function testGetAnonymous(): void
 		Config::set('services.github.client_secret', 'something');
 		Config::set('services.github.redirect', 'something');
 
-		$response = $this->getJson('OauthProviders');
+		$response = $this->getJson('Oauth::providers');
 		$this->assertOk($response);
 		$response->assertJson(['github']);
 
@@ -46,7 +46,7 @@ public function testUser(): void
 		Config::set('services.github.client_secret', 'something');
 		Config::set('services.github.redirect', 'something');
 
-		$response = $this->actingAs($this->userMayUpload1)->getJson('OauthProviders');
+		$response = $this->actingAs($this->userMayUpload1)->getJson('Oauth::providers');
 		$this->assertOk($response);
 		$response->assertJson(['github']);