Skip to content

test: update deltachat 2.50 ci compatibility#989

Closed
ccclxxiii wants to merge 6 commits into
mainfrom
ccclxxxiii/2.50-regression-parity-fix
Closed

test: update deltachat 2.50 ci compatibility#989
ccclxxiii wants to merge 6 commits into
mainfrom
ccclxxxiii/2.50-regression-parity-fix

Conversation

@ccclxxiii
Copy link
Copy Markdown
Collaborator

@ccclxxiii ccclxxiii commented May 28, 2026

summary

this pr updates relay ci/test behavior for compatibility with deltachat rpc/server 2.50.0.

changes

remove unsupported deltachat config

the cmdeploy test account setup previously configured:

account.set_config("delete_server_after", "10")

deltachat rpc/server 2.50.0 now rejects this key with:

unknown key "delete_server_after": matching variant not found

which caused account setup failures before tests could run.

the unsupported setting has been removed/commented out from the relay-side cmdeploy test setup.

update cmlxc workflow pin

the reusable cmlxc workflow reference is pinned to a newer commit containing the matching relay_minitest compatibility fixes, since the previously referenced v0.14.6 workflow still used the obsolete config.

improve sender ip cleanup test handling

test_hide_senders_ip_address has been updated to reduce timing sensitivity and mailbox visibility issues during raw imap inspection by:

  • enabling bcc_self
  • explicitly selecting INBOX
  • polling imap with a timeout
  • fetching using criteria="ALL"
  • inspecting the newest matching message

this allows the test to inspect a server-visible copy of the submitted message even after deltachat receive processing.

current status

the unsupported deltachat config issue now appears resolved in both cmdeploy and relay_minitest paths.

this pr also updates the sender ip cleanup tests to work with deltachat rpc/server 2.50 mailbox behavior observed during ci runs.

  • relevant pr on cmlxc with priority merge

@ccclxxiii ccclxxiii marked this pull request as ready for review May 28, 2026 15:35
@ccclxxiii ccclxxiii requested review from adbenitez, feld, hpk42, j4n, link2xt and missytake and removed request for hpk42 and missytake May 28, 2026 15:37
@link2xt link2xt removed their request for review May 28, 2026 15:39
@hpk42
Copy link
Copy Markdown
Contributor

hpk42 commented May 28, 2026

Please re-do the PR, and use human-written commit messages and PR description. Besides, we need a clear description or reference what exactly fails.

@hpk42 hpk42 closed this May 28, 2026
@ccclxxiii
Copy link
Copy Markdown
Collaborator Author

ccclxxiii commented May 28, 2026

Please re-do the PR, and use human-written commit messages and PR description. Besides, we need a clear description or reference what exactly fails.

commit message/etc were written by me, i'm not using ai/llm. i'm baffled why you keep making this mention.

@ccclxxiii ccclxxiii deleted the ccclxxxiii/2.50-regression-parity-fix branch May 28, 2026 16:28
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.

2 participants