Skip to content

Fix SwaggerCommandTest #836

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
Nov 8, 2016
Merged

Conversation

dunglas
Copy link
Member

@dunglas dunglas commented Nov 8, 2016

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

ping @Simperfit

@dunglas dunglas force-pushed the fix_swagger_cmd_test branch from d4dc4f9 to a380d0a Compare November 8, 2016 15:59
use Symfony\Component\Console\Tester\ApplicationTester;

/**
* @author Amrouche Hamza <[email protected]>
*/
class SwaggerComandTest extends WebTestCase
class SwaggerCommandTest extends KernelTestCase
Copy link
Contributor

Choose a reason for hiding this comment

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

Good catch ! I didn't see that in the first place

@dunglas dunglas merged commit bdef7dd into api-platform:master Nov 8, 2016
@dunglas dunglas deleted the fix_swagger_cmd_test branch November 8, 2016 16:06
$application = new Application(static::$kernel);
$application->setCatchExceptions(false);
$application->setAutoExit(false);

$tester = new ApplicationTester($application);
Copy link
Contributor

Choose a reason for hiding this comment

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

magarzon pushed a commit to magarzon/core that referenced this pull request Feb 12, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants