Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[PhpUnitBridge] Added ClassExistsMock #10528

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.

Sign up for GitHub

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

Jump to bottom
Closed
ro0NL opened this issue Oct 21, 2018 · 1 comment
Closed

[PhpUnitBridge] Added ClassExistsMock #10528

ro0NL opened this issue Oct 21, 2018 · 1 comment
Labels
hasPR A Pull Request has already been submitted for this issue. PHPUnitBridge
Milestone
4.3

Comments

@ro0NL
Copy link
Contributor

ro0NL commented Oct 21, 2018

see symfony/symfony#28931
@ro0NL ro0NL mentioned this issue Oct 21, 2018
Merged
@xabbuh xabbuh added PHPUnitBridge Waiting Code Merge Docs for features pending to be merged labels Oct 23, 2018
symfony-splitter pushed a commit to symfony/phpunit-bridge that referenced this issue Dec 10, 2018
@fabpot
feature #28931 [PhpUnitBridge] Added ClassExistsMock (ro0NL)
4ee6e7a 
This PR was squashed before being merged into the 4.3-dev branch (closes #28931).

Discussion
----------

[PhpUnitBridge] Added ClassExistsMock

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | #...   <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | symfony/symfony-docs#10528

I've thought about this before, and bumped into it again when trying to test #28898

This PR allows to mock `class|interface|trait_exists` to enable specific code path testing

Commits
-------

62caec1a79 [PhpUnitBridge] Added ClassExistsMock
fabpot added a commit to symfony/symfony that referenced this issue Dec 10, 2018
@fabpot
feature #28931 [PhpUnitBridge] Added ClassExistsMock (ro0NL)
a9694f7 
This PR was squashed before being merged into the 4.3-dev branch (closes #28931).

Discussion
----------

[PhpUnitBridge] Added ClassExistsMock

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | #...   <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | symfony/symfony-docs#10528

I've thought about this before, and bumped into it again when trying to test #28898

This PR allows to mock `class|interface|trait_exists` to enable specific code path testing

Commits
-------

62caec1 [PhpUnitBridge] Added ClassExistsMock
@nicolas-grekas
Copy link
Member

ping @ro0NL don't forget about this if you can work on it please.

@xabbuh xabbuh added this to the 4.3 milestone Jun 18, 2019
@ro0NL ro0NL mentioned this issue Jul 7, 2019
Merged
@xabbuh xabbuh added hasPR A Pull Request has already been submitted for this issue. and removed Waiting Code Merge Docs for features pending to be merged labels Jul 30, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue. PHPUnitBridge
Projects
None yet
Milestone
4.3
Development

No branches or pull requests
3 participants
@nicolas-grekas @ro0NL @xabbuh