xevion/Pac-Man
1
0
Fork 0
mirror of https://github.com/Xevion/Pac-Man.git synced 2025-12-06 07:15:41 -06:00
Code Issues Packages Projects Releases Wiki Activity

fix(tests): update all test routes to use /api prefix

Browse Source 
All API routes were moved under /api prefix as part of the unified
deployment architecture. Updated test files to reflect this change:

- basics.rs: Update root and auth/providers routes
- health.rs: Update health endpoint routes
- oauth.rs: Update all OAuth and auth callback routes, plus redirect locations
- sessions.rs: Update profile and logout routes

This fixes 9 failing tests that were expecting routes without the /api prefix.
This commit is contained in:
Ryan Walters
2025-11-02 19:51:52 -06:00
parent 45e6131121
commit 8f8f82630f
4 changed files with 16 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
pacman-server/tests/basics.rs
View File

@@ -7,7 +7,7 @@ use crate::common::{test_context, TestContext};
// A basic test of all the server's routes that aren't covered by other tests. // A basic test of all the server's routes that aren't covered by other tests.
#[tokio::test] #[tokio::test]
async fn test_basic_routes() { async fn test_basic_routes() {
let routes = vec!["/", "/auth/providers"]; let routes = vec!["/api/", "/api/auth/providers"];
for route in routes { for route in routes {
let TestContext { server, .. } = test_context().use_database(false).call().await; let TestContext { server, .. } = test_context().use_database(false).call().await;

4
pacman-server/tests/health.rs
View File

@@ -10,7 +10,7 @@ async fn test_health_endpoint() {
let TestContext { server, container, .. } = test_context().use_database(true).call().await; let TestContext { server, container, .. } = test_context().use_database(true).call().await;
// First, verify health endpoint works when database is healthy // First, verify health endpoint works when database is healthy
let response = server.get("/health").await; let response = server.get("/api/health").await;
assert_eq!(response.status_code(), 200); assert_eq!(response.status_code(), 200);
let health_json: serde_json::Value = response.json(); let health_json: serde_json::Value = response.json();
assert_eq!(health_json["ok"], true); assert_eq!(health_json["ok"], true);
@@ -19,7 +19,7 @@ async fn test_health_endpoint() {
drop(container); drop(container);
// Now verify health endpoint reports bad health // Now verify health endpoint reports bad health
let response = server.get("/health?force").await; let response = server.get("/api/health?force").await;
assert_eq!(response.status_code(), 503); // SERVICE_UNAVAILABLE assert_eq!(response.status_code(), 503); // SERVICE_UNAVAILABLE
let health_json: serde_json::Value = response.json(); let health_json: serde_json::Value = response.json();
assert_eq!(health_json["ok"], false); assert_eq!(health_json["ok"], false);

20
pacman-server/tests/oauth.rs
View File

@@ -32,7 +32,7 @@ async fn test_oauth_authorization_redirect() {
let TestContext { server, app_state, .. } = test_context().auth_registry(mock_registry).call().await; let TestContext { server, app_state, .. } = test_context().auth_registry(mock_registry).call().await;
let response = server.get("/auth/mock").await; let response = server.get("/api/auth/mock").await;
assert_eq!(response.status_code(), 303); assert_eq!(response.status_code(), 303);
assert_eq!(response.headers().get("location").unwrap(), "https://example.com/auth"); assert_eq!(response.headers().get("location").unwrap(), "https://example.com/auth");
@@ -63,9 +63,9 @@ async fn test_new_user_registration() {
let context = test_context().use_database(true).auth_registry(mock_registry).call().await; let context = test_context().use_database(true).auth_registry(mock_registry).call().await;
let response = context.server.get("/auth/mock/callback?code=a&state=b").await; let response = context.server.get("/api/auth/mock/callback?code=a&state=b").await;
assert_eq!(response.status_code(), 302); assert_eq!(response.status_code(), 302);
assert_eq!(response.headers().get("location").unwrap(), "/profile"); assert_eq!(response.headers().get("location").unwrap(), "/api/profile");
// Verify user and oauth_account were created // Verify user and oauth_account were created
let user = user_repo::find_user_by_email(&context.app_state.db, "new@example.com") let user = user_repo::find_user_by_email(&context.app_state.db, "new@example.com")
@@ -119,9 +119,9 @@ async fn test_existing_user_signin() {
.await .await
.unwrap(); .unwrap();
let response = context.server.get("/auth/mock/callback?code=a&state=b").await; let response = context.server.get("/api/auth/mock/callback?code=a&state=b").await;
assert_eq!(response.status_code(), 302, "Should sign in successfully"); assert_eq!(response.status_code(), 302, "Should sign in successfully");
assert_eq!(response.headers().get("location").unwrap(), "/profile"); assert_eq!(response.headers().get("location").unwrap(), "/api/profile");
// Verify no new user was created // Verify no new user was created
let users = sqlx::query("SELECT * FROM users") let users = sqlx::query("SELECT * FROM users")
@@ -169,7 +169,7 @@ async fn test_implicit_account_linking() {
let context = test_context().use_database(true).auth_registry(mock_registry).call().await; let context = test_context().use_database(true).auth_registry(mock_registry).call().await;
// Action 1: Sign in with provider-a, creating the initial user // Action 1: Sign in with provider-a, creating the initial user
let response1 = context.server.get("/auth/provider-a/callback?code=a&state=b").await; let response1 = context.server.get("/api/auth/provider-a/callback?code=a&state=b").await;
assert_eq!(response1.status_code(), 302); assert_eq!(response1.status_code(), 302);
let user = user_repo::find_user_by_email(&context.app_state.db, "shared@example.com") let user = user_repo::find_user_by_email(&context.app_state.db, "shared@example.com")
@@ -181,7 +181,7 @@ async fn test_implicit_account_linking() {
assert_eq!(providers1[0].provider, "provider-a"); assert_eq!(providers1[0].provider, "provider-a");
// Action 2: Sign in with provider-b // Action 2: Sign in with provider-b
let response2 = context.server.get("/auth/provider-b/callback?code=a&state=b").await; let response2 = context.server.get("/api/auth/provider-b/callback?code=a&state=b").await;
assert_eq!(response2.status_code(), 302); assert_eq!(response2.status_code(), 302);
// Assertions: No new user, but a new provider link // Assertions: No new user, but a new provider link
@@ -224,7 +224,7 @@ async fn test_unverified_email_creates_new_account() {
.await .await
.unwrap(); .unwrap();
let response = context.server.get("/auth/mock/callback?code=a&state=b").await; let response = context.server.get("/api/auth/mock/callback?code=a&state=b").await;
assert_eq!(response.status_code(), 302); assert_eq!(response.status_code(), 302);
// Should create a second user because the email wasn't trusted for linking // Should create a second user because the email wasn't trusted for linking
@@ -257,11 +257,11 @@ async fn test_logout_functionality() {
let context = test_context().use_database(true).auth_registry(mock_registry).call().await; let context = test_context().use_database(true).auth_registry(mock_registry).call().await;
// Sign in to establish a session // Sign in to establish a session
let response = context.server.get("/auth/mock/callback?code=a&state=b").await; let response = context.server.get("/api/auth/mock/callback?code=a&state=b").await;
assert_eq!(response.status_code(), 302); assert_eq!(response.status_code(), 302);
// Test that the logout handler clears the session cookie and redirects // Test that the logout handler clears the session cookie and redirects
let response = context.server.get("/logout").await; let response = context.server.get("/api/logout").await;
assert_eq!(response.status_code(), 302); assert_eq!(response.status_code(), 302);
assert!(response.headers().contains_key("location")); assert!(response.headers().contains_key("location"));

6
pacman-server/tests/sessions.rs
View File

@@ -40,16 +40,16 @@ async fn test_session_management() {
// 3. Make a request to the protected route WITH the session, expect success // 3. Make a request to the protected route WITH the session, expect success
let response = context let response = context
.server .server
.get("/profile") .get("/api/profile")
.add_cookie(Cookie::new(session::SESSION_COOKIE_NAME, token)) .add_cookie(Cookie::new(session::SESSION_COOKIE_NAME, token))
.await; .await;
assert_eq!(response.status_code(), 200); assert_eq!(response.status_code(), 200);
// 4. Sign out // 4. Sign out
let response = context.server.get("/logout").await; let response = context.server.get("/api/logout").await;
assert_eq!(response.status_code(), 302); // Redirect after logout assert_eq!(response.status_code(), 302); // Redirect after logout
// 5. Make a request to the protected route without a session, expect failure // 5. Make a request to the protected route without a session, expect failure
let response = context.server.get("/profile").await; let response = context.server.get("/api/profile").await;
assert_eq!(response.status_code(), 401); // Unauthorized without session assert_eq!(response.status_code(), 401); // Unauthorized without session
} }