Skip to content

[Pangolin] Deliver changes from Sprint 14 #17

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 48 commits into from
Dec 8, 2017
Merged

[Pangolin] Deliver changes from Sprint 14 #17

merged 48 commits into from
Dec 8, 2017

Conversation

okolesnyk
Copy link
Member

@okolesnyk okolesnyk commented Dec 8, 2017

Scope

Stories

  • MQE-494: Create MFTF demo repository
  • MQE-469: use 'number' for operation data type according to swagger 2.0 spec.
  • MQE-382: Adding a "assertElementContainsAttribute" method.
  • MQE-569: added SampleTests in module blacklist and not allowed to generate them.
  • MQE-525: Include PHP Mess Detector to required checks for MFTF
  • MQE-570: Sanitize MAGENTO_BASE_URL (.env file) and Selenium params
  • MQE-549: [Unit Test] ActionMergeUtil.php
  • MQE-600: Prune README.md in EE repo

Bugs

  • MQE-507: [Action Groups] Strange behavior with argument replacement in selector
  • MQE-565: User are not able to use after updating the latest composer.json
  • MQE-586: Ignore _generated folder for mftf code sniffer
  • MQE-591: Bin scripts do not return the correct error code
  • MQE-579: waitForLoadingMaskToDisappear fails sometimes when two loading masks match the same selector

Related Pull Requests

https://github.com/magento/magento2ce/pull/1813
https://github.com/magento/magento2ee/pull/809
https://github.com/magento/magento2b2b/pull/319

Checklist

  • Jira issues have accurate summary, meaningful description and have links to relevant documentation at the story/task level
  • Static tests are green on latest commit
  • Verification tests are green on latest commit
  • Changes to Framework doesn't have backward incompatible changes for tests or have related Pull Request with fixes to tests

Created by : @okolesnyk
Internal PR: https://github.com/magento-pangolin/magento2-functional-testing-framework/pull/42

tomreece and others added 30 commits November 21, 2017 10:22
- Fail gracefully if Magento is not running
- Do not error when Data, Section, or Page XML does not exist
…n selector

- Changed str_replace to preg_replace, regex disallows replacement when the argument is attached to another word (cases like categorySideButton versus category.name)
- Added Verification test for useCase
…test composer.json

- add new test actions type
- updated verification tests to include check for actionGroup in hook type
- Do not blow up if admin username or admin password not set in .env file
    - Adding the "assertElementContainsAttribute" method to the MagentoWebDriver file.
    - Adding the "assertElementContainsAttribute" method to the Test Schema.
    - Adding the "assertElementContainsAttribute" method to the di.xml so it can be listed multiple times in a test.
    - Adding the "assertElementContainsAttribute" method to the MagentoWebDriver.php file.
    - Adding verification tests.
    - Expanding comments for the function to make it clearer.
    - Replaced "userInput" with "expectedValue".
    - Adding "expectedValue" to the TestGenerator.
- updated composer to pull phpmd, ported Magento rulesets to work with phpmd on MFTF
- updated static-checks with PHPMD, no changes to devops required
- commented Mess Detector Lines out to enable them during MQE-590
…ng masks match the same selector

- Added parenthesis according to the AC.
- addressed all base AC of issue.
- Broke out sanitization into its own method, defined _resetConfig to prevent variables from being overwritten on test startup.
- Added step to configure rest URL based on sanitized version of MAGENTO_BASE_URL.
- Copy readme from tests (magento2) repo
- Add extension to license to make opening easier
- add new removal action
- add new deprecation message for incorrect usage of removal attribute
- update verification tests
- removing reference to remove attribute
- fix codesniffer error from static check
jilu1 and others added 18 commits December 7, 2017 14:09
# Conflicts:
#	dev/tests/verification/TestModule/ActionGroup/MergeFunctionalActionGroup.xml
#	dev/tests/verification/TestModule/Cest/MergeInFunctionalCest.xml
#	etc/di.xml
#	src/Magento/FunctionalTestingFramework/Test/etc/testSchema.xsd
- replace hardcoded dir references with DIRECTORY_SEPARATOR constant
- changes to test suite in order to account for os pathing in
verification tests
- add .gitattributes to the mftf
# Conflicts:
#	src/Magento/FunctionalTestingFramework/Test/etc/testSchema.xsd
- Adding a call to the "closeAdminNotification" function, because Admin notifications randomly appear after some tests due to Cache Management.
- Removing the PhantomJS yml file because the PhantomJS project is not maintained.
- add changes for MagentoWebDriver wait and pageload timeout settings
@okolesnyk okolesnyk merged commit 70c220f into magento:develop Dec 8, 2017
mmansoor-magento pushed a commit that referenced this pull request Dec 16, 2020
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.

8 participants