I forgot we have tests

also thank you utoipa

Author: DrVilepis

src/api/leaderboards.rs

@@ -133,7 +133,7 @@ pub struct LeaderboardInvite {
 }
 
 #[utoipa::path(
-    get,
+    post,
     path = "/leaderboards/join",
     security(
         ("api_key" = [])

src/main.rs

@@ -193,10 +193,14 @@ fn create_router_with_openapi(config: &TestaustimeConfig) -> (Router, utoipa::op
                 .routes(routes!(api::users::get_current_activity))
                 .routes(routes!(api::users::get_activity_summary))
                 .routes(routes!(api::leaderboards::create_leaderboard))
-                .routes(routes!(api::leaderboards::get_leaderboard))
+                .routes(routes!(
+                    // utoipa/axum will not resolve these correctly based on method if registered
+                    // seperately, go figure
+                    api::leaderboards::get_leaderboard,
+                    api::leaderboards::delete_leaderboard
+                ))
                 .routes(routes!(api::leaderboards::join_leaderboard))
                 .routes(routes!(api::leaderboards::leave_leaderboard))
-                .routes(routes!(api::leaderboards::delete_leaderboard))
                 .routes(routes!(api::leaderboards::promote_member))
                 .routes(routes!(api::leaderboards::demote_member))
                 .routes(routes!(api::leaderboards::kick_member))

src/tests/leaderboards.rs

@@ -2,7 +2,7 @@ use serde_json::json;
 
 use super::{macros::*, *};
 use crate::{
-    api::leaderboards::{LeaderboardInvite, LeaderboardName},
+    api::leaderboards::{LeaderboardCreateRequest, LeaderboardInvite},
     models::{NewUserIdentity, PrivateLeaderboard},
 };
 
@@ -23,7 +23,7 @@ async fn creation_joining_and_deletion() {
 
     let member: NewUserIdentity = body_to_json(resp).await;
 
-    let create = LeaderboardName {
+    let create = LeaderboardCreateRequest {
         name: "board".to_string(),
     };
 

src/tests/mod.rs

@@ -17,7 +17,7 @@ use tower::ServiceExt;
 // NOTE: We would like to use diesels Connection::begin_test_transaction
 // But cannot use them because our database uses transactions to implement
 // some of the routes and there cannot exists transactions within transactions :'(
-use crate::create_router;
+use crate::create_router_with_openapi;
 
 const TEST_ADDR: SocketAddr = SocketAddr::V4(SocketAddrV4::new(Ipv4Addr::new(127, 0, 0, 1), 3000));
 
@@ -41,7 +41,9 @@ fn create_test_router() -> Router {
         mail_password: "".to_string(),
     };
 
-    create_router(&config).layer(MockConnectInfo(SocketAddr::from(([127, 0, 0, 1], 3000))))
+    create_router_with_openapi(&config)
+        .0
+        .layer(MockConnectInfo(SocketAddr::from(([127, 0, 0, 1], 3000))))
 }
 
 #[tokio::test]