Skip to content

magento/magento2-functional-testing-framework#54: Set a global value for timeouts that's used across all wait related actions #71

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 9 commits into from
Aug 29, 2018

Conversation

vasylmalanka
Copy link
Contributor

@vasylmalanka vasylmalanka commented Mar 24, 2018

Description

Edited wait action methods to use wait timeout value from .env file.

Fixed Issues (if relevant)

  1. Set a global value for timeouts that's used across all wait related actions #54: Set a global value for timeouts that's used across all wait related actions

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

…ted actions

- Added WAIT_TIMEOUT environment variable to .env file
- Implemented getDefaultWaitTimeout method that retrieves WAIT_TIMEOUT environment variable or a constant value
- Edited wait action methods to use default timeout if not set
…ted actions

- Added Unit test for getDefaultWaitTimeout method
…ted actions

- Added WAIT_TIMEOUT environment variable to .env file
…ted actions

- Removed .env and updated .env.example file
…ted actions

    - Transformed 'getDefaultWaitTimeout' into static method
…ted actions

    - Used .env.example configuration file instead of .env
    - Changed hardcoded value to environment variable
…ted actions

- Added PHPDoc block to getDefaultWaitTimeout method
- Changed usage of getDefaultWaitTimeout static method in other methods
@coveralls
Copy link

coveralls commented Mar 29, 2018

Coverage Status

Coverage increased (+0.03%) to 52.471% when pulling 8ceca86 on vasylmalanka:issue-54 into 2109b0d on magento:CD-develop.

@okolesnyk
Copy link
Member

Communicated with @vasylmalanka regarding this issue via Slack. He is trying to use native Codeception Wait class to define global value for all waits. In case of any problems Vasyl can ask any questions in Slack or left any comments here.

…ted actions

- Reverted MagentoWebDriver class
- Moved getDefaultWaitTimeout method to ActionObject class
- Changed generateStepsPhp method in TestGenerator class to use timeout from getDefaultWaitTimeout method if not set
- Moved and fixed unit test for the implementation
@okolesnyk okolesnyk modified the milestones: 2.2.0, 2.3.0 Jun 18, 2018
@okolesnyk okolesnyk modified the milestones: 2.3.0, 2.3.1 Jul 18, 2018
@okolesnyk okolesnyk merged commit 5f5d3d9 into magento:CD-develop Aug 29, 2018
magento-devops-reposync-svc pushed a commit that referenced this pull request Jul 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants