From 5f120309f9f79f9e59e43437b72d98941953a62e Mon Sep 17 00:00:00 2001
From: Manos Pitsidianakis <manos@pitsidianak.is>
Date: Sat, 14 Sep 2024 12:52:52 +0300
Subject: [PATCH] nntp: add select_group_by_name() method

Add NntpConnection::select_group_by_name() method.

Signed-off-by: Manos Pitsidianakis <manos@pitsidianak.is>
---
 melib/src/nntp/connection.rs | 27 ++++++++++++++++++++++-----
 1 file changed, 22 insertions(+), 5 deletions(-)

diff --git a/melib/src/nntp/connection.rs b/melib/src/nntp/connection.rs
index d9b4c535..3c158c60 100644
--- a/melib/src/nntp/connection.rs
+++ b/melib/src/nntp/connection.rs
@@ -539,15 +539,32 @@ impl NntpConnection {
         force: bool,
         res: &mut String,
     ) -> Result<()> {
+        let path = self
+            .uid_store
+            .mailboxes
+            .lock()
+            .await
+            .get(&mailbox_hash)
+            .ok_or_else(|| Error::new("No such mailbox").set_kind(ErrorKind::NotFound))?
+            .name()
+            .to_string();
+        self.select_group_by_name(&path, force, res).await?;
+        Ok(())
+    }
+
+    pub async fn select_group_by_name(
+        &mut self,
+        path: &str,
+        force: bool,
+        res: &mut String,
+    ) -> Result<MailboxHash> {
+        let mailbox_hash = MailboxHash::from_bytes(path.as_bytes());
         if !force {
             match self.stream.as_ref()?.current_mailbox {
-                MailboxSelection::Select(m) if m == mailbox_hash => return Ok(()),
+                MailboxSelection::Select(m) if m == mailbox_hash => return Ok(mailbox_hash),
                 _ => {}
             }
         }
-        let path = self.uid_store.mailboxes.lock().await[&mailbox_hash]
-            .name()
-            .to_string();
         self.send_command(format!("GROUP {}", path).as_bytes())
             .await?;
         self.read_response(res, false, command_to_replycodes("GROUP"))
@@ -559,7 +576,7 @@ impl NntpConnection {
                 )
             })?;
         self.stream.as_mut()?.current_mailbox = MailboxSelection::Select(mailbox_hash);
-        Ok(())
+        Ok(mailbox_hash)
     }
 
     pub async fn send_multiline_data_block(&mut self, message: &[u8]) -> Result<()> {