Skip to content

test: avoid race in sender ip cleanup imap check#984

Closed
ccclxxiii wants to merge 3 commits into
mainfrom
ccclxxiii/ci-failure-investigation
Closed

test: avoid race in sender ip cleanup imap check#984
ccclxxiii wants to merge 3 commits into
mainfrom
ccclxxiii/ci-failure-investigation

Conversation

@ccclxxiii
Copy link
Copy Markdown
Collaborator

pr overview:

remove the wait_for_incoming_msg() call from test_hide_senders_ip_address and poll imap directly before delta chat consumes or moves the message.

this may fix the ci failure where:

assert msgs, "expected at least one message"

fails after the message has already been processed by the client.

changes

  • open imap connection before sending the message
  • remove user2.wait_for_incoming_msg()
  • poll mailbox.fetch(mark_seen=False) with a short timeout
  • preserve the existing sender ip cleanup assertion

notes

this is intended as a ci stabilization change for integration testing and should not affect runtime behavior.

@ccclxxiii
Copy link
Copy Markdown
Collaborator Author

closing while we wait on chatmail/cmlxc#19

@ccclxxiii ccclxxiii closed this May 28, 2026
@ccclxxiii ccclxxiii deleted the ccclxxiii/ci-failure-investigation branch May 28, 2026 14:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant