Allow different methods of batching

[rpander93]

Q	A
Bug fix?	no
New feature?	no
BC breaks?	no
Deprecations?	no
Tests pass?	yes
Documented?	no
Fixed tickets	#163
License	MIT

Makes the batching method configurable. Apollo just wants the plain payload without any id attached to it. Not sure how to properly test the code since it requires switching a config value at run-time which is not possible as I just found out..

/**
 * @param $uri
 * @dataProvider graphQLBatchEndpointUriProvider
 */
public function testBatchEndpointApolloStyleAction($uri)
{
	$client = static::createClient(['test_case' => 'connection']);

	$data = [
		[
			'query' => $this->friendsQuery
		],
		[
			'query' => $this->friendsTotalCountQuery
		]
	];

	$client->request('POST', $uri, [], [], ['CONTENT_TYPE' => 'application/json'], json_encode($data));
	$result = $client->getResponse()->getContent();

	$expected = [
		['data' => $this->expectedData],
		['data' => ['user' => ['friends' => ['totalCount' => 4]]]]
	];
	$this->assertEquals($expected, json_decode($result, true), $result);
}

[lucasfeliciano]

I need this so much. 😄

[mcg-web]

Thank you for this contribution @rpander93. The best way to test this right now is to add a new test case in config that inherits the connection config and override the batching_method then test it in a new test... Can you also provide some documentation please?

[rpander93]

Ok, I added the documentation. I dont get you on the testing stuff unfortunately.

[mcg-web]

I'll add tests later. Need document for the way to add functional tests. Thanks again for contribution 👍

[lucasfeliciano]

🎉

When this will be released?

[rpander93]

Great, thanks!

[mcg-web]

@lucasfeliciano no release date right now but you can use 0.9-dev.