Skip to content
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

[VarDumper] Fix test patern to handle callstack with/without return typehint #33067

Merged

Conversation

jderusse
Copy link
Member

@jderusse jderusse commented Aug 8, 2019

Q A
Branch? 4.3
Bug fix? yes
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #32844
License MIT
Doc PR NA

The TestCase::tearDownAfterClass methods does not always have the same signature which change the output of the reflection. This use another methods for testing

@jderusse jderusse force-pushed the fix-test-testFromCallableClosureCaster branch from c480bd9 to feaadd1 Compare August 8, 2019 19:16
@jderusse jderusse changed the base branch from 4.4 to 4.3 August 8, 2019 19:35
@nicolas-grekas nicolas-grekas added this to the 4.3 milestone Aug 8, 2019
@nicolas-grekas
Copy link
Member

Thank you @jderusse.

@nicolas-grekas nicolas-grekas merged commit feaadd1 into symfony:4.3 Aug 8, 2019
nicolas-grekas added a commit that referenced this pull request Aug 8, 2019
…hout return typehint (jderusse)

This PR was merged into the 4.3 branch.

Discussion
----------

[VarDumper] Fix test patern to handle callstack with/without return typehint

| Q             | A
| ------------- | ---
| Branch?       | 4.3
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #32844
| License       | MIT
| Doc PR        | NA

The TestCase::tearDownAfterClass methods does not always have the same signature which change the output of the reflection. This use another methods for testing

Commits
-------

feaadd1 Fix tst patern to handle callstack with/without return typehint
@jderusse jderusse deleted the fix-test-testFromCallableClosureCaster branch March 5, 2020 20:03
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.

None yet

3 participants