Merge pull request #45 from groupon/pr-44

Implement driver click method

Author: jkrems

README.md

@@ -83,6 +83,7 @@ Method | Description
 `driver.setGeolocation(location)` | Set the browser's HTML5 geolocation. Expects `location` to be an object like `{latitude: 37.425, longitude: -122.136, altitude: 10.0}`. Not supported in all browsers.
 `driver.buttonDown(button)` | Presses down the pointer button (default 0 = left, can be 1 = middle, 2 = right) at location of last `element.movePointerRelativeTo()`
 `driver.buttonUp(button)` | Like `driver.buttonDown()`
+`driver.click(button)` | Like `driver.buttonDown()`
 `driver.close()` | Closes the WebDriver session.
 
 ### Element
@@ -177,7 +178,7 @@ Status | HTTP Method | Path  | Summary
 ![impl] | POST | `/session/:sessionId/accept_alert` |  Accepts the currently displayed alert dialog.
 ![impl] | POST | `/session/:sessionId/dismiss_alert` | Dismisses the currently displayed alert dialog.
 ![impl] | POST | `/session/:sessionId/moveto` |  Move the pointer by an offset of the specified element.
-![not-yet] | POST | `/session/:sessionId/click` | Click any pointer button (at the coordinates set by the last moveto command).
+![impl] | POST | `/session/:sessionId/click` | Click any pointer button (at the coordinates set by the last moveto command).
 ![impl] | POST | `/session/:sessionId/buttondown` |  Click and hold the left pointer button (at the coordinates set by the last moveto command).
 ![impl] | POST | `/session/:sessionId/buttonup` |  Releases the pointer button previously held (where the pointer is currently at).
 ![not-yet] | POST | `/session/:sessionId/doubleclick` | Double-clicks at the current pointer coordinates (set by moveto).

lib/element.js

@@ -171,7 +171,7 @@ module.exports = Element = (function() {
 })();
 
 elementOrNull = Element.elementOrNull = function(create) {
-  var error, error1;
+  var error;
   try {
     return create();
   } catch (error1) {

lib/index.js

@@ -113,7 +113,7 @@ module.exports = WebDriver = (function() {
   };
 
   WebDriver.prototype.evaluate = function(clientFunctionString) {
-    var args, error, error1, friendlyError, response;
+    var args, error, friendlyError, response;
     if (arguments.length > 1) {
       clientFunctionString = arguments[0], args = 2 <= arguments.length ? slice.call(arguments, 1) : [];
     }
@@ -135,7 +135,7 @@ module.exports = WebDriver = (function() {
   };
 
   WebDriver.prototype.evaluateAsync = function(clientFunctionString) {
-    var args, error, error1, friendlyError, response;
+    var args, error, friendlyError, response;
     if (arguments.length > 1) {
       clientFunctionString = arguments[0], args = 2 <= arguments.length ? slice.call(arguments, 1) : [];
     }

lib/json.js

@@ -31,7 +31,7 @@ NEGLIGENCE OR OTHERWISE) ARISING IN ANY WAY OUT OF THE USE OF THIS
 SOFTWARE, EVEN IF ADVISED OF THE POSSIBILITY OF SUCH DAMAGE.
  */
 module.exports.tryParse = function(jsonString) {
-  var error, error1;
+  var error;
   try {
     return JSON.parse(jsonString);
   } catch (error1) {

lib/pointer_api.js

@@ -47,6 +47,14 @@ module.exports = function(http) {
       http.post('/buttonup', {
         button: button
       });
+    },
+    click: function(button) {
+      if (button == null) {
+        button = 0;
+      }
+      http.post('/click', {
+        button: button
+      });
     }
   };
 };

lib/request.js

@@ -89,7 +89,7 @@ module.exports = function(arg) {
     connectTimeout = CONNECT_TIMEOUT;
   }
   return function(url, method, data) {
-    var body, elapsed, err, error, headers, ref1, requestBody, response, startTime;
+    var body, elapsed, err, headers, ref1, requestBody, response, startTime;
     if (method == null) {
       method = 'GET';
     }

lib/session.js

@@ -39,7 +39,7 @@ json = require('./json');
 parseResponseData = require('./parse_response');
 
 module.exports = function(request, serverUrl, desiredCapabilities) {
-  var capabilities, data, error, error1, response, sessionId, url;
+  var capabilities, data, error, response, sessionId, url;
   url = serverUrl + "/session";
   data = {
     desiredCapabilities: desiredCapabilities

src/pointer_api.coffee

@@ -37,3 +37,7 @@ module.exports = (http) ->
   buttonUp: (button=0) ->
     http.post '/buttonup', { button }
     return
+
+  click: (button=0) ->
+    http.post '/click', { button }
+    return