From 32b77d970b9cc82aefe232ef60a54265cf7b538c Mon Sep 17 00:00:00 2001 From: Tim Abbott Date: Mon, 20 Mar 2017 23:21:38 -0700 Subject: [PATCH] casper: Fix missing casper.then in subscriptions tests. --- .../casper_tests/05-subscriptions.js | 29 +++++++++++-------- 1 file changed, 17 insertions(+), 12 deletions(-) diff --git a/frontend_tests/casper_tests/05-subscriptions.js b/frontend_tests/casper_tests/05-subscriptions.js index 196ce50204..48434ec75b 100644 --- a/frontend_tests/casper_tests/05-subscriptions.js +++ b/frontend_tests/casper_tests/05-subscriptions.js @@ -99,20 +99,25 @@ casper.then(function () { true, "Rome is visible again"); }); -casper.waitUntilVisible('#stream_creation_form', function () { - casper.test.assertTextExists('Create stream', 'New stream creation panel'); - casper.fill('form#stream_creation_form', {stream_name: 'Waseemio', stream_description: 'Oimeesaw'}); - casper.click('input[value="Scotland"] ~ span'); - casper.click('input[value="cordelia@zulip.com"] ~ span'); - casper.click('input[value="othello@zulip.com"] ~ span'); - casper.click('form#stream_creation_form button.btn.btn-primary'); -}); - -casper.waitFor(function () { - return casper.evaluate(function () { - return $('.stream-name').is(':contains("Waseemio")'); +casper.then(function () { + casper.waitUntilVisible('#stream_creation_form', function () { + casper.test.assertTextExists('Create stream', 'New stream creation panel'); + casper.fill('form#stream_creation_form', {stream_name: 'Waseemio', stream_description: 'Oimeesaw'}); + casper.click('input[value="Scotland"] ~ span'); + casper.click('input[value="cordelia@zulip.com"] ~ span'); + casper.click('input[value="othello@zulip.com"] ~ span'); + casper.click('form#stream_creation_form button.btn.btn-primary'); }); }); + +casper.then(function () { + casper.waitFor(function () { + return casper.evaluate(function () { + return $('.stream-name').is(':contains("Waseemio")'); + }); + }); +}); + casper.then(function () { casper.test.info("User should be subscribed to stream Waseemio"); casper.test.assertSelectorHasText('.stream-name', 'Waseemio');