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

Disable broken cross-signing test #1095

Merged
merged 3 commits into from
Nov 28, 2019
Merged

Disable broken cross-signing test #1095

merged 3 commits into from
Nov 28, 2019

Conversation

turt2live
Copy link
Member

I don't know why it's broken, but the two errors I can get out of it are "unknown device for verification" and "user_signing key does not match". Someone who knows a bit more about cross-signing will probably need to take a look at this one.

Fixes element-hq/element-web#11520 (technically)
Opened element-hq/element-web#11545 to fix this correctly. After hours of trying to fix it, I'm giving up and disabling it in the interest of time.

I don't know why it's broken, but the two errors I can get out of it are "unknown device for verification" and "user_signing key does not match". Someone who knows a bit more about cross-signing will probably need to take a look at this one.

Fixes element-hq/element-web#11520 (technically)
Opened element-hq/element-web#11545 to fix this correctly.
@turt2live turt2live requested a review from a team November 28, 2019 02:22
@t3chguy
Copy link
Member

t3chguy commented Nov 28, 2019

Can we not mark it as xit to mark to skip?

@turt2live
Copy link
Member Author

Sure, though it doesn't seem to matter in the long run.

@turt2live turt2live merged commit a6f7936 into develop Nov 28, 2019
@turt2live turt2live deleted the travis/fix-tests-18 branch November 28, 2019 16:11
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.

js-sdk tests are hosed
2 participants