Make long_press works with 'duration' parameter.

ianxiaohanxu · ianxiaohanxu · commit 258f67066a42 · 2014-09-09T09:25:18.000+08:00
Add a new parameter 'duration = None' to _get_opts
diff --git a/appium/webdriver/common/touch_action.py b/appium/webdriver/common/touch_action.py
@@ -57,7 +57,7 @@ def press(self, el=None, x=None, y=None):
     def long_press(self, el=None, x=None, y=None, duration=1000):
         """Begin a chain with a press down that lasts `duration` milliseconds
         """
-        self._add_action('longPress', self._get_opts(el, x, y))
+        self._add_action('longPress', self._get_opts(el, x, y, duration))
 
         return self
 
@@ -112,7 +112,7 @@ def _add_action(self, action, options):
         }
         self._actions.append(gesture)
 
-    def _get_opts(self, element, x, y):
+    def _get_opts(self, element, x, y, duration = None):
         opts = {}
         if element is not None:
             opts['element'] = element.id
@@ -122,6 +122,9 @@ def _get_opts(self, element, x, y):
             x, y = None, None
         opts['x'] = x
         opts['y'] = y
+        
+        if duration is not None:
+            opts['duration'] = duration
 
         return opts
 
