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

[PhpUnitBridge] respect skipped and incomplete tests #21121

Merged

Conversation

xabbuh
Copy link
Member

@xabbuh xabbuh commented Jan 1, 2017

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

@xabbuh xabbuh force-pushed the phpunit-skipped-incomplete-tests branch from fad9f0a to 0c06093 Compare January 2, 2017 14:25
@nicolas-grekas
Copy link
Member

Thank you @xabbuh.

@nicolas-grekas nicolas-grekas merged commit 0c06093 into symfony:3.2 Jan 2, 2017
nicolas-grekas added a commit that referenced this pull request Jan 2, 2017
This PR was merged into the 3.2 branch.

Discussion
----------

[PhpUnitBridge] respect skipped and incomplete tests

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

Commits
-------

0c06093 respect skipped and incomplete tests
@stof
Copy link
Member

stof commented Jan 2, 2017

what happens when it is already in errored state ?

@xabbuh xabbuh deleted the phpunit-skipped-incomplete-tests branch January 2, 2017 14:54
fabpot added a commit that referenced this pull request Jan 2, 2017
…oo (xabbuh)

This PR was merged into the 3.2 branch.

Discussion
----------

[PhpUnitBridge] skip tests with failure and error states too

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

I was actually wrong by not excluding failures and errors to in #21121.

Commits
-------

f3bb206 skip tests with failure and error states too
@fabpot fabpot mentioned this pull request Jan 12, 2017
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

4 participants