From 4ca009b9f7e8cd09e36362d4b8defc417b24d89c Mon Sep 17 00:00:00 2001 From: Richard van der Hoff Date: Wed, 14 Jun 2017 18:00:02 +0100 Subject: [PATCH] Remove onload simulator from loading test This used to exist to reproduce the functionality in the window.onload handler in vector/index.js -- which got removed in 31915db. Having it here is confusing at best. --- test/app-tests/loading.js | 31 +++---------------------------- 1 file changed, 3 insertions(+), 28 deletions(-) diff --git a/test/app-tests/loading.js b/test/app-tests/loading.js index 0132be3ce0..80fcc0202b 100644 --- a/test/app-tests/loading.js +++ b/test/app-tests/loading.js @@ -99,21 +99,14 @@ describe('loading:', function () { toString: function() { return this.search + this.hash; }, }; - let lastLoadedScreen = null; - let appLoaded = false; - let loadCompleteDefer = q.defer(); loadCompletePromise = loadCompleteDefer.promise; function onNewScreen(screen) { console.log(Date.now() + " newscreen "+screen); - if (!appLoaded) { - lastLoadedScreen = screen; - } else { - var hash = '#/' + screen; - windowLocation.hash = hash; - console.log(Date.now() + " browser URI now "+ windowLocation); - } + var hash = '#/' + screen; + windowLocation.hash = hash; + console.log(Date.now() + " browser URI now "+ windowLocation); } // Parse the given window.location and return parameters that can be used when calling @@ -126,13 +119,6 @@ describe('loading:', function () { } } - function routeUrl(location, matrixChat) { - console.log(Date.now() + ` routing URL '${location}'`); - const s = getScreenFromLocation(location); - console.log("Showing screen ", s); - matrixChat.showScreen(s.screen, s.params); - } - const MatrixChat = sdk.getComponent('structures.MatrixChat'); const fragParts = parseQsFromFragment(windowLocation); var params = parseQs(windowLocation); @@ -151,17 +137,6 @@ describe('loading:', function () { makeRegistrationUrl={() => {throw new Error('Not implemented');}} />, parentDiv ); - - // pause for a cycle, then simulate the window.onload handler - window.setTimeout(() => { - console.log(Date.now() + " simulating window.onload"); - routeUrl(windowLocation, matrixChat); - appLoaded = true; - if (lastLoadedScreen) { - onNewScreen(lastLoadedScreen); - lastLoadedScreen = null; - } - }, 0); } // set an expectation that we will get a call to /sync, then flush