Parse operation name from operationName instead of operation

This commit is contained in:
Chris Muthig 2018-03-03 15:41:55 -08:00
parent 94525c0025
commit 8aa6dc17a5
No known key found for this signature in database
GPG Key ID: 28E4EB85B5AB189A
3 changed files with 15 additions and 15 deletions

View File

@ -67,7 +67,7 @@ class OperationParams
'queryid' => null,
'documentid' => null, // alias to queryid
'id' => null, // alias to queryid
'operation' => null,
'operationname' => null,
'variables' => null
];
@ -84,7 +84,7 @@ class OperationParams
$instance->query = $params['query'];
$instance->queryId = $params['queryid'] ?: $params['documentid'] ?: $params['id'];
$instance->operation = $params['operation'];
$instance->operation = $params['operationname'];
$instance->variables = $params['variables'];
$instance->readOnly = (bool) $readonly;

View File

@ -34,7 +34,7 @@ class RequestParsingTest extends \PHPUnit_Framework_TestCase
$post = [
'query' => $query,
'variables' => $variables,
'operation' => $operation
'operationName' => $operation
];
$parsed = [
'raw' => $this->parseRawFormUrlencodedRequest($post),
@ -56,7 +56,7 @@ class RequestParsingTest extends \PHPUnit_Framework_TestCase
$get = [
'query' => $query,
'variables' => $variables,
'operation' => $operation
'operationName' => $operation
];
$parsed = [
'raw' => $this->parseRawGetRequest($get),
@ -78,7 +78,7 @@ class RequestParsingTest extends \PHPUnit_Framework_TestCase
$body = [
'query' => $query,
'variables' => $variables,
'operation' => $operation
'operationName' => $operation
];
$parsed = [
'raw' => $this->parseRawRequest('application/json', json_encode($body)),
@ -99,7 +99,7 @@ class RequestParsingTest extends \PHPUnit_Framework_TestCase
$body = [
'query' => $query,
'variables' => json_encode($variables),
'operation' => $operation
'operationName' => $operation
];
$parsed = [
'raw' => $this->parseRawRequest('application/json', json_encode($body)),
@ -120,7 +120,7 @@ class RequestParsingTest extends \PHPUnit_Framework_TestCase
$body = [
'query' => $query,
'variables' => $variables,
'operation' => $operation
'operationName' => $operation
];
$parsed = [
'raw' => $this->parseRawRequest('application/json', json_encode($body)),
@ -138,12 +138,12 @@ class RequestParsingTest extends \PHPUnit_Framework_TestCase
[
'query' => '{my query}',
'variables' => ['test' => 1, 'test2' => 2],
'operation' => 'op'
'operationName' => 'op'
],
[
'queryId' => 'my-query-id',
'variables' => ['test' => 1, 'test2' => 2],
'operation' => 'op2'
'operationName' => 'op2'
],
];
$parsed = [
@ -153,8 +153,8 @@ class RequestParsingTest extends \PHPUnit_Framework_TestCase
foreach ($parsed as $method => $parsedBody) {
$this->assertInternalType('array', $parsedBody, $method);
$this->assertCount(2, $parsedBody, $method);
$this->assertValidOperationParams($parsedBody[0], $body[0]['query'], null, $body[0]['variables'], $body[0]['operation'], $method);
$this->assertValidOperationParams($parsedBody[1], null, $body[1]['queryId'], $body[1]['variables'], $body[1]['operation'], $method);
$this->assertValidOperationParams($parsedBody[0], $body[0]['query'], null, $body[0]['variables'], $body[0]['operationName'], $method);
$this->assertValidOperationParams($parsedBody[1], null, $body[1]['queryId'], $body[1]['variables'], $body[1]['operationName'], $method);
}
}

View File

@ -15,7 +15,7 @@ class RequestValidationTest extends \PHPUnit_Framework_TestCase
$parsedBody = OperationParams::create([
'query' => $query,
'variables' => $variables,
'operation' => $operation,
'operationName' => $operation,
]);
$this->assertValid($parsedBody);
@ -30,7 +30,7 @@ class RequestValidationTest extends \PHPUnit_Framework_TestCase
$parsedBody = OperationParams::create([
'queryId' => $queryId,
'variables' => $variables,
'operation' => $operation,
'operationName' => $operation,
]);
$this->assertValid($parsedBody);
@ -40,7 +40,7 @@ class RequestValidationTest extends \PHPUnit_Framework_TestCase
{
$parsedBody = OperationParams::create([
'variables' => ['foo' => 'bar'],
'operation' => 'op',
'operationName' => 'op',
]);
$this->assertInputError(
@ -90,7 +90,7 @@ class RequestValidationTest extends \PHPUnit_Framework_TestCase
{
$parsedBody = OperationParams::create([
'query' => '{my query}',
'operation' => []
'operationName' => []
]);
$this->assertInputError(