Skip to content

Commit

Permalink
Fixed issues().delete_comment_reaction() (#566)
Browse files Browse the repository at this point in the history
- Added a test for 204 and 500 cases
- Fixed broken functionality in the 204 case.
  • Loading branch information
manchicken authored Feb 8, 2024
1 parent 255a1cf commit 26548d6
Show file tree
Hide file tree
Showing 2 changed files with 83 additions and 1 deletion.
4 changes: 3 additions & 1 deletion src/api/issues.rs
Original file line number Diff line number Diff line change
Expand Up @@ -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(())
}
}
80 changes: 80 additions & 0 deletions tests/issues_comments_reactions_delete_test.rs
Original file line number Diff line number Diff line change
@@ -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
);
}

0 comments on commit 26548d6

Please sign in to comment.