Skip to content

Commit 7a4fa48

Browse files
committed
MQE-1068: Require Issue ID for Skipped Test
- Made TODO messages more informative
1 parent b4bdf77 commit 7a4fa48

File tree

2 files changed

+2
-2
lines changed

2 files changed

+2
-2
lines changed

src/Magento/FunctionalTestingFramework/Test/Objects/TestObject.php

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -126,7 +126,7 @@ public function getParentName()
126126
*/
127127
public function isSkipped()
128128
{
129-
// TODO remove elseif when group skip is no longer allowed
129+
// TODO deprecation|deprecate MFTF 3.0.0 remove elseif when group skip is no longer allowed
130130
if (array_key_exists('skip', $this->annotations)) {
131131
return true;
132132
} elseif (array_key_exists('group', $this->annotations) && (in_array("skip", $this->annotations['group']))) {

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

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -71,7 +71,7 @@ public function extractAnnotations($testAnnotations, $filename)
7171
foreach ($annotationData as $annotationValue) {
7272
$annotationValues[] = $annotationValue[self::ANNOTATION_VALUE];
7373
}
74-
74+
// TODO deprecation|deprecate MFTF 3.0.0
7575
if ($annotationKey == "group" && in_array("skip", $annotationValues)) {
7676
print(
7777
"Test: $filename" .

0 commit comments

Comments
 (0)