Documentation might need updating (Webdriver Promises)

[matfin]

Hi,

I'm making good use of this module but can I suggest the following changes are added to the documentation:

1. Regarding the promises as documented in the Selenium Webdriver JS documentation (on Google Code), they say to create a promise as follows:

var deferred = webdriver.promise.defer();

This is incorrect and I could not get it working. From looking at your source code it seems the following is correct:

var deferred = new webdriver.promise.Deferred();

2. In the Selenium Webdriver documentation (on Google Code), we are told to call the following to resolve a promise:

deferred.fulfill();

This is incorrect and the following should instead be called:

deferred.resolve();

Hope this helps others who have been stuck with these problems.

Thanks,

Matt

[bernardlv]

Doesn't work. I'm trying to make this work for hours now. It's just TypeErrors of "undefined" all the time.

var test = require('../testing'),
webdriver = require('..'),
until = webdriver.until,
By = webdriver.By,
driver;

test.before(function() {
driver = new webdriver.Builder()
.forBrowser('firefox')
.build();
});

[..]

errorsDefer = driver.promise.Deferred();
errorsDefer.resolve();

"TypeError: Cannot call method 'Deferred' of undefined"

errorsDefer = webdriver.promise.Deferred();
errorsDefer.resolve();

TypeError: Cannot call method 'resolve' of undefined

EDIT: Actually the only thing you say wouldn't work, worked. I just didn't know I need the other namespace.

errorsDefer = webdriver.promise.defer();
errorsDefer.fulfill();
return webdriver.promise.all(mainDefer);