Skip to content

Improve logging for a failing test#709

Merged
richvdh merged 1 commit intodevelopfrom
rav/diagnostics
Sep 24, 2019
Merged

Improve logging for a failing test#709
richvdh merged 1 commit intodevelopfrom
rav/diagnostics

Conversation

@richvdh
Copy link
Member

@richvdh richvdh commented Sep 24, 2019

This test died in a surprising way. Attempt to make the logging make a bit more
sense.

This test died in a surprising way. Attempt to make the logging make a bit more
sense.
@richvdh richvdh requested a review from a team September 24, 2019 11:34
Copy link
Contributor

@neilisfragile neilisfragile left a comment

Choose a reason for hiding this comment

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

Lgtm

@richvdh richvdh merged commit 8191623 into develop Sep 24, 2019
@richvdh richvdh deleted the rav/diagnostics branch September 24, 2019 13:53
anoadragon453 added a commit that referenced this pull request Mar 18, 2020
…ase-v1.4.0

* origin/release-v1.4.0: (36 commits)
  Improve logging for a failing test (#709)
  Make 3PID binding tests use /account/3pid/bind ala MSC2290 (#703)
  Use unstable prefix for 3PID unbind API
  Add support for handling email validation challenges (#707)
  do requestToken before adding a 3pid (#706)
  Configure synapse to use the test mail server (#705)
  Implement a mail server to help with 3pid testing (#704)
  federated_rooms_fixture (#701)
  Use Sytest develop for Dendrite's master branch (#700)
  Don't assume atomicity
  Fix flakiness due to create_room_synced (#702)
  Don't require an avatar_url
  Replace HOMESERVER_INFO incantations (#699)
  add some logging for flaky sync test (#698)
  Fix problems when using hash
  Accept access_token instead of id_access_token on the IS, accept Authorization header (#697)
  Use hash and handle variable sized final chunk
  Have the in-built identity server support v2 (#689)
  Add tests for 3PID /unbind API (#691)
  Fix typo
  ...
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.

3 participants