fix: override action error

hrp/pkg/uixt/action.go

@@ -385,7 +385,7 @@ func (dExt *DriverExt) GenAbsScope(x1, y1, x2, y2 float64) AbsScope {
 	return AbsScope{absX1, absY1, absX2, absY2}
 }
 
-func (dExt *DriverExt) DoAction(action MobileAction) error {
+func (dExt *DriverExt) DoAction(action MobileAction) (err error) {
 	log.Debug().
 		Str("method", string(action.Method)).
 		Interface("params", action.Params).
@@ -393,11 +393,19 @@ func (dExt *DriverExt) DoAction(action MobileAction) error {
 	actionStartTime := time.Now()
 
 	defer func() {
+		if err != nil {
+			log.Error().Err(err).
+				Str("method", string(action.Method)).
+				Interface("params", action.Params).
+				Int64("elapsed(ms)", time.Since(actionStartTime).Milliseconds()).
+				Msg("uixt action end")
+		} else {
 			log.Debug().
 				Str("method", string(action.Method)).
 				Interface("params", action.Params).
 				Int64("elapsed(ms)", time.Since(actionStartTime).Milliseconds()).
 				Msg("uixt action end")
+		}
 	}()
 
 	switch action.Method {

hrp/step_mobile_ui.go

@@ -604,8 +604,8 @@ func runStepMobileUI(s *SessionRunner, step *TStep) (stepResult *StepResult, err
 		}
 
 		// automatic handling of pop-up windows on each step finished
-		if err = uiDriver.AutoPopupHandler(); err != nil {
+		if err2 := uiDriver.AutoPopupHandler(); err2 != nil {
-			log.Error().Err(err).Str("step", step.Name).Msg("auto handle popup failed")
+			log.Error().Err(err2).Str("step", step.Name).Msg("auto handle popup failed")
 		}
 
 		// save attachments