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

Skip to content

[PhpUnitBridge] Mock DNS functions#18181

Merged
fabpot merged 1 commit into
symfony:masterfrom
nicolas-grekas:mock-checkdnsrr
Mar 16, 2016
Merged

[PhpUnitBridge] Mock DNS functions#18181
fabpot merged 1 commit into
symfony:masterfrom
nicolas-grekas:mock-checkdnsrr

Conversation

@nicolas-grekas
Copy link
Copy Markdown
Member

@nicolas-grekas nicolas-grekas commented Mar 15, 2016

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

This was referenced Mar 15, 2016
Closed
Closed
@natechicago
Copy link
Copy Markdown
Contributor

natechicago commented Mar 15, 2016

Thanks for this, @nicolas-grekas !

fabpot
fabpot reviewed Mar 15, 2016
View reviewed changes
Comment thread src/Symfony/Bridge/PhpUnit/DnsMock.php Outdated
{
return isset(self::$resolutions[$type][$host])
? self::$resolutions[$type][$host]
: \checkdnsrr($host, $type);
Copy link
Copy Markdown
Member

@fabpot fabpot Mar 15, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

should be on one line :)

@patrick-mcdougle patrick-mcdougle mentioned this pull request Mar 15, 2016
Closed
@nicolas-grekas nicolas-grekas force-pushed the mock-checkdnsrr branch 3 times, most recently from 9f5b856 to d21746c Compare March 16, 2016 10:15
@nicolas-grekas nicolas-grekas changed the title [PhpUnitBridge] Mock checkdnsrr() [PhpUnitBridge] Mock DNS functions Mar 16, 2016
@nicolas-grekas
Copy link
Copy Markdown
Member Author

nicolas-grekas commented Mar 16, 2016

This PR now has mocks for all DNS functions, and unit tests.
Ready for merge.

xabbuh
xabbuh reviewed Mar 16, 2016
View reviewed changes
Comment thread composer.json Outdated
"Symfony\\Bridge\\Monolog\\": "src/Symfony/Bridge/Monolog/",
"Symfony\\Bridge\\ProxyManager\\": "src/Symfony/Bridge/ProxyManager/",
"Symfony\\Bridge\\Twig\\": "src/Symfony/Bridge/Twig/",
"Symfony\\Bridge\\": "src/Symfony/Bridge/",
Copy link
Copy Markdown
Member

@xabbuh xabbuh Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What was the reason we didn't have this before?

Copy link
Copy Markdown
Member Author

@nicolas-grekas nicolas-grekas Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

reverted

@nicolas-grekas nicolas-grekas force-pushed the mock-checkdnsrr branch 2 times, most recently from 0a79ba1 to 7a8b925 Compare March 16, 2016 11:08
nicolas-grekas
nicolas-grekas reviewed Mar 16, 2016
View reviewed changes
Comment thread src/Symfony/Bridge/PhpUnit/Tests/DnsMockTest.php

use Symfony\Bridge\PhpUnit\DnsMock;

require_once __DIR__.'/../DnsMock.php';
Copy link
Copy Markdown
Member Author

@nicolas-grekas nicolas-grekas Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this is temporary until our phpunit wrapper is updated (PR coming on 2.3 after merge)

@nicolas-grekas nicolas-grekas force-pushed the mock-checkdnsrr branch 5 times, most recently from 84455f4 to 1a8bb65 Compare March 16, 2016 12:44
fabpot
fabpot reviewed Mar 16, 2016
View reviewed changes
Comment thread src/Symfony/Bridge/PhpUnit/SymfonyTestsListener.php Outdated
private $isSkipped = array();

public function __construct(array $extraClockMockedNamespaces = array())
public function __construct(array $extraClockMockedNamespaces = array(), array $extraDnsMockedNamespaces = array())
Copy link
Copy Markdown
Member

@fabpot fabpot Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That does not scale well. I think we need to think about another way to inject those mocked configs.

Copy link
Copy Markdown
Member Author

@nicolas-grekas nicolas-grekas Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

fixed

@nicolas-grekas nicolas-grekas force-pushed the mock-checkdnsrr branch 2 times, most recently from 98e58fa to 4619210 Compare March 16, 2016 13:05
xabbuh
xabbuh reviewed Mar 16, 2016
View reviewed changes
Comment thread src/Symfony/Bridge/PhpUnit/SymfonyTestsListener.php
private $isSkipped = array();

public function __construct(array $extraClockMockedNamespaces = array())
public function __construct(array $mockedNamespaces = array())
Copy link
Copy Markdown
Member

@xabbuh xabbuh Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think we should now document which keys (groups) are supported by the listener.

Copy link
Copy Markdown
Member Author

@nicolas-grekas nicolas-grekas Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

documented, ok for you ?

Copy link
Copy Markdown
Member

@xabbuh xabbuh Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looks good

@fabpot
Copy link
Copy Markdown
Member

fabpot commented Mar 16, 2016

Thank you @nicolas-grekas.

@fabpot fabpot merged commit 0b31285 into symfony:master Mar 16, 2016
fabpot added a commit that referenced this pull request Mar 16, 2016
@fabpot
feature #18181 [PhpUnitBridge] Mock DNS functions (nicolas-grekas)
8b1d91f 
This PR was merged into the 3.1-dev branch.

Discussion
----------

[PhpUnitBridge] Mock DNS functions

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

Commits
-------

0b31285 [PhpUnitBridge] Mock DNS functions
@nicolas-grekas nicolas-grekas deleted the mock-checkdnsrr branch March 16, 2016 15:06
stof
stof reviewed Mar 16, 2016
View reviewed changes
Comment thread src/Symfony/Bridge/PhpUnit/SymfonyTestsListener.php
} else {
self::$globallyEnabled = true;
if ($warn) {
echo "Clock-mocked namespaces for SymfonyTestsListener need to be nested in a \"time-sensitive\" key. This will be enforced in Symfony 4.0.\n";
Copy link
Copy Markdown
Member

@stof stof Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

writing this to stderr would be better than in stdout

Copy link
Copy Markdown
Member Author

@nicolas-grekas nicolas-grekas Mar 16, 2016

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

but did not work when I tried so I resorted to stdout, which is where phpunit and this listener echo in other cases...

@xabbuh xabbuh mentioned this pull request Mar 16, 2016
Closed
@fabpot fabpot mentioned this pull request May 13, 2016
Merged
@xabbuh xabbuh mentioned this pull request May 21, 2017
Merged
fabpot added a commit that referenced this pull request May 21, 2017
@fabpot
minor #22835 [PhpUnitBridge] add a changelog file (xabbuh)
7fbda2e 
This PR was merged into the 3.2 branch.

Discussion
----------

[PhpUnitBridge] add a changelog file

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

Commits
-------

e5455db [PhpUnitBridge] add a changelog file
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

Feature PhpUnitBridge Ready Status: Needs Review

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

@nicolas-grekas @natechicago @fabpot @stof @xabbuh @javiereguiluz @carsonbot