updated all examples to use promises (done is a bad idea)

Author: John Doherty

step-definitions/buy-workwear-steps.js

@@ -1,33 +1,25 @@
 module.exports = function () {
 
-    this.Given(/^I am on the Mammoth Workwear home page$/, function (done) {
+    this.Given(/^I am on the Mammoth Workwear home page$/, function () {
 
         // load google
-        helpers.loadPage(page.mammothWorkwear.url).then(function(){
-            done();
-        });
+        return helpers.loadPage(page.mammothWorkwear.url);
     });
 
-    this.When(/^I click navigation item "([^"]*)"$/, function (linkTitle, done) {
+    this.When(/^I click navigation item "([^"]*)"$/, function (linkTitle) {
 
         // click an item in the search results via the google page object
-        page.mammothWorkwear.clickNavigationItem(linkTitle).then(function(){
-            done();
-        });
+        return page.mammothWorkwear.clickNavigationItem(linkTitle);
     });
 
-    this.Then(/^I click product item "([^"]*)"$/, function (productTitle, done) {
+    this.Then(/^I click product item "([^"]*)"$/, function (productTitle) {
 
         // click an item in the search results via the google page object
-        page.mammothWorkwear.clickProductItem(productTitle).then(function(){
-            done();
-        });
+        return page.mammothWorkwear.clickProductItem(productTitle);
     });
 
-    this.Then(/^I should see product detail with title "([^"]*)"$/, function (pageTitle, done) {
+    this.Then(/^I should see product detail with title "([^"]*)"$/, function (pageTitle) {
 
-        page.mammothWorkwear.titleContains(pageTitle).then(function(){
-            done();
-        });
+        return page.mammothWorkwear.titleContains(pageTitle);
     });
 };

step-definitions/google-search-steps.js

@@ -1,24 +1,22 @@
 module.exports = function () {
 
-    this.When(/^I search Google for "([^"]*)"$/, function (searchQuery, done) {
+    this.When(/^I search Google for "([^"]*)"$/, function (searchQuery) {
 
-        driver.get('http://www.google.com');
-        
-        var input = driver.findElement(by.name('q'));
-        
-        input.sendKeys(searchQuery);
-        input.sendKeys(selenium.Key.ENTER);
-
-        done();
+        return driver.get('http://www.google.com').then(function(){
+            return driver.findElement(by.name('q'));
+        })
+        .then(function(el){
+            return el.sendKeys(searchQuery + selenium.Key.ENTER);
+        });
     });
 
-    this.Then(/^I should see some results$/, function (done) {
+    this.Then(/^I should see some results$/, function () {
 
-        driver.wait(until.elementsLocated(by.css('div.g')), 10000);
-
-        driver.findElements(by.css('div.g')).then(function (elements) {
+        return driver.wait(until.elementsLocated(by.css('div.g')), 10000).then(function(){
+            return driver.findElements(by.css('div.g'));
+        })
+        .then(function (elements) {
             expect(elements.length).to.not.equal(0);
-            done();
-        });
+        });        
     });
 };