From 5e3ca1861dce7ddbba30dc8ef35cc1bbc6fcdeaf Mon Sep 17 00:00:00 2001 From: KD Date: Sat, 26 Apr 2025 16:11:46 -0400 Subject: [PATCH] Next queued song (#123) * Added code to fetch next queue item * Updated README * Give it a go * Minor docker compose change * Added docker-compose defaults * Code refactor * Removed data from being fetched * Added test * Formatting --- README.md | 4 ++- docker-compose.yaml | 7 ++-- src/callers/mod.rs | 1 + src/callers/song.rs | 82 ++++++++++++++++++++++++++++++++++++++++++++- src/main.rs | 82 +++++++++++++++++++++++++++++++++++++++++++++ 5 files changed, 172 insertions(+), 4 deletions(-) diff --git a/README.md b/README.md index 64ee5d9..ca22c83 100644 --- a/README.md +++ b/README.md @@ -12,4 +12,6 @@ docker compose build --ssh default api auth_api ``` Bring it up -docker compose up -d --force-recreate api auth_api \ No newline at end of file +``` +docker compose up -d --force-recreate api auth_api +``` diff --git a/docker-compose.yaml b/docker-compose.yaml index 2a77885..e840a51 100644 --- a/docker-compose.yaml +++ b/docker-compose.yaml @@ -3,7 +3,9 @@ version: '3.8' # Use a recent version services: # --- Web API --- api: - build: . # Tells docker-compose to build the Dockerfile in the current directory + build: # Tells docker-compose to build the Dockerfile in the current directory + context: . + ssh: ["default"] # Uses host's SSH agent container_name: icarus # Optional: Give the container a specific name ports: # Map host port 8000 to container port 3000 (adjust as needed) @@ -22,6 +24,7 @@ services: auth_api: build: context: ../icarus_auth # IMPORTANT: Relative path to the local checkout of your web API repo (containing the Dockerfile) + ssh: ["default"] # Uses host's SSH agent dockerfile: Dockerfile # Optional: Specify if your Dockerfile has a non-standard name container_name: auth_api restart: unless-stopped @@ -106,4 +109,4 @@ networks: main-api-network: driver: bridge auth-api-network: - driver: bridge \ No newline at end of file + driver: bridge diff --git a/src/callers/mod.rs b/src/callers/mod.rs index 5abbb9f..e6ff83a 100644 --- a/src/callers/mod.rs +++ b/src/callers/mod.rs @@ -3,5 +3,6 @@ pub mod song; pub mod endpoints { pub const QUEUESONG: &str = "/api/v2/song/queue"; + pub const NEXTQUEUESONG: &str = "/api/v2/song/queue/next"; pub const QUEUEMETADATA: &str = "/api/v2/song/metadata/queue"; } diff --git a/src/callers/song.rs b/src/callers/song.rs index 0f89887..d914a55 100644 --- a/src/callers/song.rs +++ b/src/callers/song.rs @@ -15,6 +15,16 @@ pub mod response { pub message: String, pub data: Vec, } + + pub mod fetch_queue_song { + use serde::{Deserialize, Serialize}; + + #[derive(Default, Deserialize, Serialize)] + pub struct Response { + pub message: String, + pub data: Vec, + } + } } mod song_queue { @@ -23,7 +33,7 @@ mod song_queue { pub mod status { pub const PENDING: &str = "pending"; // Will be used later on - pub const _PROCESSING: &str = "processing"; + pub const PROCESSING: &str = "processing"; pub const _DONE: &str = "done"; } @@ -32,6 +42,13 @@ mod song_queue { pub id: uuid::Uuid, } + #[derive(Debug, serde::Deserialize, serde::Serialize, sqlx::FromRow)] + pub struct SongQueue { + pub id: uuid::Uuid, + pub filename: String, + pub status: String, + } + pub async fn insert( pool: &sqlx::PgPool, data: &Vec, @@ -63,6 +80,48 @@ mod song_queue { Err(_err) => Err(sqlx::Error::RowNotFound), } } + + pub async fn get_most_recent_and_update(pool: &sqlx::PgPool) -> Result { + let result = sqlx::query( + r#" + UPDATE "songQueue" + SET status = $1 + WHERE id = ( + SELECT id FROM "songQueue" + WHERE status = $2 + ORDER BY id + FOR UPDATE SKIP LOCKED + LIMIT 1 + ) + RETURNING id, filename, status; + "#, + ) + .bind(status::PROCESSING) + .bind(status::PENDING) + .fetch_one(pool) + .await + .map_err(|e| { + eprintln!("Error inserting: {}", e); + }); + + match result { + Ok(row) => Ok(SongQueue { + id: row + .try_get("id") + .map_err(|_e| sqlx::Error::RowNotFound) + .unwrap(), + filename: row + .try_get("filename") + .map_err(|_e| sqlx::Error::RowNotFound) + .unwrap(), + status: row + .try_get("status") + .map_err(|_e| sqlx::Error::RowNotFound) + .unwrap(), + }), + Err(_err) => Err(sqlx::Error::RowNotFound), + } + } } pub mod endpoint { @@ -117,4 +176,25 @@ pub mod endpoint { (StatusCode::OK, Json(response)) } + + pub async fn fetch_queue_song( + axum::Extension(pool): axum::Extension, + ) -> ( + StatusCode, + Json, + ) { + let mut response = super::response::fetch_queue_song::Response::default(); + + match song_queue::get_most_recent_and_update(&pool).await { + Ok(item) => { + response.message = String::from("Successful"); + response.data.push(item); + (StatusCode::OK, Json(response)) + } + Err(err) => { + response.message = err.to_string(); + (StatusCode::BAD_REQUEST, Json(response)) + } + } + } } diff --git a/src/main.rs b/src/main.rs index aafcc52..f2d73dc 100644 --- a/src/main.rs +++ b/src/main.rs @@ -69,6 +69,10 @@ pub mod init { crate::callers::endpoints::QUEUESONG, post(crate::callers::song::endpoint::queue_song), ) + .route( + crate::callers::endpoints::NEXTQUEUESONG, + get(crate::callers::song::endpoint::fetch_queue_song), + ) .route( crate::callers::endpoints::QUEUEMETADATA, post(crate::callers::metadata::endpoint::queue_metadata), @@ -250,6 +254,84 @@ mod tests { let _ = db_mgr::drop_database(&tm_pool, &db_name).await; } + #[tokio::test] + async fn test_song_fetch_queue_item() { + let tm_pool = db_mgr::get_pool().await.unwrap(); + let db_name = db_mgr::generate_db_name().await; + + match db_mgr::create_database(&tm_pool, &db_name).await { + Ok(_) => { + println!("Success"); + } + Err(err) => { + assert!(false, "Error: {:?}", err); + } + } + + let pool = db_mgr::connect_to_db(&db_name).await.unwrap(); + db::migrations(&pool).await; + + let app = crate::init::routes() + .await + .layer(axum::Extension(pool)) + .layer(axum::extract::DefaultBodyLimit::max(1024 * 1024 * 1024)) + .layer(TimeoutLayer::new(Duration::from_secs(300))); + + // Create multipart form + let mut form = MultipartForm::default(); + let _ = form.add_file("flac", "tests/Machine_gun/track01.flac"); + + // Create request + let content_type = form.content_type(); + let body = MultipartBody::from(form); + let req = axum::http::Request::builder() + .method(axum::http::Method::POST) + .uri(crate::callers::endpoints::QUEUESONG) + .header(axum::http::header::CONTENT_TYPE, content_type) + .body(axum::body::Body::from_stream(body)) + .unwrap(); + + // Send request + match app.clone().oneshot(req).await { + Ok(response) => { + let body = axum::body::to_bytes(response.into_body(), usize::MAX) + .await + .unwrap(); + println!("Body: {:?}", body); + let resp: crate::callers::song::response::Response = + serde_json::from_slice(&body).unwrap(); + assert_eq!(false, resp.data.is_empty(), "Should not be empty"); + assert_eq!(false, resp.data[0].is_nil(), "Should not be empty"); + + let fetch_req = axum::http::Request::builder() + .method(axum::http::Method::GET) + .uri(crate::callers::endpoints::NEXTQUEUESONG) + .header(axum::http::header::CONTENT_TYPE, "application/json") + .body(axum::body::Body::empty()) + .unwrap(); + + match app.clone().oneshot(fetch_req).await { + Ok(response) => { + let body = axum::body::to_bytes(response.into_body(), usize::MAX) + .await + .unwrap(); + let resp: crate::callers::song::response::fetch_queue_song::Response = + serde_json::from_slice(&body).unwrap(); + assert_eq!(false, resp.data.is_empty(), "Should not be empty"); + } + Err(err) => { + assert!(false, "Error: {:?}", err); + } + } + } + Err(err) => { + assert!(false, "Error: {:?}", err); + } + }; + + let _ = db_mgr::drop_database(&tm_pool, &db_name).await; + } + #[tokio::test] async fn test_song_metadata_queue() { let tm_pool = db_mgr::get_pool().await.unwrap();