Prepare Locator for async migration

browser/mapping_test.go

@@ -504,7 +504,7 @@ type locatorAPI interface {
 	Check(opts goja.Value) error
 	Uncheck(opts goja.Value) error
 	IsChecked(opts goja.Value) (bool, error)
-	IsEditable(opts goja.Value) bool
+	IsEditable(opts goja.Value) (bool, error)
 	IsEnabled(opts goja.Value) bool
 	IsDisabled(opts goja.Value) bool
 	IsVisible(opts goja.Value) bool

common/locator.go

@@ -179,19 +179,19 @@ func (l *Locator) isChecked(opts *FrameIsCheckedOptions) (bool, error) {
 
 // IsEditable returns true if the element matches the locator's
 // selector and is Editable. Otherwise, returns false.
-func (l *Locator) IsEditable(opts goja.Value) bool {
+func (l *Locator) IsEditable(opts goja.Value) (bool, error) {
 	l.log.Debugf("Locator:IsEditable", "fid:%s furl:%q sel:%q opts:%+v", l.frame.ID(), l.frame.URL(), l.selector, opts)
 
 	copts := NewFrameIsEditableOptions(l.frame.defaultTimeout())
 	if err := copts.Parse(l.ctx, opts); err != nil {
-		k6ext.Panic(l.ctx, "parsing is editable options: %w", err)
+		return false, fmt.Errorf("parsing is editable options: %w", err)
 	}
 	editable, err := l.isEditable(copts)
 	if err != nil {
-		k6ext.Panic(l.ctx, "checking is %q editable: %w", l.selector, err)
+		return false, fmt.Errorf("checking is %q editable: %w", l.selector, err)
 	}
 
-	return editable
+	return editable, nil
 }
 
 // isEditable is like IsEditable but takes parsed options and does not

tests/locator_test.go

@@ -442,7 +442,7 @@ func TestLocatorElementState(t *testing.T) {
 		{
 			"readOnly",
 			`() => document.getElementById('inputText').readOnly = true`,
-			func(l *common.Locator) bool { return l.IsEditable(nil) },
+			func(l *common.Locator) bool { resp, _ := l.IsEditable(nil); return resp },
 		},
 		{
 			"visible",
@@ -493,7 +493,12 @@ func TestLocatorElementState(t *testing.T) {
 			},
 		},
 		{
-			"IsEditable", func(l *common.Locator, tb *testBrowser) { l.IsEditable(timeout(tb)) },
+			"IsEditable", func(l *common.Locator, tb *testBrowser) {
+				if _, err := l.IsEditable(timeout(tb)); err != nil {
+					// TODO: remove panic and update tests when all locator methods return error.
+					panic(err)
+				}
+			},
 		},
 		{
 			"IsEnabled", func(l *common.Locator, tb *testBrowser) { l.IsEnabled(timeout(tb)) },