MQE-1068: Require Issue ID for Skipped Test

- Moved deprecation warning

Author: aljcalandra

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

@@ -130,7 +130,6 @@ public function isSkipped()
         if (array_key_exists('skip', $this->annotations)) {
             return true;
         } elseif (array_key_exists('group', $this->annotations) && (in_array("skip", $this->annotations['group']))) {
-            print("Use of group skip will be deprecated in MFTF 3.0.0. Please update tests to use skip tags.");
             return true;
         }
         return false;

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

@@ -62,14 +62,25 @@ public function extractAnnotations($testAnnotations, $filename)
                 );
                 continue;
             }
+
             if ($annotationKey == "skip") {
                 $annotationData = $annotationData['issueId'];
                 $this->validateSkippedIssues($annotationData, $filename);
             }
+
             foreach ($annotationData as $annotationValue) {
                 $annotationValues[] = $annotationValue[self::ANNOTATION_VALUE];
             }
 
+            if ($annotationKey == "group" && in_array("skip", $annotationValues)) {
+                print(
+                    "Test: $filename" .
+                    PHP_EOL .
+                    "Use of group skip will be deprecated in MFTF 3.0.0. Please update tests to use skip tags." .
+                    PHP_EOL
+                );
+            }
+
             $annotationObjects[$annotationKey] = $annotationValues;
         }
         $this->addStoryTitleToMap($annotationObjects, $filename);