Skip to content

Commit f43763f

Browse files
authored
Merge pull request #406 from magento-gl/ACQE-6716
ACQE-6716 | Unneeded reports are shown when MFTF Static tests fail
2 parents ef72d95 + 30f8fcc commit f43763f

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/Magento/FunctionalTestingFramework/StaticCheck/ClassFileNamingCheck.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -96,7 +96,7 @@ public function execute(InputInterface $input)
9696
StaticChecksList::getErrorFilesPath() . DIRECTORY_SEPARATOR . self::ERROR_LOG_FILENAME . '.txt',
9797
self::ERROR_LOG_MESSAGE
9898
);
99-
if (!empty($this->warnings)) {
99+
if (!empty($this->warnings) && !empty($this->errors)) {
100100
$this->output .= "\n " . $this->scriptUtil->printWarningsToFile(
101101
$this->warnings,
102102
StaticChecksList::getErrorFilesPath() . DIRECTORY_SEPARATOR . self::WARNING_LOG_FILENAME . '.txt',

src/Magento/FunctionalTestingFramework/StaticCheck/TestDependencyCheck.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -155,7 +155,7 @@ public function execute(InputInterface $input)
155155
StaticChecksList::getErrorFilesPath() . DIRECTORY_SEPARATOR . self::ERROR_LOG_FILENAME . '.txt',
156156
self::ERROR_LOG_MESSAGE
157157
);
158-
if (!empty($this->warnings)) {
158+
if (!empty($this->warnings) && !empty($this->errors)) {
159159
$this->output .= "\n " . $this->scriptUtil->printWarningsToFile(
160160
$this->warnings,
161161
StaticChecksList::getErrorFilesPath() . DIRECTORY_SEPARATOR . self::WARNING_LOG_FILENAME . '.txt',

0 commit comments

Comments
 (0)