Skip to content

MQE-1382: New PWA Actions #288

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 6 commits into from
Jan 14, 2019
Merged

Conversation

ivy00johns
Copy link
Contributor

@ivy00johns ivy00johns commented Jan 9, 2019

Description

This PR contains 2 new Actions for the PWA project that is under development. These new actions replicate the original Codeception waitForElementNotVisible/waitForElementVisible actions except JavaScript instead of PHP.

  • waitForPwaElementNotVisible
  • waitForPwaElementVisible

Contribution checklist

  • Pull request has a meaningful description of its purpose
  • All commits are accompanied by meaningful commit messages
  • All new or changed code is covered with unit/verification tests (if applicable)
  • All automated tests passed successfully (all builds on Travis CI are green)
  • Changes to Framework doesn't have backward incompatible changes for tests or have related Pull Request with fixes to tests

- Adding a new action for waitForReactPageLoad to the schema.
- Adding basic structure for the new action to the MagentoWebDriver.
- Switched gears with this attempt, created a waitForElementVisible action instead of a waitForReactPageLoad action.
- Adding new action, DI updates and Actions updates.
- Renaming the new Actions to make it clear what they are for. Replaced "Js" with "Pwa".
@magento-cicd2
Copy link

magento-cicd2 commented Jan 9, 2019

CLA assistant check
All committers have signed the CLA.

@ivy00johns
Copy link
Contributor Author

@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 57.062% when pulling f5ba105 on MQE-1382-WaitForReactPageLoad into a88548b on develop.

1 similar comment
@coveralls
Copy link

Coverage Status

Coverage increased (+0.02%) to 57.062% when pulling f5ba105 on MQE-1382-WaitForReactPageLoad into a88548b on develop.

@coveralls
Copy link

coveralls commented Jan 9, 2019

Coverage Status

Coverage increased (+0.008%) to 57.054% when pulling cbe914b on MQE-1382-WaitForReactPageLoad into a88548b on develop.

@ivy00johns ivy00johns added this to the 2.3.1 milestone Jan 9, 2019
- Updating the action name in the Generator file.
- Created a separate Class for the custom PWA actions.
- Removed the custom PWA actions from MagentoWebDriver.
@ivy00johns ivy00johns merged commit 727c9c4 into develop Jan 14, 2019
@ivy00johns ivy00johns deleted the MQE-1382-WaitForReactPageLoad branch June 4, 2019 19:26
magento-devops-reposync-svc pushed a commit that referenced this pull request Feb 22, 2023
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