CODEWITHSUNDEEP
promiseprotractor
jmcollin92
jmcollin92

Reputation: 2996

When should we use .then with Protractor Promise?

I've got many instability with Protractor, and I'm sure there is something I don't understand. Sometimes I need use the .then() when clicking on a button before continuing, sometimes it don't have any impact and I should not use .then() or the test failed.

I wonder when should I use the .then() callback when testing in Protractor ? Example :

createAccountForm = $('#form-create-account');
submitButton = createAccountForm.$('button[type=submit]');

browser.wait(EC.elementToBeClickable(submitButton), 5000);
submitButton.click(); // .then(function(){   <-- uncomment in the .then form

// find the confirmation message
var message = $('.alert-success');
browser.wait(EC.visibilityOf(message), 5000);
log.debug('After visibilityOf');

expect(message.isPresent()).to.be.eventually.true;
// }); --> uncomment when in .then form

When I use this form of test (without .then()) I see on browser that the click on the button is not done, the test continue with the following expect and then stop.

If I use the .then() form, the click on the button is done, and the test continue without error.

On other test, I don't need to use the then() callback when clicking on button.

So , when should I use the .then() and when not ?

Jean-Marc

Upvotes: 37

Views: 34604

Answers (2)

Sergey Pleshakov
Sergey Pleshakov

Reputation: 8948

Give yourself a favor, and avoid .then

Today, async/await is a lot more appropriate for handing promises

But in nutshell, open protractor API page https://www.protractortest.org/#/api, find the method you're about to use and see what it returns. If it says promise, just add await before calling it. Make sure, to make your wrapping function async

it('test case 1', async () => {
  await elem.click()
})

Upvotes: 0

jmcollin92
jmcollin92

Reputation: 2996

The answer of this question can be found in this post : http://spin.atomicobject.com/2014/12/17/asynchronous-testing-protractor-angular/

That is :

  1. Protractor enqueue all driver commands in the ControlFlow,
  2. when you need the result of a driver command you should use .then,
  3. when you don't need the result of a driver you can avoid .then but all following instructions must be enqueued in the ControlFlow else they will be run before commands in the queue leading to unpredictable result. So, if you want to run a non driver tests command, you should add it into the .then callback or wrap the test into a Promise and enqueue the test in the ControlFlow. See example below.

Here is an example of my test working without .then :

log.debug('test0');

// enqueue the click
submitButton.click(); 
var message = $('.alert-success'); 

// enqueue the wait for message to be visible  
browser.wait(EC.visibilityOf(message), 5000);  

log.debug('test1');

// enqueue a test
expect(message.isPresent()).to.be.eventually.true;
log.debug('test2');

// a function returning a promise that does an async test (check in MongoDB Collection)
var testAccount = function () {           
    var deferred = protractor.promise.defer();

    // Verify that an account has been created
    accountColl.find({}).toArray(function (err, accs) {
        log.debug('test5');
        expect(err).to.not.exist;
        log.debug('test6');
        expect(accs.length).to.equal(1);
        return deferred.fulfill();
    });
    return deferred.promise;
};

log.debug('test3');

// Enqueue the testAccount function
browser.controlFlow().execute(testAccount);  
log.debug('test4');

Output is now what we expect :

test0

test1

test2

test3

test4

test5

test6

Upvotes: 41

Related Questions