Skip to content

Conversation

@herndlm
Copy link
Contributor

@herndlm herndlm commented Aug 18, 2021

Q A
Branch? 4.4
Bug fix? yes
New feature? no
Deprecations? no
Tickets Part of #41552
License MIT
Doc PR N/A

getSize returns int|false and cannot be mocked with INF which is float.
Fixes the failing test in PHP 8.1.

@carsonbot carsonbot added this to the 4.4 milestone Aug 18, 2021
@herndlm herndlm force-pushed the bugfix/fix-spl-file-info-mock-in-http-kernel-browser-test branch from 1e85dd5 to f44a9d1 Compare August 18, 2021 08:34
@herndlm herndlm changed the title Fix SplFileInfo mock in HttpKernelBrowserTest` Fix SplFileInfo mock in HttpKernelBrowserTest Aug 18, 2021
@herndlm herndlm force-pushed the bugfix/fix-spl-file-info-mock-in-http-kernel-browser-test branch from f44a9d1 to ad0c933 Compare August 18, 2021 08:35
@herndlm
Copy link
Contributor Author

herndlm commented Aug 18, 2021

No idea why the high-deps fail, should not be related 🤔

@herndlm herndlm force-pushed the bugfix/fix-spl-file-info-mock-in-http-kernel-browser-test branch from ad0c933 to 6990458 Compare August 18, 2021 08:44
@herndlm herndlm changed the title Fix SplFileInfo mock in HttpKernelBrowserTest [HttpKernel] Fix SplFileInfo mock in HttpKernelBrowserTest Aug 18, 2021
@carsonbot carsonbot changed the title [HttpKernel] Fix SplFileInfo mock in HttpKernelBrowserTest Fix SplFileInfo mock in HttpKernelBrowserTest Aug 18, 2021
@fabpot
Copy link
Member

fabpot commented Aug 18, 2021

Thank you @herndlm.

@fabpot fabpot merged commit 83357b1 into symfony:4.4 Aug 18, 2021
@herndlm herndlm deleted the bugfix/fix-spl-file-info-mock-in-http-kernel-browser-test branch August 18, 2021 11:09
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants