From 26548d677f5de4794614927f0e9914207ad00c33 Mon Sep 17 00:00:00 2001
From: Mike Stemle <hello@mikestemle.com>
Date: Thu, 8 Feb 2024 09:01:40 -0500
Subject: [PATCH] Fixed `issues().delete_comment_reaction()` (#566)

- Added a test for 204 and 500 cases
- Fixed broken functionality in the 204 case.
---
 src/api/issues.rs                             |  4 +-
 .../issues_comments_reactions_delete_test.rs  | 80 +++++++++++++++++++
 2 files changed, 83 insertions(+), 1 deletion(-)
 create mode 100644 tests/issues_comments_reactions_delete_test.rs

diff --git a/src/api/issues.rs b/src/api/issues.rs
index c4a40505..90ffb1a9 100644
--- a/src/api/issues.rs
+++ b/src/api/issues.rs
@@ -1089,6 +1089,8 @@ impl<'octo> IssueHandler<'octo> {
             reaction_id = reaction_id.into(),
         );
 
-        self.crab.delete(route, None::<&()>).await
+        crate::map_github_error(self.crab._delete(route, None::<&()>).await?).await?;
+
+        Ok(())
     }
 }
diff --git a/tests/issues_comments_reactions_delete_test.rs b/tests/issues_comments_reactions_delete_test.rs
new file mode 100644
index 00000000..96c1a2ab
--- /dev/null
+++ b/tests/issues_comments_reactions_delete_test.rs
@@ -0,0 +1,80 @@
+mod mock_error;
+
+use mock_error::setup_error_handler;
+use octocrab::Octocrab;
+use wiremock::{
+    matchers::{method, path},
+    Mock, MockServer, ResponseTemplate,
+};
+
+async fn setup_remove_comment_reaction_api(template: ResponseTemplate) -> MockServer {
+    let owner: &str = "org";
+    let repo: &str = "some-repo";
+    let comment_id: u64 = 123;
+    let reaction_id: u64 = 456;
+
+    let mock_server = MockServer::start().await;
+
+    Mock::given(method("DELETE"))
+        .and(path(format!(
+            "/repos/{owner}/{repo}/issues/comments/{comment_id}/reactions/{reaction_id}"
+        )))
+        .respond_with(template.clone())
+        .mount(&mock_server)
+        .await;
+
+    setup_error_handler(
+        &mock_server,
+        &format!("DELETE on /repos/{owner}/{repo}/issues/comments/{comment_id}/reactions/{reaction_id} was not received"),
+    )
+    .await;
+    mock_server
+}
+
+fn setup_octocrab(uri: &str) -> Octocrab {
+    Octocrab::builder().base_uri(uri).unwrap().build().unwrap()
+}
+
+const OWNER: &str = "org";
+const REPO: &str = "some-repo";
+const COMMENT_ID: u64 = 123;
+const REACTION_ID: u64 = 456;
+
+#[tokio::test]
+async fn should_delete_comment_reaction() {
+    let template = ResponseTemplate::new(204);
+    let mock_server = setup_remove_comment_reaction_api(template).await;
+    let client = setup_octocrab(&mock_server.uri());
+
+    let issues = client.issues(OWNER.to_owned(), REPO.to_owned());
+
+    let result = issues
+        .delete_comment_reaction(COMMENT_ID.to_owned(), REACTION_ID.to_owned())
+        .await;
+
+    assert!(
+        result.is_ok(),
+        "expected successful result, got error: {:#?}",
+        result
+    );
+}
+
+#[tokio::test]
+// Quick negative test, just to make sure errors still report properly
+async fn should_delete_comment_reaction_500() {
+    let template = ResponseTemplate::new(500);
+    let mock_server = setup_remove_comment_reaction_api(template).await;
+    let client = setup_octocrab(&mock_server.uri());
+
+    let issues = client.issues(OWNER.to_owned(), REPO.to_owned());
+
+    let result = issues
+        .delete_comment_reaction(COMMENT_ID.to_owned(), REACTION_ID.to_owned())
+        .await;
+
+    assert!(
+        result.is_err(),
+        "expected error result, got success somehow: {:#?}",
+        result
+    );
+}