Commit 00934e2
committed
bug symfony#61690 [Console] Don’t automatically append EOT to multiline test inputs (MatTheCat)
This PR was squashed before being merged into the 6.4 branch.
Discussion
----------
[Console] Don’t automatically append EOT to multiline test inputs
| Q | A
| ------------- | ---
| Branch? | 6.4
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Issues | N/A
| License | MIT
symfony#61501 assumed multiline inputs were answers to multiline questions and suffixed them with an EOT character to mark their end.
But if you’re directly reading stdin, you can have line breaks without the need for an EOT: symfony#61501 (comment)
Since the `TesterTrait` doesn’t know the inputs’ context, this PR reverts this behavior. Multiline questions will still be able to be tested by manually suffixing the answer with an EOT (`x04`) like in the added test.
If it is accepted it’ll be followed by a doc PR.
Commits
-------
938fabb [Console] Don’t automatically append EOT to multiline test inputsFile tree
2 files changed
+28
-4
lines changed- src/Symfony/Component/Console
- Tester
- Tests/Tester
2 files changed
+28
-4
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
169 | 169 | | |
170 | 170 | | |
171 | 171 | | |
172 | | - | |
173 | | - | |
174 | | - | |
175 | | - | |
176 | 172 | | |
177 | 173 | | |
178 | 174 | | |
| |||
Lines changed: 28 additions & 0 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
16 | 16 | | |
17 | 17 | | |
18 | 18 | | |
| 19 | + | |
19 | 20 | | |
| 21 | + | |
20 | 22 | | |
21 | 23 | | |
22 | 24 | | |
| |||
127 | 129 | | |
128 | 130 | | |
129 | 131 | | |
| 132 | + | |
| 133 | + | |
| 134 | + | |
| 135 | + | |
| 136 | + | |
| 137 | + | |
| 138 | + | |
| 139 | + | |
| 140 | + | |
| 141 | + | |
| 142 | + | |
| 143 | + | |
| 144 | + | |
| 145 | + | |
| 146 | + | |
| 147 | + | |
| 148 | + | |
| 149 | + | |
| 150 | + | |
| 151 | + | |
| 152 | + | |
| 153 | + | |
| 154 | + | |
| 155 | + | |
| 156 | + | |
| 157 | + | |
130 | 158 | | |
131 | 159 | | |
132 | 160 | | |
| |||
0 commit comments