Skip to content

Add Page object - #23

Merged
camillobruni merged 30 commits into
WebKit:mainfrom
camillobruni:2022-10-11_page_object
Nov 22, 2022
Merged

camillobruni merged 30 commits into
WebKit:mainfrom
camillobruni:2022-10-11_page_object

Conversation

@camillobruni

@camillobruni camillobruni commented Oct 13, 2022 •

Copy link
Copy Markdown
Contributor

Goal: Add a fixed interface to interact with test frames and their DOM nodes.
This will allow us to run more complex scenarios in the future without sacrificing flexibility or performance for the existing tests.

  • Add Page object
  • Refactor tests to interact with the frame only through the Page object

@camillobruni

camillobruni commented Oct 13, 2022 •

Copy link
Copy Markdown
Contributor Author

Depends on the module migration PR #19.
Depending on the resolution of PR #22 I might have to fix the mail client implementation to work with the Page API.

Comment thread index.html Resolved
Comment thread resources/benchmark-runner.mjs Resolved
@camillobruni camillobruni changed the title WIP Add Page object Add Page object Nov 7, 2022
@camillobruni
camillobruni requested a review from rniwa November 7, 2022 13:41
Comment thread resources/benchmark-runner.mjs Resolved
Comment thread resources/benchmark-runner.mjs Resolved
Comment thread resources/benchmark-runner.mjs Resolved
Comment thread resources/benchmark-runner.mjs Resolved
Comment thread resources/benchmark-runner.mjs Resolved
Comment thread resources/benchmark-runner.mjs Resolved
Comment thread resources/tests.mjs Resolved
Comment thread resources/tests.mjs Resolved
Comment thread resources/tests.mjs Resolved
Comment thread resources/tests.mjs Resolved
Comment thread resources/tests.mjs Resolved

@squarewave squarewave left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This all looks good to me, as far as the substance of the change goes. Thanks for taking this on.

@camillobruni

Copy link
Copy Markdown
Contributor Author

I've addressed your comments.
Will upload the async-conversion PR soon.

Comment thread resources/benchmark-runner.mjs Resolved
Comment thread resources/tests.mjs Resolved
Comment thread resources/tests.mjs Resolved

@rniwa rniwa left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

r=me

@bgrins
bgrins self-requested a review November 18, 2022 15:52

@bgrins bgrins left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Approving as per Doug's comment

@bgrins

bgrins commented Nov 18, 2022

Copy link
Copy Markdown
Contributor

@camillobruni is this good to land?

@camillobruni

Copy link
Copy Markdown
Contributor Author

Sorry, long weekend. double checking now.

@camillobruni
camillobruni merged commit e1125d9 into WebKit:main Nov 22, 2022
@camillobruni
camillobruni deleted the 2022-10-11_page_object branch November 30, 2022 18:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants