Skip to content

[FrameworkBundle] Decouple ControllerResolverTest from HttpKernel #61328

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

Merged
merged 1 commit into from
Aug 4, 2025

Conversation

nicolas-grekas
Copy link
Member

Q A
Branch? 7.4
Bug fix? no
New feature? no
Deprecations? no
Issues -
License MIT

Using non-public API for cross-components test case dependencies complicates maintenance.

@fabpot fabpot added this to the 7.4 milestone Aug 4, 2025
@nicolas-grekas nicolas-grekas merged commit 9c835fc into symfony:7.4 Aug 4, 2025
12 checks passed
nicolas-grekas added a commit that referenced this pull request Aug 4, 2025
…ttpKernel (nicolas-grekas)

This PR was merged into the 7.4 branch.

Discussion
----------

[FrameworkBundle] Decouple ControllerResolverTest from HttpKernel

| Q             | A
| ------------- | ---
| Branch?       | 7.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Issues        | -
| License       | MIT

Using non-public API for cross-components test case dependencies complicates maintenance.

Commits
-------

4e9c9c1 [FrameworkBundle] Decouple ControllerResolverTest from HttpKernel
@nicolas-grekas nicolas-grekas deleted the fwb-test-ctrl branch August 4, 2025 16:59
nicolas-grekas added a commit that referenced this pull request Aug 5, 2025
* 7.3:
  [Scheduler] Fix `scheduler.task` tag arguments optionality
  use false instead of null to hide the currency symbol
  [FrameworkBundle] Fix block type from `OK` to `ERROR` when local vault is disabled in `SecretsRemoveCommand`
  Fix wrong boolean values
  minor #61328 [FrameworkBundle] Decouple ControllerResolverTest from HttpKernel (nicolas-grekas)
  [Messenger] Fix NoAutoAckStamp handling in Worker::flush()
nicolas-grekas added a commit that referenced this pull request Aug 5, 2025
* 7.4:
  Use phpunit attribute
  Remove some implicit bool type juggling
  [Scheduler] Fix `scheduler.task` tag arguments optionality
  use false instead of null to hide the currency symbol
  [FrameworkBundle] Fix block type from `OK` to `ERROR` when local vault is disabled in `SecretsRemoveCommand`
  run tests with PHPUnit 12.1 on PHP >= 8.3
  Fix wrong boolean values
  minor #61328 [FrameworkBundle] Decouple ControllerResolverTest from HttpKernel (nicolas-grekas)
  [Messenger] Fix NoAutoAckStamp handling in Worker::flush()
  [DependencyInjection] Dump XML using plain PHP, no DOM needed
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.

2 participants
pFad - Phonifier reborn

Pfad - The Proxy pFad of © 2024 Garber Painting. All rights reserved.

Note: This service is not intended for secure transactions such as banking, social media, email, or purchasing. Use at your own risk. We assume no liability whatsoever for broken pages.


Alternative Proxies:

Alternative Proxy

pFad Proxy

pFad v3 Proxy

pFad v4 Proxy