feat(readarr): delete a book and expose the command on the CLI
The delete flags travel as query parameters rather than a request body, and Readarr spells the exclusion key addImportListExclusion, not the addImportExclusion that Radarr uses.
This commit is contained in:
6 files changed
+228
-3
No files matched your search
@@ -4,7 +4,7 @@ use serde_json::{Value, json};
|
||||
|
||||
use crate::models::Route;
|
||||
use crate::models::readarr_models::{
|
||||
Book, BookFile, Edition, ReadarrCommandBody, ReadarrHistoryItem,
|
||||
Book, BookFile, DeleteParams, Edition, ReadarrCommandBody, ReadarrHistoryItem,
|
||||
};
|
||||
use crate::models::servarr_data::readarr::readarr_data::ActiveReadarrBlock;
|
||||
use crate::network::readarr_network::ReadarrEvent;
|
||||
@@ -15,6 +15,38 @@ use crate::network::{Network, RequestMethod};
|
||||
mod readarr_books_network_tests;
|
||||
|
||||
impl Network<'_, '_> {
|
||||
pub(in crate::network::readarr_network) async fn delete_book(
|
||||
&mut self,
|
||||
delete_book_params: DeleteParams,
|
||||
) -> Result<()> {
|
||||
let event = ReadarrEvent::DeleteBook(DeleteParams::default());
|
||||
let DeleteParams {
|
||||
id,
|
||||
delete_files,
|
||||
add_import_list_exclusion,
|
||||
} = delete_book_params;
|
||||
|
||||
info!(
|
||||
"Deleting Readarr book with ID: {id} with deleteFiles={delete_files} and addImportListExclusion={add_import_list_exclusion}"
|
||||
);
|
||||
|
||||
let request_props = self
|
||||
.request_props_from(
|
||||
event,
|
||||
RequestMethod::Delete,
|
||||
None::<()>,
|
||||
Some(format!("/{id}")),
|
||||
Some(format!(
|
||||
"deleteFiles={delete_files}&addImportListExclusion={add_import_list_exclusion}"
|
||||
)),
|
||||
)
|
||||
.await;
|
||||
|
||||
self
|
||||
.handle_request::<(), ()>(request_props, |_, _| ())
|
||||
.await
|
||||
}
|
||||
|
||||
pub(in crate::network::readarr_network) async fn get_book_details(
|
||||
&mut self,
|
||||
book_id: i64,
|
||||
|
||||
@@ -1,7 +1,7 @@
|
||||
#[cfg(test)]
|
||||
mod tests {
|
||||
use crate::models::readarr_models::{
|
||||
Book, BookFile, Edition, ReadarrHistoryItem, ReadarrSerdeable,
|
||||
Book, BookFile, DeleteParams, Edition, ReadarrHistoryItem, ReadarrSerdeable,
|
||||
};
|
||||
use crate::models::servarr_data::readarr::modals::BookDetailsModal;
|
||||
use crate::models::servarr_data::readarr::readarr_data::ActiveReadarrBlock;
|
||||
@@ -18,6 +18,60 @@ mod tests {
|
||||
use rstest::rstest;
|
||||
use serde_json::{Value, json};
|
||||
|
||||
#[rstest]
|
||||
#[case(true, false, "deleteFiles=true&addImportListExclusion=false")]
|
||||
#[case(false, true, "deleteFiles=false&addImportListExclusion=true")]
|
||||
#[tokio::test]
|
||||
async fn test_handle_delete_book_event(
|
||||
#[case] delete_files: bool,
|
||||
#[case] add_import_list_exclusion: bool,
|
||||
#[case] expected_query: &str,
|
||||
) {
|
||||
let delete_book_params = DeleteParams {
|
||||
id: 1,
|
||||
delete_files,
|
||||
add_import_list_exclusion,
|
||||
};
|
||||
let (mock, app, _server) = MockServarrApi::delete()
|
||||
.path("/1")
|
||||
.query(expected_query)
|
||||
.build_for(ReadarrEvent::DeleteBook(delete_book_params.clone()))
|
||||
.await;
|
||||
app.lock().await.server_tabs.set_index(3);
|
||||
let mut network = test_network(&app);
|
||||
|
||||
let result = network
|
||||
.handle_readarr_event(ReadarrEvent::DeleteBook(delete_book_params))
|
||||
.await;
|
||||
|
||||
mock.assert_async().await;
|
||||
assert_ok!(result);
|
||||
}
|
||||
|
||||
#[tokio::test]
|
||||
async fn test_handle_delete_book_event_failure() {
|
||||
let delete_book_params = DeleteParams {
|
||||
id: 1,
|
||||
delete_files: true,
|
||||
add_import_list_exclusion: false,
|
||||
};
|
||||
let (mock, app, _server) = MockServarrApi::delete()
|
||||
.path("/1")
|
||||
.query("deleteFiles=true&addImportListExclusion=false")
|
||||
.status(500)
|
||||
.build_for(ReadarrEvent::DeleteBook(delete_book_params.clone()))
|
||||
.await;
|
||||
app.lock().await.server_tabs.set_index(3);
|
||||
let mut network = test_network(&app);
|
||||
|
||||
let result = network
|
||||
.handle_readarr_event(ReadarrEvent::DeleteBook(delete_book_params))
|
||||
.await;
|
||||
|
||||
mock.assert_async().await;
|
||||
assert_err!(result);
|
||||
}
|
||||
|
||||
#[tokio::test]
|
||||
async fn test_handle_get_book_details_event() {
|
||||
let expected_book: Book = serde_json::from_str(BOOK_JSON).unwrap();
|
||||
|
||||
@@ -28,6 +28,7 @@ pub enum ReadarrEvent {
|
||||
AddRootFolder(AddReadarrRootFolderBody),
|
||||
AddTag(String),
|
||||
DeleteAuthor(DeleteParams),
|
||||
DeleteBook(DeleteParams),
|
||||
DeleteRootFolder(i64),
|
||||
DeleteTag(i64),
|
||||
EditAuthor(EditAuthorParams),
|
||||
@@ -72,7 +73,8 @@ impl NetworkResource for ReadarrEvent {
|
||||
| ReadarrEvent::ListAuthors
|
||||
| ReadarrEvent::ToggleAuthorMonitoring(_) => "/author",
|
||||
ReadarrEvent::SearchNewAuthor(_) => "/author/lookup",
|
||||
ReadarrEvent::GetBookDetails(_)
|
||||
ReadarrEvent::DeleteBook(_)
|
||||
| ReadarrEvent::GetBookDetails(_)
|
||||
| ReadarrEvent::GetBooks(_)
|
||||
| ReadarrEvent::ToggleBookMonitoring(_) => "/book",
|
||||
ReadarrEvent::GetBookFiles(_) => "/bookfile",
|
||||
@@ -123,6 +125,10 @@ impl Network<'_, '_> {
|
||||
.delete_author(delete_author_params)
|
||||
.await
|
||||
.map(ReadarrSerdeable::from),
|
||||
ReadarrEvent::DeleteBook(delete_book_params) => self
|
||||
.delete_book(delete_book_params)
|
||||
.await
|
||||
.map(ReadarrSerdeable::from),
|
||||
ReadarrEvent::DeleteRootFolder(root_folder_id) => self
|
||||
.delete_readarr_root_folder(root_folder_id)
|
||||
.await
|
||||
|
||||
@@ -22,6 +22,7 @@ mod tests {
|
||||
#[case(ReadarrEvent::ListAuthors, "/author")]
|
||||
#[case(ReadarrEvent::ToggleAuthorMonitoring(1), "/author")]
|
||||
#[case(ReadarrEvent::SearchNewAuthor(String::new()), "/author/lookup")]
|
||||
#[case(ReadarrEvent::DeleteBook(DeleteParams::default()), "/book")]
|
||||
#[case(ReadarrEvent::GetBookDetails(1), "/book")]
|
||||
#[case(ReadarrEvent::GetBooks(1), "/book")]
|
||||
#[case(ReadarrEvent::ToggleBookMonitoring(1), "/book")]
|
||||
|
||||
Reference in new issue
Block a user