Skip to content

Commit

Permalink
Feature/focus (#340)
Browse files Browse the repository at this point in the history
* Added and implemented Focus and FocusBySelector methods

* Added e2e tests

* Updated CHANGELOG

* Fixed linting errors
  • Loading branch information
ziflex authored Jul 23, 2019
1 parent 996d565 commit 7e6b3bf
Show file tree
Hide file tree
Showing 12 changed files with 154 additions and 0 deletions.
1 change: 1 addition & 0 deletions CHANGELOG.md
Original file line number Diff line number Diff line change
Expand Up @@ -11,6 +11,7 @@
- Regular expression operator. [#326](https://github.com/MontFerret/ferret/pull/326)
- ``INNER_HTML_SET`` and ``INNER_TEXT_SET`` functions. [#329](https://github.com/MontFerret/ferret/pull/329)
- Possibility to set viewport size. [#334](https://github.com/MontFerret/ferret/pull/334)
- ``FOCUS`` function. [#340](https://github.com/MontFerret/ferret/pull/340)

#### Changed
- ``RAND`` accepts optional upper and lower limits. [#271](https://github.com/MontFerret/ferret/pull/271)
Expand Down
66 changes: 66 additions & 0 deletions e2e/pages/dynamic/components/pages/events/focusable.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,66 @@
import random from "../../../utils/random.js";

const e = React.createElement;

export default class FocusableComponent extends React.PureComponent {
constructor(props) {
super(props);

this.state = {
show: props.show === true
};
}

toggle(show) {
let timeout = 500;

if (this.props.randomTimeout) {
timeout = random();
}

setTimeout(() => {
this.setState({
show: show
})
}, timeout)
}

handleFocus() {
this.toggle(true);
}

handleBlur() {
this.toggle(false);
}

render() {
const btnId = `${this.props.id}-input`;
const contentId = `${this.props.id}-content`;
const classNames = ["alert"];

if (this.state.show === true) {
classNames.push("alert-success");
}

return e("div", {className: "card focusable"}, [
e("div", { className: "card-header"}, [
e("div", { classNames: "form-group" }, [
e("input", {
id: btnId,
className: "form-control",
onFocus: this.handleFocus.bind(this),
onBlur: this.handleBlur.bind(this)

})
])
]),
e("div", { className: "card-body"}, [
e("div", { id: contentId, className: classNames.join(" ")}, [
e("p", null, [
"Lorem ipsum dolor sit amet."
])
])
])
]);
}
}
10 changes: 10 additions & 0 deletions e2e/pages/dynamic/components/pages/events/index.js
Original file line number Diff line number Diff line change
@@ -1,6 +1,7 @@
import Hoverable from "./hoverable.js";
import Clickable from "./clickable.js";
import Appearable from "./appearable.js";
import Focusable from "./focusable.js";

const e = React.createElement;

Expand Down Expand Up @@ -73,6 +74,15 @@ export default class EventsPage extends React.Component {
useStyle: true,
})
]),
]),
e("div", { className: "row" }, [
e("div", { className: "col-lg-4"}, [
e(Focusable, {
id: "focus",
appear: false,
title: "Focusable"
})
]),
])
])
}
Expand Down
8 changes: 8 additions & 0 deletions e2e/tests/dynamic/doc/focus/selector.fql
Original file line number Diff line number Diff line change
@@ -0,0 +1,8 @@
LET url = @dynamic + "/#/events"
LET page = DOCUMENT(url, true)

FOCUS(page, "#focus-input")

WAIT_CLASS(page, "#focus-content", "alert-success")

RETURN ""
10 changes: 10 additions & 0 deletions e2e/tests/dynamic/element/focus/focus.fql
Original file line number Diff line number Diff line change
@@ -0,0 +1,10 @@
LET url = @dynamic + "/#/events"
LET page = DOCUMENT(url, true)

LET input = ELEMENT(page, "#focus-input")

FOCUS(input)

WAIT_CLASS(page, "#focus-content", "alert-success")

RETURN ""
4 changes: 4 additions & 0 deletions pkg/drivers/cdp/document.go
Original file line number Diff line number Diff line change
Expand Up @@ -345,6 +345,10 @@ func (doc *HTMLDocument) SelectBySelector(ctx context.Context, selector values.S
return doc.input.SelectBySelector(ctx, doc.element.id.nodeID, selector, value)
}

func (doc *HTMLDocument) FocusBySelector(ctx context.Context, selector values.String) error {
return doc.input.FocusBySelector(ctx, doc.element.id.nodeID, selector)
}

func (doc *HTMLDocument) MoveMouseBySelector(ctx context.Context, selector values.String) error {
return doc.input.MoveMouseBySelector(ctx, doc.element.id.nodeID, selector)
}
Expand Down
4 changes: 4 additions & 0 deletions pkg/drivers/cdp/element.go
Original file line number Diff line number Diff line change
Expand Up @@ -971,6 +971,10 @@ func (el *HTMLElement) ScrollIntoView(ctx context.Context) error {
return el.input.ScrollIntoView(ctx, el.id.objectID)
}

func (el *HTMLElement) Focus(ctx context.Context) error {
return el.input.Focus(ctx, el.id.objectID)
}

func (el *HTMLElement) Hover(ctx context.Context) error {
return el.input.MoveMouse(ctx, el.id.objectID)
}
Expand Down
4 changes: 4 additions & 0 deletions pkg/drivers/http/document.go
Original file line number Diff line number Diff line change
Expand Up @@ -247,6 +247,10 @@ func (doc *HTMLDocument) ScrollByXY(_ context.Context, _, _ values.Float) error
return core.ErrNotSupported
}

func (doc *HTMLDocument) FocusBySelector(_ context.Context, _ values.String) error {
return core.ErrNotSupported
}

func (doc *HTMLDocument) MoveMouseBySelector(_ context.Context, _ values.String) error {
return core.ErrNotSupported
}
Expand Down
4 changes: 4 additions & 0 deletions pkg/drivers/http/element.go
Original file line number Diff line number Diff line change
Expand Up @@ -505,6 +505,10 @@ func (el *HTMLElement) ScrollIntoView(_ context.Context) error {
return core.ErrNotSupported
}

func (el *HTMLElement) Focus(_ context.Context) error {
return core.ErrNotSupported
}

func (el *HTMLElement) Hover(_ context.Context) error {
return core.ErrNotSupported
}
Expand Down
4 changes: 4 additions & 0 deletions pkg/drivers/value.go
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,8 @@ type (

ScrollIntoView(ctx context.Context) error

Focus(ctx context.Context) error

Hover(ctx context.Context) error

WaitForAttribute(ctx context.Context, name values.String, value core.Value, when WaitEvent) error
Expand Down Expand Up @@ -141,6 +143,8 @@ type (

ScrollByXY(ctx context.Context, x, y values.Float) error

FocusBySelector(ctx context.Context, selector values.String) error

MoveMouseByXY(ctx context.Context, x, y values.Float) error

MoveMouseBySelector(ctx context.Context, selector values.String) error
Expand Down
38 changes: 38 additions & 0 deletions pkg/stdlib/html/focus.go
Original file line number Diff line number Diff line change
@@ -0,0 +1,38 @@
package html

import (
"context"
"github.com/MontFerret/ferret/pkg/drivers"
"github.com/MontFerret/ferret/pkg/runtime/core"
"github.com/MontFerret/ferret/pkg/runtime/values"
)

// FOCUS Calls focus on the element.
// @param target (HTMLPage | HTMLDocument | HTMLElement) - Target node.
// @param selector (String, optional) - Optional CSS selector. Required when target is HTMLPage or HTMLDocument.
func Focus(ctx context.Context, args ...core.Value) (core.Value, error) {
err := core.ValidateArgs(args, 1, 2)

if err != nil {
return values.None, err
}

// Document with selector
if len(args) == 2 {
doc, err := drivers.ToDocument(args[0])

if err != nil {
return values.None, err
}

return values.None, doc.FocusBySelector(ctx, values.ToString(args[1]))
}

el, err := drivers.ToElement(args[0])

if err != nil {
return values.None, err
}

return values.None, el.Focus(ctx)
}
1 change: 1 addition & 0 deletions pkg/stdlib/html/lib.go
Original file line number Diff line number Diff line change
Expand Up @@ -28,6 +28,7 @@ func RegisterLib(ns core.Namespace) error {
"ELEMENT_EXISTS": ElementExists,
"ELEMENTS": Elements,
"ELEMENTS_COUNT": ElementsCount,
"FOCUS": Focus,
"HOVER": Hover,
"INNER_HTML": GetInnerHTML,
"INNER_HTML_SET": SetInnerHTML,
Expand Down

0 comments on commit 7e6b3bf

Please sign in to comment.