-
Notifications
You must be signed in to change notification settings - Fork 132
MQE-1755: mftf run:test Test1 Test2 does not run before/after hooks c… #448
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
Conversation
…orrectly fixed unit tests
…orrectly fix unit tests
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changes seem to be working correctly, just have a small duped code change request.
//for tests in suites, run them as a group to run before and after block | ||
foreach (array_keys($suitesConfig) as $suite) { | ||
$fullCommand = $codeceptionCommand . " -g {$suite}"; | ||
$process = new Process($fullCommand); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ln 152 - 158 are duplicates of 126-132. Abstract it out into its own function so that you can just call something like
$subreturnCode = $this->executeTestCommand($fullCommand);
$this->returnCode = max($this->returnCode, $subreturnCode);
Double check for any more duped variable assignments/etc for consolidation as well.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
created a new function executeTestCommand that runs the codecept command.
…orrectly fixed unit test failures
MFTF 5.0.3 Release
…orrectly
Description
Fixed Issues (if relevant)
Contribution checklist