[Attributes] Cover slash newline in AnnotationToAttributeRector with …

…values (#6557)

* use stubs for Then and When

* cover slash newline in AnnotationToAttributeRector

rules-tests/Php80/Rector/Class_/AnnotationToAttributeRector/Fixture/Behat/with_next_line.php.inc

@@ -0,0 +1,30 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Behat;
+
+final class WithNextLine
+{
+    /**
+     * @Then then :value should \
+     * continue here
+     */
+    public function someStep(): void
+    {
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Behat;
+
+final class WithNextLine
+{
+    #[\Behat\Step\Then('then :value should continue here')]
+    public function someStep(): void
+    {
+    }
+}
+
+?>

rules-tests/Php80/Rector/Class_/AnnotationToAttributeRector/Fixture/Behat/with_value_edge_cases.php.inc

@@ -0,0 +1,29 @@
+<?php
+
+namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Behat;
+
+final class WithValueEdgeCases
+{
+    /**
+     * @Then then :value should be have been finished
+     */
+    public function someStep(): void
+    {
+    }
+}
+
+?>
+-----
+<?php
+
+namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture\Behat;
+
+final class WithValueEdgeCases
+{
+    #[\Behat\Step\Then('then :value should be have been finished')]
+    public function someStep(): void
+    {
+    }
+}
+
+?>

rules-tests/Php80/Rector/Class_/AnnotationToAttributeRector/Fixture/with_value_as_argument.php.inc

@@ -24,11 +24,11 @@ namespace Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Fixture;
 
 final class WithValueAsArgument
 {
-    #[\Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\Behat\When('this is a simple annotation with a value')]
-    #[\Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\Behat\When]
-    #[\Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\Behat\When('"this value is within quotes"')]
-    #[\Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\Behat\When('this value has a \' character')]
-    #[\Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\Behat\When(key: 'value')] // this annotation has parameters so won't use this option
+    #[\Behat\Step\When('this is a simple annotation with a value')]
+    #[\Behat\Step\When]
+    #[\Behat\Step\When('"this value is within quotes"')]
+    #[\Behat\Step\When('this value has a \' character')]
+    #[\Behat\Step\When(key: 'value')] // this annotation has parameters so won't use this option
     public function someStep(): void
     {
     }

rules-tests/Php80/Rector/Class_/AnnotationToAttributeRector/config/configured_rule.php

@@ -7,7 +7,6 @@
 use Rector\Php80\ValueObject\AnnotationToAttribute;
 use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Annotation\OpenApi\Annotation\NestedPastAnnotation;
 use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Annotation\OpenApi\PastAnnotation;
-use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\Behat\When;
 use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\OpenApi\Attribute\NestedFutureAttribute;
 use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\Attribute\OpenApi\FutureAttribute;
 use Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\GenericAnnotation;
@@ -48,6 +47,9 @@
                 'Rector\Tests\Php80\Rector\Class_\AnnotationToAttributeRector\Source\UseAlias\TestOther'
             ),
             new AnnotationToAttribute('Sensio\Bundle\FrameworkExtraBundle\Configuration\Security'),
-            new AnnotationToAttribute('When', When::class, useValueAsAttributeArgument: true),
+
+            // special case with following comment becoming a inner value
+            new AnnotationToAttribute('When', \Behat\Step\When::class, useValueAsAttributeArgument: true),
+            new AnnotationToAttribute('Then', \Behat\Step\Then::class, useValueAsAttributeArgument: true),
         ]);
 };

rules/Php80/Rector/Class_/AnnotationToAttributeRector.php

@@ -4,6 +4,7 @@
 
 namespace Rector\Php80\Rector\Class_;
 
+use Nette\Utils\Strings;
 use PhpParser\Node;
 use PhpParser\Node\AttributeGroup;
 use PhpParser\Node\Expr\ArrowFunction;
@@ -204,10 +205,20 @@ private function processGenericTags(PhpDocInfo $phpDocInfo): array
                     continue;
                 }
 
+                $docValue = null;
+                if ($annotationToAttribute->getUseValueAsAttributeArgument()) {
+                    // special case for newline
+                    $docValue = (string) $docNode->value;
+                    if (str_contains($docValue, '\\')) {
+                        $docValue = Strings::replace($docValue, "#\\\\\n#", '');
+                    }
+                }
+
                 $attributeGroups[] = $this->phpAttributeGroupFactory->createFromSimpleTag(
                     $annotationToAttribute,
-                    $annotationToAttribute->getUseValueAsAttributeArgument() ? (string) $docNode->value : null
+                    $docValue
                 );
+
                 return PhpDocNodeTraverser::NODE_REMOVE;
             }
 

stubs/Behat/Step/Then.php

@@ -0,0 +1,12 @@
+<?php
+
+namespace Behat\Step;
+
+if (class_exists('Behat\Step\Then')) {
+    return;
+}
+
+class Then
+{
+
+}

stubs/Behat/Step/When.php

@@ -0,0 +1,12 @@
+<?php
+
+namespace Behat\Step;
+
+if (class_exists('Behat\Step\When')) {
+    return;
+}
+
+class When
+{
+
+}