refactored specific PR usage; tests pending

XAMPPRocky · XAMPPRocky · Jul 29, 2024 · Mar 11, 2024 · Mar 11, 2024 · Mar 12, 2024
commit 377342b0838f048b2b56b3274075bc25884577fe
diff --git a/src/api/pulls.rs b/src/api/pulls.rs
@@ -2,10 +2,13 @@
 
 use http::request::Builder;
 use http::{Method, Uri};
+use serde_json::json;
 use snafu::ResultExt;
 
 use crate::error::HttpSnafu;
+use crate::models::pulls::ReviewComment;
 use crate::models::CommentId;
+use crate::pulls::specific_pr::pr_reviews::specific_review::SpecificReviewBuilder;
 use crate::pulls::specific_pr::SpecificPullRequestBuilder;
 use crate::{Octocrab, Page};
 
@@ -398,10 +401,63 @@ impl<'octo> PullRequestHandler<'octo> {
     /// * /repos/{owner}/{repo}/pulls/{pull_number}/reviews/{review_id}/dismissals
     /// * /repos/{owner}/{repo}/pulls/{pull_number}/comments/{comment_id}/replies
     ///
+    #[deprecated(
+        since = "0.34.4",
+        note = "specific PR builder transitioned to pr_review_actions, reply_to_comment, reply_to_comment"
+    )]
+    //FIXME: remove?
     pub fn pull_number(&self, pull_nr: u64) -> SpecificPullRequestBuilder {
         SpecificPullRequestBuilder::new(self, pull_nr)
     }
 
+    // /repos/{owner}/{repo}/pulls/{pull_number}/reviews/{review_id}/events
+    // /repos/{owner}/{repo}/pulls/{pull_number}/reviews/{review_id}
+    // repos/{owner}/{repo}/pulls/{pull_number}/reviews/{review_id}/comments
+    // repos/{owner}/{repo}/pulls/{pull_number}/reviews/{review_id}/dismissals
+    pub fn pr_review_actions(
+        &self,
+        pull_nr: u64,
+        review_id: u64,
+    ) -> SpecificReviewBuilder<'octo, '_> {
+        SpecificReviewBuilder::new(self, pull_nr, review_id)
+    }
+
+    /// /repos/{owner}/{repo}/pulls/{pull_number}/commits
+    // pub fn pr_commits(&self, pull_nr: u64) -> SpecificPullRequestCommentBuilder<'octo, '_> {
+    //     SpecificPullRequestCommentBuilder::new(self, pull_nr, 0)
+    // }
+
+    // /repos/{owner}/{repo}/pulls/{pull_number}/comments/{comment_id}/replies
+    /// Creates a reply to a specific comment of a pull request specified in the first argument
+    /// ```no_run
+    /// # use octocrab::models::CommentId;
+    ///  async fn run() -> octocrab::Result<()> {
+    /// # let octocrab = octocrab::Octocrab::default();
+    /// use octocrab::params;
+    ///
+    /// let page = octocrab.pulls("owner", "repo").reply_to_comment(142, CommentId(24), "This is my reply")
+    ///     .await?;
+    /// # Ok(())
+    /// # }
+    /// ```
+    pub async fn reply_to_comment(
+        &self,
+        pull_nr: u64,
+        comment_id: CommentId,
+        comment: impl Into<String>,
+    ) -> crate::Result<ReviewComment> {
+        let route = format!(
+            "/repos/{owner}/{repo}/pulls/{pull_number}/comments/{comment_id}/replies",
+            owner = self.owner,
+            repo = self.repo,
+            pull_number = pull_nr,
+            comment_id = comment_id
+        );
+        self.crab
+            .post(route, Some(&json!({ "body": comment.into() })))
+            .await
+    }
+
     /// Creates a new `MergePullRequestsBuilder` that can be configured used to
     /// merge a pull request.
     /// ```no_run

diff --git a/src/api/pulls/specific_pr.rs b/src/api/pulls/specific_pr.rs
@@ -6,7 +6,7 @@ use crate::pulls::PullRequestHandler;
 use crate::Page;
 
 mod pr_comment;
-mod pr_reviews;
+pub(crate) mod pr_reviews;
 /// A builder pattern struct for working with a specific pull request data,
 /// e.g. reviews, commits, comments, etc.
 ///

diff --git a/src/api/pulls/specific_pr/pr_reviews.rs b/src/api/pulls/specific_pr/pr_reviews.rs
@@ -1,7 +1,7 @@
 use crate::pulls::specific_pr::pr_reviews::specific_review::SpecificReviewBuilder;
 use crate::pulls::PullRequestHandler;
 
-mod specific_review;
+pub mod specific_review;
 
 #[derive(serde::Serialize)]
 pub struct ReviewsBuilder<'octo, 'b> {