Skip to content

Commit f71cf31

Browse files
committed
Fixing unit tests
1 parent d12eaaa commit f71cf31

File tree

2 files changed

+4
-61
lines changed

2 files changed

+4
-61
lines changed

src/Controllers/GraphQLiteController.php

Lines changed: 1 addition & 58 deletions
Original file line numberDiff line numberDiff line change
@@ -27,13 +27,6 @@ class GraphQLiteController
2727
private $httpMessageFactory;
2828
/** @var StandardServer */
2929
private $standardServer;
30-
/** @var string[] */
31-
private $graphqlHeaderList = ['application/graphql'];
32-
/** @var string[] */
33-
private $allowedMethods = [
34-
'GET',
35-
'POST',
36-
];
3730
/** @var bool|int */
3831
private $debug;
3932

@@ -53,10 +46,6 @@ public function __construct(StandardServer $standardServer, HttpMessageFactoryIn
5346
*/
5447
public function index(Request $request): JsonResponse
5548
{
56-
/*if (!$this->isGraphqlRequest($request)) {
57-
return $next($request);
58-
}*/
59-
6049
$psr7Request = $this->httpMessageFactory->createRequest($request);
6150

6251
if (strtoupper($request->getMethod()) === "POST" && empty($psr7Request->getParsedBody())) {
@@ -72,26 +61,13 @@ public function index(Request $request): JsonResponse
7261
$uploadMiddleware = new UploadMiddleware();
7362
$psr7Request = $uploadMiddleware->processRequest($psr7Request);
7463

75-
76-
// Hack for Graph
77-
/*if (strtoupper($request->getMethod()) == "GET") {
78-
$params = $request->getQueryParams();
79-
$params["variables"] = $params["variables"] === 'undefined' ? null : $params["variables"];
80-
$request = $request->withQueryParams($params);
81-
} else {
82-
$params = $request->getParsedBody();
83-
$params["variables"] = $params["variables"] === 'undefined' ? null : $params["variables"];
84-
$request = $request->withParsedBody($params);
85-
}*/
86-
8764
$result = $this->handlePsr7Request($psr7Request);
8865

8966
$response = new JsonResponse($result);
9067

9168
return $response;
9269
}
9370

94-
9571
private function handlePsr7Request(ServerRequestInterface $request): array
9672
{
9773
$result = $this->standardServer->executePsrRequest($request);
@@ -109,37 +85,4 @@ private function handlePsr7Request(ServerRequestInterface $request): array
10985
}
11086
throw new RuntimeException('Unexpected response from StandardServer::executePsrRequest'); // @codeCoverageIgnore
11187
}
112-
113-
/*private function isGraphqlRequest(Request $request) : bool
114-
{
115-
return $this->isMethodAllowed($request) && ($this->hasUri($request) || $this->hasGraphqlHeader($request));
116-
}
117-
118-
private function isMethodAllowed(Request $request) : bool
119-
{
120-
return in_array($request->getMethod(), $this->allowedMethods, true);
121-
}
122-
123-
private function hasUri(Request $request) : bool
124-
{
125-
return $this->graphqlUri === $request->getPathInfo();
126-
}
127-
128-
private function hasGraphqlHeader(Request $request) : bool
129-
{
130-
if (! $request->headers->has('content-type')) {
131-
return false;
132-
}
133-
134-
$requestHeaderList = $request->headers->get('content-type', null, false);
135-
if ($requestHeaderList === null) {
136-
return false;
137-
}
138-
foreach ($this->graphqlHeaderList as $allowedHeader) {
139-
if (in_array($allowedHeader, $requestHeaderList, true)) {
140-
return true;
141-
}
142-
}
143-
return false;
144-
}*/
145-
}
88+
}

tests/Providers/GraphQLiteServiceProviderTest.php

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,8 +22,8 @@ public function testServiceProvider()
2222

2323
public function testHttpQuery()
2424
{
25-
$response = $this->json('POST', '/graphql', ['query' => '{ hello(name: "David") }']);
26-
echo $response->getStatusCode();
27-
echo $response->getContent();
25+
$response = $this->json('POST', '/graphql', ['query' => '{ dummyQuery }']);
26+
$this->assertSame(200, $response->getStatusCode());
27+
$response->assertJson(["data" => ["dummyQuery" => "This is a placeholder query. Please create a query using the @Query annotation."]]);
2828
}
2929
}

0 commit comments

Comments
 (0)