Skip to content

Fix SplFileInfo mock in HttpKernelBrowserTest #42618

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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