Skip to content
This repository has been archived by the owner on Nov 3, 2021. It is now read-only.

Small fixes to marionette-apps. r=me #34287

Merged
merged 2 commits into from
May 5, 2016
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ marionette('Homescreen - Appcache installed download', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_blacklist_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ marionette('Homescreen - App Blacklist', function() {
system.waitForFullyLoaded();

client.apps.launch(home.URL);
home.waitForLaunch();

// Fail finding elements quickly.
client.setSearchTimeout(20);
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_default_order_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ marionette('Homescreen - Default app order', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Default app order is respected', function() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ marionette('Homescreen - Hosted app failed icon fetch', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_hosted_install_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ marionette('Homescreen - Hosted App Install', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ marionette('Homescreen - Hosted app failed icon fetch', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -25,7 +25,6 @@ marionette('Homescreen - Hosted app cached icon fetch', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_order_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ marionette('Homescreen - App order', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('App order is retained after restart', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_packaged_fail_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ marionette('Homescreen - Packaged App Failed Download', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ marionette('Homescreen - Packaged App Install', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ marionette('Homescreen - Packaged App Pending', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ marionette('Homescreen - Packaged App Pending Update', function() {
// wait for the system app to be running
system.waitForFullyLoaded();
client.apps.launch(home.URL);
home.waitForLaunch();

// install the app
client.switchToFrame();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -23,7 +23,6 @@ marionette('Homescreen - Packaged App Resuming Downloads', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();

done(err);
});
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ marionette('Homescreen - Packaged App Resume Update', function() {

// wait for the system app to be running
system.waitForFullyLoaded();
home.waitForLaunch();

// install the app
client.switchToFrame();
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -26,7 +26,6 @@ marionette('Homescreen - Packaged App Pending', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -31,7 +31,6 @@ marionette('Homescreen - Packaged App Update', function() {

// Launch the homescreen first, then go to the system app.
// Make sure we do this before installing an application.
home.waitForLaunch();
client.switchToFrame();

// install the app with a broken icon
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_reorder_hint_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ marionette('Homescreen - App reordering', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('App reordering has a visual hint', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_short_name_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -21,7 +21,6 @@ marionette('Homescreen - App manifest', function() {
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
client.apps.launch(home.URL);
home.waitForLaunch();
});

test('short_name', function() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -37,7 +37,6 @@ marionette('Homescreen - App uninstall while pending', function() {
// wait for the system app to be running
system.waitForFullyLoaded();
client.apps.launch(home.URL);
home.waitForLaunch();

// install the app
client.switchToFrame();
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/app_uninstall_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -33,7 +33,6 @@ marionette('Homescreen - App Uninstall', function() {
system.waitForFullyLoaded();

client.apps.launch(home.URL);
home.waitForLaunch();
});

function test_app_uninstall(manifestURL) {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ marionette('Homescreen - App unrecoverable error', function() {
appInstall = new AppInstall(client);

system.waitForFullyLoaded();
home.waitForLaunch();
});

teardown(function(done) {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/group_append_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ marionette('Homescreen - Group appending', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Appending to a group', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/group_create_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,6 @@ marionette('Homescreen - Group creation', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Creating a group', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/group_destroy_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -12,7 +12,6 @@ marionette('Homescreen - Group destruction', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Removing the penultimate icon in a group', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/group_launch_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,6 @@ marionette('Homescreen - Group launching', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Launching an icon in a group', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/group_order_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -17,7 +17,6 @@ marionette('Homescreen - Group ordering', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Reordering an icon in a group', function() {
Expand Down
3 changes: 0 additions & 3 deletions apps/homescreen/test/marionette/home_button_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ marionette('Homescreen - Home button', function() {
pinning = new Pinning(client);
bookmark = new Bookmark(client, server);
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('scroll to the top of the screen', function() {
Expand All @@ -55,15 +54,13 @@ marionette('Homescreen - Home button', function() {
system.dismissBanner();

system.tapHome();
home.waitForLaunch();

var icon = home.getIcon(url);

home.scrollIconToCenter(icon);
actions.longPress(icon, 1).perform();

system.tapHome();
home.waitForLaunch();

assert.equal(client.executeScript(function() {
return document.body.classList.contains('edit-mode');
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/layout_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ marionette('Homescreen - Layout', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('apps grid is LTR in LTR language', function() {
Expand Down
17 changes: 0 additions & 17 deletions apps/homescreen/test/marionette/lib/homescreen.js
Original file line number Diff line number Diff line change
Expand Up @@ -98,23 +98,6 @@ Homescreen.prototype = {
return this.client.findElement(Homescreen.Selectors.resumeDownload);
},

/**
* Waits for the homescreen to launch and switches to the frame.
*/
waitForLaunch: function() {
var client = this.client;
client.switchToFrame();
client.helper.waitForElement('body');
client.apps.switchToApp(Homescreen.URL);

// Wait for startup to complete
client.waitFor(function() {
return client.executeScript(function() {
return window.wrappedJSObject.appWindow.apps.startupMetadata === null;
});
});
},

/**
* Waits for the edit bar to fully appear.
*/
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/localization_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,6 @@ marionette('Homescreen - Localization', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Localization updates icon names', function() {
Expand Down
2 changes: 0 additions & 2 deletions apps/homescreen/test/marionette/pinning_edit_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -31,15 +31,13 @@ marionette('Homescreen - Pinned Site Edit', function() {
pinning = new Pinning(client);
bookmark = new Bookmark(client, server);
system.waitForFullyLoaded();
home.waitForLaunch();

url = server.url('sample.html');
client.switchToFrame();
pinning.openAndPinSiteFromBrowser(url);
system.dismissBanner();

system.tapHome();
home.waitForLaunch();
});

test('pressing enter after renaming the pinned site', function() {
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -32,7 +32,6 @@ marionette('Homescreen - Pinning Favicon Failure', function() {
system = client.loader.getAppClass('system');
pinning = new Pinning(client, server);
system.waitForFullyLoaded();
home.waitForLaunch();

url = server.url('sample.html');
client.switchToFrame();
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/pinning_order_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -29,7 +29,6 @@ marionette('Homescreen - Pin order', function() {
home = client.loader.getAppClass('homescreen');
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Pin order is retained after restart', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/pinning_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -27,7 +27,6 @@ marionette('Homescreen - Pinned Sites', function() {
pinning = new Pinning(client);
system = client.loader.getAppClass('system');
system.waitForFullyLoaded();
home.waitForLaunch();
});

test('Pinning a site adds pin to homescreen', function() {
Expand Down
1 change: 0 additions & 1 deletion apps/homescreen/test/marionette/pinning_the_web_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,6 @@ marionette('Homescreen - Pin the web', function() {
system = client.loader.getAppClass('system');
pinning = new Rocketbar(client);
system.waitForFullyLoaded();
home.waitForLaunch();
});

// Skip test since we are disabling pinning door hanger in 2.5
Expand Down
2 changes: 0 additions & 2 deletions apps/homescreen/test/marionette/pinning_uninstall_test.js
Original file line number Diff line number Diff line change
Expand Up @@ -34,15 +34,13 @@ marionette('Homescreen - Pinned Site Uninstall', function() {
pinning = new Pinning(client);
bookmark = new Bookmark(client);
system.waitForFullyLoaded();
home.waitForLaunch();

url = server.url('sample.html');
client.switchToFrame();
pinning.openAndPinSiteFromBrowser(url);
system.dismissBanner();

system.tapHome();
home.waitForLaunch();
});

test('removal of pinned site', function() {
Expand Down
60 changes: 0 additions & 60 deletions apps/homescreen/test/marionette/verticalhome_migration_test.js

This file was deleted.

Loading