Commit 361a133
committed
minor #42630 [Form] Remove broken test case (derrabus)
This PR was merged into the 4.4 branch.
Discussion
----------
[Form] Remove broken test case
| Q | A
| ------------- | ---
| Branch? | 4.4
| Bug fix? | yes
| New feature? | no
| Deprecations? | no
| Tickets | Part of #41552
| License | MIT
| Doc PR | N/A
If we try to parse a `DateTime` object solely from a day of year, like this…
```php
$date = \DateTime::createFromFormat('z', '33');
```
… PHP 8.1 will refuse to do that with the error message
> A 'day of year' can only come after a year has been found
The commit that caused this was php/php-src@8426623 by `@derickr`.
Our `DateTimeToStringTransformerTest` tests if that kind of transformation works, but I doubt that this covers an actual use case. This is why I propose to simply remove the failing test case.
Commits
-------
ee614c3d30 Remove broken test caseFile tree
1 file changed
+2
-5
lines changed- Tests/Extension/Core/DataTransformer
1 file changed
+2
-5
lines changedLines changed: 2 additions & 5 deletions
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
17 | 17 | | |
18 | 18 | | |
19 | 19 | | |
20 | | - | |
| 20 | + | |
21 | 21 | | |
22 | | - | |
| 22 | + | |
23 | 23 | | |
24 | 24 | | |
25 | 25 | | |
| |||
36 | 36 | | |
37 | 37 | | |
38 | 38 | | |
39 | | - | |
40 | 39 | | |
41 | 40 | | |
42 | 41 | | |
| |||
63 | 62 | | |
64 | 63 | | |
65 | 64 | | |
66 | | - | |
67 | | - | |
68 | 65 | | |
69 | 66 | | |
70 | 67 | | |
| |||
0 commit comments