Skip to content

Remove chat test timing sensitivity

Nick Sellen requested to merge fix-chat-test-timing into master

What does this MR do?

Chat tests failed due to suspected timing sensitivity in https://gitlab.com/foodsharing-dev/foodsharing/-/jobs/34815006

This change should resolve that as it uses the code the rest of the tests do to wait for the websocket to be connected before proceeding.

How confident are you it won't break things if deployed?

Just a test, so very confident.

Links to related issues

Checklist

  • added a test, or explain why one is not needed/possible...
  • no unrelated changes
  • asked someone for a code review
  • joined #foodsharing-beta channel at https://slackin.yunity.org
  • added an entry to CHANGELOG.md (description, merge request link, username(s))
Edited by Nick Sellen

Merge request reports