Skip to content

MQE-783:Investigate allure reporting when tests run in multiple suites #87

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
Apr 9, 2018

Conversation

magterskine
Copy link
Contributor

Description

the name of group used for test selection by codecept runner will be appended to the suite name in Allure

Fixed Issues (if relevant)

N/A

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

magterskine and others added 8 commits March 28, 2018 14:19
- semi working solution
…ple suites

- Cleaned up static check failures
…ple suites

- Added null checking for test runs without group argument
- Refactored suiteBefore in light of the above
…ple suites

- Updated MagentoAllureAdapter to remove instance varaibles and hardcode the groupKey to 'groups' as this will not change
…ple suites

- change suiteName to be of format <default>\<groupName>
@coveralls
Copy link

coveralls commented Apr 5, 2018

Coverage Status

Coverage remained the same at 51.306% when pulling 64cdddd on MQE-783 into 2fe0d2a on develop.

@magterskine magterskine merged commit 320dc63 into develop Apr 9, 2018
@magterskine magterskine deleted the MQE-783 branch April 9, 2018 15:50
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.

3 participants