Skip to content

Commit

Permalink
Refactor tests for nelmio#565
Browse files Browse the repository at this point in the history
  • Loading branch information
rodrigm committed Jan 26, 2016
1 parent e007715 commit 7336130
Show file tree
Hide file tree
Showing 2 changed files with 4 additions and 4 deletions.
4 changes: 2 additions & 2 deletions Tests/Extractor/ApiDocExtractorTest.php
Expand Up @@ -286,13 +286,13 @@ public function testPostRequestDoesRequireParametersWhenMarkedAsSuch()
$this->assertTrue($parameters['required_field']['required']);
}

public function testPutRequestDoesNeverRequireParameters()
public function testPatchRequestDoesNeverRequireParameters()
{
$container = $this->getContainer();
/** @var ApiDocExtractor $extractor */
$extractor = $container->get('nelmio_api_doc.extractor.api_doc_extractor');
/** @var ApiDoc $annotation */
$annotation = $extractor->get('Nelmio\ApiDocBundle\Tests\Fixtures\Controller\TestController::requiredParametersAction', 'test_put_disables_required_parameters');
$annotation = $extractor->get('Nelmio\ApiDocBundle\Tests\Fixtures\Controller\TestController::requiredParametersAction', 'test_patch_disables_required_parameters');

$parameters = $annotation->getParameters();
$this->assertFalse($parameters['required_field']['required']);
Expand Down
4 changes: 2 additions & 2 deletions Tests/Fixtures/app/config/routing.yml
Expand Up @@ -209,9 +209,9 @@ test_required_parameters:
requirements:
_format: json|xml|html

test_put_disables_required_parameters:
test_patch_disables_required_parameters:
path: /api/other-resources/{id}.{_format}
methods: [PUT]
methods: [PATCH]
defaults: { _controller: NelmioApiDocTestBundle:Resource:requiredParametersAction, _format: json }
requirements:
_format: json|xml|html
Expand Down

0 comments on commit 7336130

Please sign in to comment.