Skip to content

MQE-1123: Impossible remove data created in preconditions from test steps #209

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

Conversation

KevinBKozan
Copy link
Contributor

Description

  • Introduced PersistedObjectHandler, which handles persisted objects in tests/hooks/suites.
  • Tests now call the above when performing CRUD operations against data, and when retrieving data via $data.field$ syntax
  • $$data.field$$ and $data.field$ references now do the same thing; can be deprecated.

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

- TestGenerator keeps track stepKeys in the block of test actions, assumes createDataKey references are scoped to the class if they are not present in the test.
- Refactored TestGenerator code to use new Handler
- Introduced Handler
- Further TestGenerator refactor work to use constants
- Scope now passed into generateStepsPhp (to support test/hook/suite)
- refactored $data.key$ references to use handler for value retrieval
- Incorrect generation fixes.
- Unit tests created
- Update to all verification tests
- Updated Generator to use private variable for currentScope, refactored signatures to account for this
- verification test update to increase coverage
- Fixed merged verification test
- ActionGroups were not replacing their createDataKey references correctly.
- Added new test for above use case
- No hooks no longer assign a variable (not necessary), verification test updates
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.3%) to 57.132% when pulling b90e314 on MQE-1123-DATAREFACTOR into dc102e7 on develop.

@KevinBKozan KevinBKozan merged commit 00e78f8 into develop Aug 29, 2018
@KevinBKozan KevinBKozan deleted the MQE-1123-DATAREFACTOR branch August 29, 2018 14:22
magento-devops-reposync-svc pushed a commit that referenced this pull request Aug 24, 2022
3.10.2 RC merging master back to develop
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.

2 participants