Skip to content

[Pangolins] Sprint 13 #15

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 39 commits into from
Nov 21, 2017
Merged

[Pangolins] Sprint 13 #15

merged 39 commits into from
Nov 21, 2017

Conversation

okolesnyk
Copy link
Member

@okolesnyk okolesnyk commented Nov 21, 2017

Scope

Stories

  • MQE-388 Write store settings before tests
  • MQE-508 before and after tags should have their own group/type in the schema
  • MQE-483 Sanitize test naming to prevent Jenkins build failure
  • MQE-432 Create variable reference verification tests
  • MQE-421 Create Unit Tests for the object model
  • MQE-235 [Generator] Add additional Assertion methods
  • MQE-336 [Generator] Add support for Codeception "Locator" selectors
  • MQE-448 Create a script to generate full Product metadata from Swagger
  • MQE-462 Update waitForLoadingMaskToDisappear to handle duplicated masks
  • MQE-497 Add useCaseId annotation
  • MQE-434 Create operation api unit tests

Bugs

  • MQE-472 [Data Input] Unable to resolve array data input in MFTF.
  • MQE-531 Unit and Verification tests do not run with MFTF PR
  • MQE-544 Change verification test filenames to begin with Uppercase
  • MQE-516 Duplicated $data.key$ references are not replaced correctly
  • MQE-406 Omitting defaultValue for actionGroup argument causes error

Related Pull Requests

imeron2433 and others added 30 commits October 27, 2017 11:18
- add new metadata generator util
- add new example input file
- change dir setup util to exposer public recursive delete

(cherry picked from commit 8d698c4)
- change output deletion beahvior
- fix codesniffer method

(cherry picked from commit 63dfb63)
…e schema

- add new type for before/after schema
- add new validation util
- update test object extractor to use validation util
- add new unit test
- add invalid char to error message, review feedback
- Created all verification tests.
- Quick bugfix for conditionalClick in di.xml
- Add Uniqueness Call to dependentSelector.
- Remove previous persistedReplacement test.
- Added locatorFunction attribute to elements and ElementObject
- Changed ActionObject to look for selector, then locatorFunction when trying to resolve element references.
- TestGenerator fixes to add double quotes to locatorFunction arguments, necessary for good output.
- Verification test fixes and additions.
- edit travis.yml file to include reference to all-checks command
- Fixed CR feedback
- Merged in sprint-develop and fixed conflicts
- Changed loadingMasks to an array
- Changed waitForLoadingMaskToDisappear to recurse through all loading mask elements if multiple are found.
- Add a test for both selector and locatorFunction null
- replacement now always appends "" . and . "" to replacement, then strips if unecessary.
- edited expected output in verification tests.
KevinBKozan and others added 8 commits November 15, 2017 13:33
- added useCaseId to Test Schema.
- di.xml changes
- Refarctored TestGenerator to use a single Annotation Generation mechanism.
- Updated Verification Tests, turns out there was a bug in existing code.
# Conflicts:
#	dev/tests/verification/Resources/PersistedReplacementCest.txt
#	dev/tests/verification/TestModule/Cest/PersistedReplacementCest.xml
#	etc/di.xml
    #       dev/tests/verification/Resources/PersistedReplacementCest.txt
- add new unit tests
- refactor old unit test to use new mock tool - AspectMock
- fix static code sniff failures
- add static checks for unit and verification test
@okolesnyk okolesnyk changed the title Sprint develop [Pangolins] Sprint 13 Nov 21, 2017
@okolesnyk okolesnyk merged commit c724e5d into magento:develop Nov 21, 2017
mmansoor-magento pushed a commit that referenced this pull request Dec 14, 2020
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.

5 participants