minor #57570 [Console] Test convert CompletionInput into string (seriquynh)

nicolas-grekas · nicolas-grekas · commit 7beff57ca190 · 2024-06-28T11:40:57.000+02:00
This PR was merged into the 5.4 branch. Discussion ---------- [Console] Test convert CompletionInput into string | Q | A | ------------- | --- | Branch? | 5.4 | Bug fix? | no | New feature? | no | Deprecations? | no | Issues | N/A | License | MIT CompletionInput can be converted into string but there are no tests for it. So, I a simple test here based on it's current behavior. Commits ------- 0a7477d Test convert CompletionInput into string
diff --git a/src/Symfony/Component/Console/Completion/CompletionInput.php b/src/Symfony/Component/Console/Completion/CompletionInput.php
@@ -53,7 +53,7 @@ public static function fromString(string $inputStr, int $currentIndex): self
      * Create an input based on an COMP_WORDS token list.
      *
      * @param string[] $tokens       the set of split tokens (e.g. COMP_WORDS or argv)
-     * @param          $currentIndex the index of the cursor (e.g. COMP_CWORD)
+     * @param int      $currentIndex the index of the cursor (e.g. COMP_CWORD)
      */
     public static function fromTokens(array $tokens, int $currentIndex): self
     {
diff --git a/src/Symfony/Component/Console/Tests/Completion/CompletionInputTest.php b/src/Symfony/Component/Console/Tests/Completion/CompletionInputTest.php
@@ -133,4 +133,19 @@ public static function provideFromStringData()
         yield ['bin/console cache:clear "multi word string"', ['bin/console', 'cache:clear', '"multi word string"']];
         yield ['bin/console cache:clear \'multi word string\'', ['bin/console', 'cache:clear', '\'multi word string\'']];
     }
+
+    public function testToString()
+    {
+        $input = CompletionInput::fromTokens(['foo', 'bar', 'baz'], 0);
+        $this->assertSame('foo| bar baz', (string) $input);
+
+        $input = CompletionInput::fromTokens(['foo', 'bar', 'baz'], 1);
+        $this->assertSame('foo bar| baz', (string) $input);
+
+        $input = CompletionInput::fromTokens(['foo', 'bar', 'baz'], 2);
+        $this->assertSame('foo bar baz|', (string) $input);
+
+        $input = CompletionInput::fromTokens(['foo', 'bar', 'baz'], 11);
+        $this->assertSame('foo bar baz |', (string) $input);
+    }
 }

Original file line number	Diff line number	Diff line change
`@@ -53,7 +53,7 @@ public static function fromString(string $inputStr, int $currentIndex): self`
`53`	`53`	`* Create an input based on an COMP_WORDS token list.`
`54`	`54`	`*`
`55`	`55`	`* @param string[] $tokens the set of split tokens (e.g. COMP_WORDS or argv)`
`56`		`- * @param $currentIndex the index of the cursor (e.g. COMP_CWORD)`
	`56`	`+ * @param int $currentIndex the index of the cursor (e.g. COMP_CWORD)`
`57`	`57`	`*/`
`58`	`58`	`public static function fromTokens(array $tokens, int $currentIndex): self`
`59`	`59`	`{`