Fix invite list command crashing when having more than one entry (#…

…9175)

This is a parcially cherry-pick of commit `100152ac72a262790468ceac444256ac25399c93`

Closes #9176

cli/tests/integration/invitations/list.rs

@@ -1,14 +1,29 @@
 use libparsec::{
     authenticated_cmds::v4::invite_new_device, get_default_config_dir, tmp_path, AuthenticatedCmds,
-    InvitationType, ParsecInvitationAddr, ProxyConfig, TmpPath,
+    InvitationToken, ProxyConfig, TmpPath,
 };
+use predicates::prelude::PredicateBooleanExt;
 
 use crate::{
     integration_tests::bootstrap_cli_test,
     testenv_utils::{TestOrganization, DEFAULT_DEVICE_PASSWORD},
     utils::{RESET, YELLOW},
 };
 
+async fn invite_device(cmds: &AuthenticatedCmds) -> InvitationToken {
+    let rep = cmds
+        .send(invite_new_device::Req { send_email: false })
+        .await
+        .unwrap();
+
+    match rep {
+        invite_new_device::InviteNewDeviceRep::Ok { token, .. } => token,
+        rep => {
+            panic!("Server refused to create device invitation: {rep:?}");
+        }
+    }
+}
+
 #[rstest::rstest]
 #[tokio::test]
 async fn list_invitations(tmp_path: TmpPath) {
@@ -21,24 +36,7 @@ async fn list_invitations(tmp_path: TmpPath) {
     )
     .unwrap();
 
-    let rep = cmds
-        .send(invite_new_device::Req { send_email: false })
-        .await
-        .unwrap();
-
-    let invitation_addr = match rep {
-        invite_new_device::InviteNewDeviceRep::Ok { token, .. } => ParsecInvitationAddr::new(
-            alice.organization_addr.clone(),
-            alice.organization_id().clone(),
-            InvitationType::Device,
-            token,
-        ),
-        rep => {
-            panic!("Server refused to create device invitation: {rep:?}");
-        }
-    };
-
-    let token = invitation_addr.token();
+    let token = invite_device(&cmds).await;
 
     crate::assert_cmd_success!(
         with_password = DEFAULT_DEVICE_PASSWORD,
@@ -53,6 +51,40 @@ async fn list_invitations(tmp_path: TmpPath) {
     )));
 }
 
+#[rstest::rstest]
+#[tokio::test]
+async fn issue_9176_list_more_than_one_invitations(tmp_path: TmpPath) {
+    let (_, TestOrganization { alice, .. }, _) = bootstrap_cli_test(&tmp_path).await.unwrap();
+
+    let cmds = AuthenticatedCmds::new(
+        &get_default_config_dir(),
+        alice.clone(),
+        ProxyConfig::new_from_env().unwrap(),
+    )
+    .unwrap();
+
+    let token1 = invite_device(&cmds).await;
+    let token2 = invite_device(&cmds).await;
+    let token3 = invite_device(&cmds).await;
+
+    let contains_invite = |token: InvitationToken| {
+        predicates::str::contains(format!("{token}\t{YELLOW}idle{RESET}\tdevice"))
+    };
+
+    crate::assert_cmd_success!(
+        with_password = DEFAULT_DEVICE_PASSWORD,
+        "invite",
+        "list",
+        "--device",
+        &alice.device_id.hex()
+    )
+    .stdout(
+        contains_invite(token1)
+            .and(contains_invite(token2))
+            .and(contains_invite(token3)),
+    );
+}
+
 #[rstest::rstest]
 #[tokio::test]
 async fn no_invitations(tmp_path: TmpPath) {

newsfragments/9176.bugfix.rst

@@ -0,0 +1 @@
+Fix CLI command ``invite list`` crashing when listing more than one entry