Check that master key is returned when querying device for first time #1166
Conversation
|
There seem to be a mix of tabs vs. spaces going on here. |
WHY WOULD YOU DO THIS TO ME VIM WHY 😭 |
lib/SyTest/Homeserver.pm
Outdated
| @@ -241,7 +241,7 @@ sub configure_logger | |||
|
|
|||
| loggers => { | |||
| synapse => { | |||
| level => "INFO" | |||
There was a problem hiding this comment.
Was this just for temporary debugging?
There was a problem hiding this comment.
Oh blast, I carefully didn't add this, and then clearly did a git commit -a.
|
https://github.com/matrix-org/sytest/runs/4091934296?check_suite_focus=true Looks like #1164. Edit: yes, it does. Bugger. https://github.com/matrix-org/sytest/runs/4091934438?check_suite_focus=true failed to install dendrite. |
|
/me waits to see how else he has messed up this PR |
| assert_json_keys( $content->{master_keys}, $user2_id ); | ||
| assert_json_keys( $content->{master_keys}->{$user2_id}, "keys"); | ||
| assert_json_keys( $content->{master_keys}->{$user2_id}{keys}, | ||
| "ed25519:nqOvzeuGWT/sRx3h7+MHoInYj3Uk2LD/unI9kDYcHwk"); |
There was a problem hiding this comment.
Should we also expect to see this in the second call to matrix_get_e2e_keys on +685?
There was a problem hiding this comment.
Yes we should. I suppose its worth checking that its still there
See matrix-org/synapse#11234