Skip to content

Commit 5ad6aca

Browse files
Manjusha.SManjusha.S
Manjusha.S
authored and
Manjusha.S
committed
fixed issue
1 parent 0c811db commit 5ad6aca

File tree

1 file changed

+5
-3
lines changed

1 file changed

+5
-3
lines changed

src/Magento/FunctionalTestingFramework/Util/TestGenerator.php

+5-3
Original file line numberDiff line numberDiff line change
@@ -564,11 +564,13 @@ public function requiredCredentials($testObject)
564564
private function generateClassAnnotations($annotationType, $annotationName, $testObject)
565565
{
566566
$annotationToAppend = null;
567-
if (!empty($annotationName['main'])) {
567+
if (!$testObject->isSkipped() && !empty($annotationName['main'])) {
568568
$requiredCredentialsMessage = $this->requiredCredentials($testObject);
569569
$credMsg = "\n\n"."This test uses the following credentials:"."\n";
570-
$annotationName = ['main'=>$annotationName['main'].', '.$credMsg.''.$requiredCredentialsMessage,
571-
'test_files'=> "\n".$annotationName['test_files'], 'deprecated'=>$annotationName['deprecated']];
570+
$annotationName = (!empty($requiredCredentialsMessage)) ?
571+
['main'=>$annotationName['main'].', '.$credMsg.''.$requiredCredentialsMessage,
572+
'test_files'=> "\n".$annotationName['test_files'], 'deprecated'=>$annotationName['deprecated']]
573+
: $annotationName;
572574
}
573575
switch ($annotationType) {
574576
case "title":

0 commit comments

Comments
 (0)