8000 fix controller tests by xabbuh · Pull Request #18282 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

fix controller tests #18282

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
Mar 23, 2016
Merged

fix controller tests #18282

merged 1 commit into from
Mar 23, 2016

Conversation

xabbuh
Copy link
Member
@xabbuh xabbuh commented Mar 23, 2016
Q A
Branch? 3.0
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets items 9 to 12 from sebastianbergmann/phpunit-mock-objects#299 (comment)
License MIT
Doc PR

Add missing public method stubs to TestController (all methods in the base Controller class from the FrameworkBundle are protected since Symfony 3.0).

@xabbuh
Copy link
Member Author
xabbuh commented Mar 23, 2016

Looks like we actually need to do some of these fixes in the 2.7 branch.

@xabbuh
Copy link
Member Author
xabbuh commented Mar 23, 2016

see #18283

@xabbuh
Copy link
Member Author
xabbuh commented Mar 23, 2016

failures on AppVeyor are not related

@xabbuh xabbuh added the Ready label Mar 23, 2016
@dunglas
Copy link
Member
dunglas commented Mar 23, 2016

👍 thanks for fixing this @xabbuh!

@xabbuh xabbuh force-pushed the controller-tests branch from 6ac0be8 to 12c0ac2 Compare March 23, 2016 20:11
Add missing public method stubs to `TestController` (all methods in the
base `Controller` class from the FrameworkBundle are `protected` since
Symfony 3.0).
@xabbuh xabbuh force-pushed the controller-tests branch from 12c0ac2 to fb963d2 Compare March 23, 2016 20:14
@xabbuh xabbuh merged commit fb963d2 into symfony:3.0 Mar 23, 2016
xabbuh added a commit that referenced this pull request Mar 23, 2016
This PR was merged into the 3.0 branch.

Discussion
----------

fix controller tests

| Q             | A
| ------------- | ---
| Branch?       | 3.0
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | items 9 to 12 from sebastianbergmann/phpunit-mock-objects#299 (comment)
| License       | MIT
| Doc PR        |

Add missing public method stubs to `TestController` (all methods in the base `Controller` class from the FrameworkBundle are `protected` since Symfony 3.0).

Commits
-------

fb963d2 fix controller tests
@xabbuh xabbuh deleted the controller-tests branch March 23, 2016 20:22
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.

3 participants
0