Skip to content

Commit 5c35c26

Browse files
tylerhawkesclaude
andcommitted
test(xmtp_mls): run concurrent DM-invite streaming on native d14n
`test_many_concurrent_dm_invites` was ignored on all d14n via `#[cfg_attr(all(feature = "d14n"), ignore)]` with no rationale — but it passes on the native d14n backend (5- and 100-concurrent-DM cases; verified twice, the 100-DM case ~30s against its 120s budget). The ignore was over-broad: only d14n+wasm needs skipping (100 concurrent streaming clients are impractical there), matching the heavy group tests' `all(feature = "d14n", target_arch = "wasm32")` gating. Narrow it so native d14n gains the concurrent-streaming coverage. Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com> Claude-Session: https://claude.ai/code/session_017AVuJ7VAgen2fFhtJQ6zid
1 parent d813333 commit 5c35c26

1 file changed

Lines changed: 3 additions & 1 deletion

File tree

crates/xmtp_mls/src/subscriptions/stream_conversations.rs

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -649,7 +649,9 @@ mod test {
649649
#[case::onehundred_dms(100)]
650650
#[xmtp_common::test]
651651
#[awt]
652-
#[cfg_attr(all(feature = "d14n"), ignore)]
652+
// Runs on native d14n (verified); still skipped on d14n+wasm, where 100
653+
// concurrent streaming clients are impractical (matches the heavy group tests).
654+
#[cfg_attr(all(feature = "d14n", target_arch = "wasm32"), ignore)]
653655
async fn test_many_concurrent_dm_invites(#[future] alix: ClientTester, #[case] dms: usize) {
654656
let alix_inbox_id = Arc::new(alix.inbox_id().to_string());
655657
let mut clients = vec![];

0 commit comments

Comments
 (0)