Skip to content

MFTF 2.3.6 - Merge to master #221

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 18 commits into from
Sep 5, 2018
Merged

MFTF 2.3.6 - Merge to master #221

merged 18 commits into from
Sep 5, 2018

Conversation

KevinBKozan
Copy link
Contributor

Description

  • Merge 2.3.6 contents to master.

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

aljcalandra and others added 18 commits August 16, 2018 16:44
* MQE-1118: Handle XML merge failures gracefully

- Added filename to error messages for loadXML usage in initDom

* MQE-1118: Handle XML merge failures gracefully

- Added Test and made error a tiny bit more specific (error is caught by child classes given context)

* MQE-1118: Handle XML merge failures gracefully

- Added exception message check in unit test

* MQE-1118: Handle XML merge failures gracefully

- Added check for false returns from loadXML
- Added list of actions that return data, actionGroupObject now only returns stepKeys if the actions are of the right list.
* MQE-1161: Extended Tests With No Parents Are Skipped

- Tests are not generated if they are skipped
- Modified verification test to reflect change

* MQE-1161: Extended Tests With No Parents Are Skipped

- Updated skip flagging to use new skip annotation

* MQE-1161: Extended Tests With No Parents Are Skipped

- Updated conditional to check for parent test existence
- Added console information in addition to the log

* MQE-1161: Extended Tests With No Parents Are Skipped

- Updated message to be more explicit
Merge 2.3.5 from Master to Develop
* MQE-1157: Add readiness bypass to test actions

- Adding skip readiness getter
- Adding skip readiness attribute

* MQE-1157: Add readiness bypass to test actions

- Adding setp generation to TestGeneration

* MQE-1157: Add readiness bypass to test actions

- Added skip command to web driver
- Added check to page readiness extension

* MQE-1157: Add readiness bypass to test actions

- Updated Tests
- Added Restriction for action groups

* MQE-1157: Add readiness bypass to test actions

- Removed testing statements and unused code

* MQE-1157: Add readiness bypass to test actions

- Removed print statements, created action steps instead

* MQE-1157: Add readiness bypass to test actions

- Adding action to readiness skip list
- Adjusting conditional to check string match
…teps

- Introduced PersistedObjectHandler, which handles persisted objects in tests/hooks/suites.
- Generator code now uses above handler
- Various Unit/verification test fixes
MQE-1189: Suite Generation - Precondition body must always get and re…
MQE-1009: Generated Suites Are Not Cleaned Up When Regenerating
MQE-1168: magentoCLI action not working properly if URL to magento co…
… Annotation

- TestCaseID is now prepended to Title annotation (or NONE FOUND)
- Unit/Verification test updates
- Added entry for 2.3.6
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.2%) to 57.159% when pulling 31ddcc8 on develop into bb1518a on master.

Copy link
Contributor

@jilu1 jilu1 left a comment

Choose a reason for hiding this comment

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

Looks good to me

@KevinBKozan KevinBKozan merged commit 57021e1 into master Sep 5, 2018
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.

7 participants