From 8b68b69e11c945c543d39e66d3a46b4bc5f0fb96 Mon Sep 17 00:00:00 2001
From: buyuxiang <347586493@qq.com>
Date: Wed, 30 Aug 2023 14:31:44 +0800
Subject: [PATCH] compatible with old swipe_to_tap_texts

---
 hrp/testcase.go | 18 +++++++++---------
 1 file changed, 9 insertions(+), 9 deletions(-)

diff --git a/hrp/testcase.go b/hrp/testcase.go
index 28dd5150..ee7f19b1 100644
--- a/hrp/testcase.go
+++ b/hrp/testcase.go
@@ -364,18 +364,18 @@ func convertCompatMobileStep(mobileStep *MobileStep) {
 	if mobileStep == nil {
 		return
 	}
-	if mobileStep.MobileAction.Method == uixt.ACTION_TapByCV {
-		uiTypes, _ := builtin.ConvertToStringSlice(mobileStep.MobileAction.Params)
-		options := mobileStep.MobileAction.ActionOptions
-		options.ScreenShotWithUITypes = append(options.ScreenShotWithUITypes, uiTypes...)
-	}
 	for i := 0; i < len(mobileStep.Actions); i++ {
 		ma := mobileStep.Actions[i]
-		if ma.Method != uixt.ACTION_TapByCV {
-			continue
+		actionOptions := uixt.NewActionOptions(ma.GetOptions()...)
+		// append tap_cv params to screenshot_with_ui_types option
+		if ma.Method == uixt.ACTION_TapByCV {
+			uiTypes, _ := builtin.ConvertToStringSlice(ma.Params)
+			ma.ActionOptions.ScreenShotWithUITypes = append(ma.ActionOptions.ScreenShotWithUITypes, uiTypes...)
+		}
+		// set default max_retry_times to 10 for swipe_to_tap_texts
+		if ma.Method == uixt.ACTION_SwipeToTapTexts && actionOptions.MaxRetryTimes == 0 {
+			ma.ActionOptions.MaxRetryTimes = 10
 		}
-		uiTypes, _ := builtin.ConvertToStringSlice(ma.Params)
-		ma.ActionOptions.ScreenShotWithUITypes = append(ma.ActionOptions.ScreenShotWithUITypes, uiTypes...)
 		mobileStep.Actions[i] = ma
 	}
 }