Update song queue changes (#145)

* Updated gitignore to disregard some .env files

.env.docker and .env.local. Primarily used for switching between local and docker development

* Changed PATCH /api/v2/song/queue/{id} endpoint

Modified the response body. Will just return a list of song queue id

* Adding some helpful info for the response body

* Tidying up and making some changes

* Code formatting
This commit was merged in pull request #145.
This commit is contained in:
KD
2025-06-28 16:45:25 -04:00
committed by GitHub
parent fbec3305b1
commit 9c67bfd8a8
3 changed files with 13 additions and 3 deletions
+2
View File
@@ -2,6 +2,8 @@
/target /target
.env .env
.env.docker
.env.local
.DS_STORE .DS_STORE
Storage/ Storage/
+4 -3
View File
@@ -123,7 +123,7 @@ pub mod response {
#[derive(Default, serde::Deserialize, serde::Serialize)] #[derive(Default, serde::Deserialize, serde::Serialize)]
pub struct Response { pub struct Response {
pub message: String, pub message: String,
pub data: Vec<Vec<u8>>, pub data: Vec<uuid::Uuid>,
} }
} }
@@ -743,8 +743,9 @@ pub mod endpoint {
let raw_data: Vec<u8> = data.to_vec(); let raw_data: Vec<u8> = data.to_vec();
match song_queue::update(&pool, &raw_data, &id).await { match song_queue::update(&pool, &raw_data, &id).await {
Ok(queued_data) => { Ok(_) => {
response.data.push(queued_data); response.message = String::from("Successful");
response.data.push(id);
(axum::http::StatusCode::OK, axum::Json(response)) (axum::http::StatusCode::OK, axum::Json(response))
} }
Err(err) => { Err(err) => {
+7
View File
@@ -822,6 +822,13 @@ mod tests {
resp.data.is_empty(), resp.data.is_empty(),
"Should not be empty" "Should not be empty"
); );
let updated_song_queued_id = resp.data[0];
assert_eq!(
updated_song_queued_id, *id,
"Song queue Id should match, but they don't. {:?} {:?}",
updated_song_queued_id, id
);
} }
Err(err) => { Err(err) => {
assert!(false, "Error: {:?}", err); assert!(false, "Error: {:?}", err);