Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test for sending to_device messages to multiple recipients at once #856

Merged
merged 2 commits into from
Apr 30, 2020

Conversation

richvdh
Copy link
Member

@richvdh richvdh commented Apr 20, 2020

I thought we had a bug here, but it turns out we don't. Still, may as well keep
the test.

I thought we had a bug here, but it turns out we don't. Still, may as well keep
the test.
@richvdh richvdh requested a review from a team April 20, 2020 18:29
Copy link
Member

@anoadragon453 anoadragon453 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm other than the below and my general lack of perl knowledge.

tests/46direct/01directmessage.pl Outdated Show resolved Hide resolved
tests/46direct/01directmessage.pl Outdated Show resolved Hide resolved
tests/46direct/01directmessage.pl Outdated Show resolved Hide resolved
Co-Authored-By: Andrew Morgan <1342360+anoadragon453@users.noreply.github.com>
@richvdh richvdh merged commit 9eea0ad into develop Apr 30, 2020
@richvdh richvdh deleted the rav/multi_dest_to_device branch April 30, 2020 16:04
anoadragon453 added a commit that referenced this pull request Jun 10, 2020
…into dinsic

* 'release-v1.13.0' of https://github.com/matrix-org/sytest: (23 commits)
  Dendrite: Show sytest compliance report after sytest runs (#864)
  Provide some examples of running sytest with worker mode synapse (#862)
  Fix build, don't build into source directory
  Maybe this time
  Better
  Just build the monolith server when testing
  Test for sending to_device messages to multiple recipients at once (#856)
  Mention required debian and cpan dependencies (#859)
  Add whitelist information to dendrite buildkite annotation (#803)
  Fix OFFLINE support for the sytest-synapse docker image (#858)
  Add support for running worker mode against redis (#857)
  Add tests for SSO + user interactive authentication. (#845)
  Modify login tests to be spec compliant
  Make message history pagination test less flaky (#851)
  Small comment fixes for consistency (#808)
  Change room_fixture to run matrix_create_room_synced by default (#848)
  Update dendrite docs with new whitelist/blacklist info (#849)
  Add test for querying remote public rooms list with local server_name (#847)
  Add more logging to "Guest user can set display names" (#846)
  Add test for updating room aliases on upgraded room join (#844)
  ...
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