chore: split different level API

williamfzc · Jun 15, 2019 · 875d72e · 875d72e
1 parent 855a840
commit 875d72e
Show file tree

Hide file tree

Showing 2 changed files with 9 additions and 10 deletions.
diff --git a/fitch/device.py b/fitch/device.py
@@ -144,6 +144,14 @@ def tap_point(self, target_point: typing.Sequence, duration: int = None):
 
         self.player.tap(target_point, duration=duration)
 
+    def tap_and_drag(self,
+                     point1: (list, tuple),
+                     point2: (list, tuple),
+                     duration: int = None,
+                     part: int = None):
+        self.player.long_tap(point1, no_up=True)
+        self.player.swipe(point1, point2, duration, part, no_down=True)
+
 
 @contextlib.contextmanager
 def safe_device(device_id: str) -> FDevice:

diff --git a/fitch/player.py b/fitch/player.py
@@ -25,6 +25,7 @@
 
 
 class ActionPlayer(object):
+    """ base, low level API here """
     def __init__(self, device_id: str):
         self.device_id = device_id
         self.mnt = MNTDevice(device_id)
@@ -73,16 +74,6 @@ def slow_swipe(self,
                    point2: (list, tuple)):
         self.swipe(point1, point2, duration=1, part=50)
 
-    # --- high level API below ---
-    def tap_and_drag(self,
-                     point1: (list, tuple),
-                     point2: (list, tuple),
-                     duration: int = None,
-                     part: int = None):
-
-        self.long_tap(point1, no_up=True)
-        self.swipe(point1, point2, duration, part, no_down=True)
-
 
 if __name__ == '__main__':
     ap = ActionPlayer('4df189487c7b6fef')