From d876e4482bd7f9ef026e250fb314e6c50327899d Mon Sep 17 00:00:00 2001 From: Kegan Dougal Date: Fri, 17 Feb 2017 14:32:08 +0000 Subject: [PATCH] Fix broken tests caused by adding IndexedDB support This test assumed that `/sync` would be called immediately after rendering `` but this isn't true in an IndexedDB world: it bounces via `store.startup()` first. It looks like the tests resolve this by adding `q.delay(1)` so that's what I've done: in the future it would be better to extend `HttpBackend` to have a `waitFor(req) Promise` function so we can removing timing from the tests. --- test/app-tests/joining.js | 5 ++++- 1 file changed, 4 insertions(+), 1 deletion(-) diff --git a/test/app-tests/joining.js b/test/app-tests/joining.js index 442b67c2d..30ef70db9 100644 --- a/test/app-tests/joining.js +++ b/test/app-tests/joining.js @@ -95,7 +95,10 @@ describe('joining a room', function () { }); var roomView; - httpBackend.flush().then(() => { + // wait for /sync to happen + return q.delay(1).then(() => { + return httpBackend.flush(); + }).then(() => { var roomDir = ReactTestUtils.findRenderedComponentWithType( matrixChat, RoomDirectory);