Merge pull request #1692 from httprunner/fix-identifier

fix: swipeToTapTexts redundant identifiers
This commit is contained in:
debugtalk
2023-09-07 11:24:50 +08:00
committed by GitHub

View File

@@ -134,6 +134,9 @@ func (dExt *DriverExt) swipeToTapTexts(texts []string, options ...ActionOption)
} }
options = append(options, WithMatchOne(true), WithRegex(true)) options = append(options, WithMatchOne(true), WithRegex(true))
actionOptions := NewActionOptions(options...)
actionOptions.Identifier = ""
optionsWithoutIdentifier := actionOptions.Options()
var point PointF var point PointF
findTexts := func(d *DriverExt) error { findTexts := func(d *DriverExt) error {
var err error var err error
@@ -145,7 +148,7 @@ func (dExt *DriverExt) swipeToTapTexts(texts []string, options ...ActionOption)
if err != nil { if err != nil {
return err return err
} }
points, err := screenResult.Texts.FindTexts(texts, dExt.ParseActionOptions(options...)...) points, err := screenResult.Texts.FindTexts(texts, dExt.ParseActionOptions(optionsWithoutIdentifier...)...)
if err != nil { if err != nil {
log.Error().Err(err).Msg("swipeToTapTexts failed") log.Error().Err(err).Msg("swipeToTapTexts failed")
// target texts not found, try to auto handle popup // target texts not found, try to auto handle popup
@@ -165,8 +168,8 @@ func (dExt *DriverExt) swipeToTapTexts(texts []string, options ...ActionOption)
return d.TapAbsXY(point.X, point.Y, options...) return d.TapAbsXY(point.X, point.Y, options...)
} }
findAction := dExt.prepareSwipeAction(options...) findAction := dExt.prepareSwipeAction(optionsWithoutIdentifier...)
return dExt.LoopUntil(findAction, findTexts, foundTextAction, options...) return dExt.LoopUntil(findAction, findTexts, foundTextAction, optionsWithoutIdentifier...)
} }
func (dExt *DriverExt) swipeToTapApp(appName string, options ...ActionOption) error { func (dExt *DriverExt) swipeToTapApp(appName string, options ...ActionOption) error {