Skip to content

Commit e7833c7

Browse files
committed
MQE-1068: Require Issue ID for Skipped Test
- Changed to log for warning
1 parent 9737bd8 commit e7833c7

File tree

1 file changed

+8
-6
lines changed

1 file changed

+8
-6
lines changed

src/Magento/FunctionalTestingFramework/Test/Util/AnnotationExtractor.php

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,9 @@
66

77
namespace Magento\FunctionalTestingFramework\Test\Util;
88

9+
use Magento\FunctionalTestingFramework\Config\MftfApplicationConfig;
910
use Magento\FunctionalTestingFramework\Exceptions\XmlException;
11+
use Magento\FunctionalTestingFramework\Util\Logger\LoggingUtil;
1012

1113
/**
1214
* Class AnnotationExtractor
@@ -73,12 +75,12 @@ public function extractAnnotations($testAnnotations, $filename)
7375
}
7476
// TODO deprecation|deprecate MFTF 3.0.0
7577
if ($annotationKey == "group" && in_array("skip", $annotationValues)) {
76-
print(
77-
"Test: $filename" .
78-
PHP_EOL .
79-
"Use of group skip will be deprecated in MFTF 3.0.0. Please update tests to use skip tags." .
80-
PHP_EOL
81-
);
78+
if (MftfApplicationConfig::getConfig()->verboseEnabled()) {
79+
LoggingUtil::getInstance()->getLogger(AnnotationExtractor::class)->warning(
80+
"Use of group skip will be deprecated in MFTF 3.0.0. Please update tests to use skip tags.",
81+
["test" => $filename]
82+
);
83+
}
8284
}
8385

8486
$annotationObjects[$annotationKey] = $annotationValues;

0 commit comments

Comments
 (0)