Closed jcoyne closed 12 years ago
@jcoyne This is ok but I'm concerned about the consequences of mocking 2I without also having integration tests. I've already started refactoring the test backend to support 2I, it should be done this week, then I can review this again and merge.
Sorry for the delay.
Thanks for submitting this, @jcoyne. I've added/tweaked a few things and pushed now that 2I support for the test server is in the client.
@jcoyne This is ok but I'm concerned about the consequences of mocking 2I without also having integration tests. I've already started refactoring the test backend to support 2I, it should be done this week, then I can review this again and merge.
Sorry for the delay.