test: add failing test

Author: link2xt

src/chat.rs

@@ -4706,6 +4706,7 @@ mod tests {
     use crate::chatlist::get_archived_cnt;
     use crate::constants::{DC_GCL_ARCHIVED_ONLY, DC_GCL_NO_SPECIALS};
     use crate::headerdef::HeaderDef;
+    use crate::imex::{has_backup, imex, ImexMode};
     use crate::message::delete_msgs;
     use crate::receive_imf::receive_imf;
     use crate::test_utils::{sync, TestContext, TestContextManager, TimeShiftFalsePositiveNote};
@@ -7935,4 +7936,112 @@ mod tests {
 
         Ok(())
     }
+
+    /// Test the case when Alice restores a backup older than 60 days
+    /// with outdated member list.
+    #[tokio::test(flavor = "multi_thread", worker_threads = 2)]
+    async fn test_restore_backup_after_60_days() -> Result<()> {
+        let backup_dir = tempfile::tempdir()?;
+
+        let mut tcm = TestContextManager::new();
+
+        let alice = &tcm.alice().await;
+        let bob = &tcm.bob().await;
+        let fiona = &tcm.fiona().await;
+
+        let bob_addr = bob.get_config(Config::Addr).await?.unwrap();
+        let alice_bob_contact_id = Contact::create(alice, "Bob", &bob_addr).await?;
+
+        let charlie_addr = "[email protected]";
+        let alice_charlie_contact_id = Contact::create(alice, "Charlie", charlie_addr).await?;
+
+        let alice_chat_id =
+            create_group_chat(alice, ProtectionStatus::Unprotected, "Group chat").await?;
+        add_contact_to_chat(alice, alice_chat_id, alice_bob_contact_id).await?;
+        add_contact_to_chat(alice, alice_chat_id, alice_charlie_contact_id).await?;
+
+        let alice_sent_promote = alice
+            .send_text(alice_chat_id, "Hi! I created a group.")
+            .await;
+        let bob_rcvd_promote = bob.recv_msg(&alice_sent_promote).await;
+        let bob_chat_id = bob_rcvd_promote.chat_id;
+        bob_chat_id.accept(bob).await?;
+
+        // Alice exports a backup.
+        imex(alice, ImexMode::ExportBackup, backup_dir.path(), None).await?;
+
+        remove_contact_from_chat(alice, alice_chat_id, alice_charlie_contact_id).await?;
+        assert_eq!(get_chat_contacts(alice, alice_chat_id).await?.len(), 2);
+        assert_eq!(get_past_chat_contacts(alice, alice_chat_id).await?.len(), 1);
+
+        let remove_message = alice.pop_sent_msg().await;
+        assert_eq!(remove_message.payload.contains(&charlie_addr), true);
+        bob.recv_msg(&remove_message).await;
+
+        // 60 days pass.
+        SystemTime::shift(Duration::from_secs(60 * 24 * 60 * 60 + 1));
+
+        assert_eq!(get_past_chat_contacts(alice, alice_chat_id).await?.len(), 0);
+
+        // Bob adds Fiona to the chat.
+        let fiona_addr = fiona.get_config(Config::Addr).await?.unwrap();
+        let bob_fiona_contact_id = Contact::create(bob, "Fiona", &fiona_addr).await?;
+        add_contact_to_chat(bob, bob_chat_id, bob_fiona_contact_id).await?;
+
+        let add_message = bob.pop_sent_msg().await;
+        alice.recv_msg(&add_message).await;
+        let fiona_add_message = fiona.recv_msg(&add_message).await;
+        let fiona_chat_id = fiona_add_message.chat_id;
+
+        // Fiona does not learn about Charlie,
+        // even from `Chat-Group-Past-Members`, because tombstone has expired.
+        assert_eq!(get_chat_contacts(fiona, fiona_chat_id).await?.len(), 3);
+        assert_eq!(get_past_chat_contacts(fiona, fiona_chat_id).await?.len(), 0);
+
+        tcm.section("Alice imports old backup");
+        let alice = &tcm.unconfigured().await;
+        let backup = has_backup(alice, backup_dir.path()).await?;
+        imex(alice, ImexMode::ImportBackup, backup.as_ref(), None).await?;
+
+        // Alice thinks Charlie is in the chat, but does not know about Fiona.
+        assert_eq!(get_chat_contacts(alice, alice_chat_id).await?.len(), 3);
+        assert_eq!(get_past_chat_contacts(alice, alice_chat_id).await?.len(), 0);
+
+        assert_eq!(get_chat_contacts(bob, bob_chat_id).await?.len(), 3);
+        assert_eq!(get_past_chat_contacts(bob, bob_chat_id).await?.len(), 0);
+
+        assert_eq!(get_chat_contacts(fiona, fiona_chat_id).await?.len(), 3);
+        assert_eq!(get_past_chat_contacts(fiona, fiona_chat_id).await?.len(), 0);
+
+        // Bob sends a text message to the chat, without a tombstone for Charlie.
+        // Alice learns about Fiona.
+        let bob_sent_text = bob.send_text(alice_chat_id, "Message.").await;
+
+        tcm.section("Alice sends a message to stale chat");
+        let alice_sent_text = alice
+            .send_text(alice_chat_id, "Hi! I just restored a backup.")
+            .await;
+
+        tcm.section("Alice sent a message to stale chat");
+        alice.recv_msg(&bob_sent_text).await;
+        fiona.recv_msg(&bob_sent_text).await;
+
+        bob.recv_msg(&alice_sent_text).await;
+        fiona.recv_msg(&alice_sent_text).await;
+
+        // This should not add or restore Charlie for Bob and Fiona,
+        // Charlie is not part of the chat.
+        assert_eq!(get_chat_contacts(bob, bob_chat_id).await?.len(), 3);
+        assert_eq!(get_past_chat_contacts(bob, bob_chat_id).await?.len(), 0);
+
+        assert_eq!(get_chat_contacts(fiona, fiona_chat_id).await?.len(), 3);
+        assert_eq!(get_past_chat_contacts(fiona, fiona_chat_id).await?.len(), 0);
+
+        // Alice should have also learned about Charlie not being part of the group
+        // by receiving Bob's message.
+        assert_eq!(get_chat_contacts(bob, bob_chat_id).await?.len(), 3);
+        assert_eq!(get_past_chat_contacts(bob, bob_chat_id).await?.len(), 0);
+
+        Ok(())
+    }
 }