Ecodev /
graphql-upload
| 1 | <?php |
||
| 2 | |||
| 3 | declare(strict_types=1); |
||
| 4 | |||
| 5 | namespace GraphQLTests\Upload; |
||
| 6 | |||
| 7 | use GraphQL\Error\DebugFlag; |
||
| 8 | use GraphQL\Error\InvariantViolation; |
||
| 9 | use GraphQL\Executor\ExecutionResult; |
||
| 10 | use GraphQL\Server\RequestError; |
||
| 11 | use GraphQL\Server\StandardServer; |
||
| 12 | use GraphQL\Type\Definition\ObjectType; |
||
| 13 | use GraphQL\Type\Definition\Type; |
||
| 14 | use GraphQL\Type\Schema; |
||
| 15 | use GraphQL\Upload\UploadMiddleware; |
||
| 16 | use GraphQL\Upload\UploadType; |
||
| 17 | use GraphQL\Upload\Utility; |
||
| 18 | use GraphQLTests\Upload\Psr7\PsrUploadedFileStub; |
||
| 19 | use Laminas\Diactoros\Response; |
||
| 20 | use Laminas\Diactoros\ServerRequest; |
||
| 21 | use Laminas\Diactoros\UploadedFile; |
||
| 22 | use PHPUnit\Framework\TestCase; |
||
| 23 | use Psr\Http\Message\ResponseInterface; |
||
| 24 | use Psr\Http\Message\ServerRequestInterface; |
||
| 25 | use Psr\Http\Message\UploadedFileInterface; |
||
| 26 | use Psr\Http\Server\RequestHandlerInterface; |
||
| 27 | use stdClass; |
||
| 28 | |||
| 29 | final class UploadMiddlewareTest extends TestCase |
||
| 30 | { |
||
| 31 | public function testParsesMultipartRequest(): void |
||
| 32 | { |
||
| 33 | $query = '{my query}'; |
||
| 34 | $variables = [ |
||
| 35 | 'test' => 1, |
||
| 36 | 'test2' => 2, |
||
| 37 | 'uploads' => [ |
||
| 38 | 0 => null, |
||
| 39 | 1 => null, |
||
| 40 | ], |
||
| 41 | ]; |
||
| 42 | $map = [ |
||
| 43 | 1 => ['variables.uploads.0'], |
||
| 44 | 2 => ['variables.uploads.1'], |
||
| 45 | ]; |
||
| 46 | |||
| 47 | $file1 = new PsrUploadedFileStub('image.jpg', 'image/jpeg'); |
||
| 48 | $file2 = new PsrUploadedFileStub('foo.txt', 'text/plain'); |
||
| 49 | $files = [ |
||
| 50 | 1 => $file1, |
||
| 51 | 2 => $file2, |
||
| 52 | ]; |
||
| 53 | |||
| 54 | $request = $this->createRequest($query, $variables, $map, $files, 'op'); |
||
| 55 | $processedRequest = $this->getProcessedRequest($request); |
||
| 56 | |||
| 57 | $variables['uploads'] = [ |
||
| 58 | 0 => $file1, |
||
| 59 | 1 => $file2, |
||
| 60 | ]; |
||
| 61 | |||
| 62 | self::assertSame('application/json', $processedRequest->getHeader('content-type')[0], 'request should have been transformed as application/json'); |
||
| 63 | self::assertSame($variables, $processedRequest->getParsedBody()['variables'], 'uploaded files should have been injected into variables'); |
||
| 64 | } |
||
| 65 | |||
| 66 | public function testEmptyRequestIsValid(): void |
||
| 67 | { |
||
| 68 | $request = $this->createRequest('{my query}', [], [], [], 'op'); |
||
| 69 | $processedRequest = $this->getProcessedRequest($request); |
||
| 70 | |||
| 71 | self::assertSame('application/json', $processedRequest->getHeader('content-type')[0], 'request should have been transformed as application/json'); |
||
| 72 | self::assertSame([], $processedRequest->getParsedBody()['variables'], 'variables should still be empty'); |
||
| 73 | } |
||
| 74 | |||
| 75 | public function testNonMultipartRequestAreNotTouched(): void |
||
| 76 | { |
||
| 77 | $request = new ServerRequest(); |
||
| 78 | $processedRequest = $this->getProcessedRequest($request); |
||
| 79 | |||
| 80 | self::assertSame($request, $processedRequest, 'request should have been transformed as application/json'); |
||
| 81 | } |
||
| 82 | |||
| 83 | public function testEmptyRequestShouldThrow(): void |
||
| 84 | { |
||
| 85 | $request = new ServerRequest(); |
||
| 86 | $request = $request |
||
| 87 | ->withHeader('content-type', ['multipart/form-data']) |
||
| 88 | ->withParsedBody([]); |
||
| 89 | |||
| 90 | $this->expectException(InvariantViolation::class); |
||
| 91 | $this->expectExceptionMessage('PSR-7 request is expected to provide parsed body for "multipart/form-data" requests but got empty array'); |
||
| 92 | $this->getProcessedRequest($request); |
||
| 93 | } |
||
| 94 | |||
| 95 | public function testNullRequestShouldThrow(): void |
||
| 96 | { |
||
| 97 | $request = new ServerRequest(); |
||
| 98 | $request = $request |
||
| 99 | ->withHeader('content-type', ['multipart/form-data']) |
||
| 100 | ->withParsedBody(null); |
||
| 101 | |||
| 102 | $this->expectException(InvariantViolation::class); |
||
| 103 | $this->expectExceptionMessage('PSR-7 request is expected to provide parsed body for "multipart/form-data" requests but got null'); |
||
| 104 | $this->getProcessedRequest($request); |
||
| 105 | } |
||
| 106 | |||
| 107 | public function testInvalidRequestShouldThrow(): void |
||
| 108 | { |
||
| 109 | $request = new ServerRequest(); |
||
| 110 | $request = $request |
||
| 111 | ->withHeader('content-type', ['multipart/form-data']) |
||
| 112 | ->withParsedBody(new stdClass()); |
||
| 113 | |||
| 114 | $this->expectException(RequestError::class); |
||
| 115 | $this->expectExceptionMessage('GraphQL Server expects JSON object or array, but got {}'); |
||
| 116 | $this->getProcessedRequest($request); |
||
| 117 | } |
||
| 118 | |||
| 119 | public function testOtherContentTypeShouldNotBeTouched(): void |
||
| 120 | { |
||
| 121 | $request = new ServerRequest(); |
||
| 122 | $request = $request |
||
| 123 | ->withHeader('content-type', ['application/json']) |
||
| 124 | ->withParsedBody(new stdClass()); |
||
| 125 | |||
| 126 | $processedRequest = $this->getProcessedRequest($request); |
||
| 127 | self::assertSame($request, $processedRequest); |
||
| 128 | } |
||
| 129 | |||
| 130 | public function testRequestWithoutMapShouldThrow(): void |
||
| 131 | { |
||
| 132 | $request = $this->createRequest('{my query}', [], [], [], 'op'); |
||
| 133 | |||
| 134 | // Remove the map |
||
| 135 | $body = $request->getParsedBody(); |
||
| 136 | unset($body['map']); |
||
| 137 | $request = $request->withParsedBody($body); |
||
| 138 | |||
| 139 | $this->expectException(RequestError::class); |
||
| 140 | $this->expectExceptionMessage('The request must define a `map`'); |
||
| 141 | $this->getProcessedRequest($request); |
||
| 142 | } |
||
| 143 | |||
| 144 | public function testRequestWithMapThatIsNotArrayShouldThrow(): void |
||
| 145 | { |
||
| 146 | $request = $this->createRequest('{my query}', [], [], [], 'op'); |
||
| 147 | |||
| 148 | // Replace map with json that is valid but no array |
||
| 149 | $body = $request->getParsedBody(); |
||
| 150 | $body['map'] = json_encode('foo'); |
||
| 151 | $request = $request->withParsedBody($body); |
||
| 152 | |||
| 153 | $this->expectException(RequestError::class); |
||
| 154 | $this->expectExceptionMessage('The `map` key must be a JSON encoded array'); |
||
| 155 | $this->getProcessedRequest($request); |
||
| 156 | } |
||
| 157 | |||
| 158 | public function testRequestWithMapThatIsNotValidJsonShouldThrow(): void |
||
| 159 | { |
||
| 160 | $request = $this->createRequest('{my query}', [], [], [], 'op'); |
||
| 161 | |||
| 162 | // Replace map with invalid json |
||
| 163 | $body = $request->getParsedBody(); |
||
| 164 | $body['map'] = 'this is not json'; |
||
| 165 | $request = $request->withParsedBody($body); |
||
| 166 | |||
| 167 | $this->expectException(RequestError::class); |
||
| 168 | $this->expectExceptionMessage('The `map` key must be a JSON encoded array'); |
||
| 169 | $this->getProcessedRequest($request); |
||
| 170 | } |
||
| 171 | |||
| 172 | public function testRequestWithTooBigPostSizeShouldReturnHttpError413WithMessage(): void |
||
| 173 | { |
||
| 174 | $postMaxSize = Utility::getPostMaxSize(); |
||
| 175 | $contentLength = (string) ($postMaxSize * 2); |
||
| 176 | $request = $this->createRequest('{my query}', [], [], [], 'op', ['CONTENT_LENGTH' => $contentLength]); |
||
| 177 | |||
| 178 | $contentLength = Utility::toMebibyte($contentLength); |
||
| 179 | $postMaxSize = Utility::toMebibyte($postMaxSize); |
||
| 180 | |||
| 181 | $response = $this->getResponse($request); |
||
| 182 | self::assertSame(413, $response->getStatusCode()); |
||
| 183 | self::assertSame('{"message":"The server `post_max_size` is configured to accept ' . $postMaxSize . ', but received ' . $contentLength . '"}', $response->getBody()->getContents()); |
||
| 184 | } |
||
| 185 | |||
| 186 | public function testRequestWithSmallerPostSizeShouldBeOK(): void |
||
| 187 | { |
||
| 188 | $postMaxSize = Utility::getPostMaxSize(); |
||
| 189 | $contentLength = (string) $postMaxSize; |
||
| 190 | $request = $this->createRequest('{my query}', [], [], [], 'op', ['CONTENT_LENGTH' => $contentLength]); |
||
| 191 | |||
| 192 | $processedRequest = $this->getProcessedRequest($request); |
||
| 193 | |||
| 194 | self::assertSame('application/json', $processedRequest->getHeader('content-type')[0], 'request should have been transformed as application/json'); |
||
| 195 | self::assertSame([], $processedRequest->getParsedBody()['variables'], 'variables should still be empty'); |
||
| 196 | |||
| 197 | } |
||
| 198 | |||
| 199 | public function testMissingUploadedFileShouldThrow(): void |
||
| 200 | { |
||
| 201 | $query = '{my query}'; |
||
| 202 | $variables = [ |
||
| 203 | 'uploads' => [ |
||
| 204 | 0 => null, |
||
| 205 | 1 => null, |
||
| 206 | ], |
||
| 207 | ]; |
||
| 208 | $map = [ |
||
| 209 | 1 => ['variables.uploads.0'], |
||
| 210 | 2 => ['variables.uploads.1'], |
||
| 211 | ]; |
||
| 212 | |||
| 213 | $file1 = new PsrUploadedFileStub('image.jpg', 'image/jpeg'); |
||
| 214 | $files = [ |
||
| 215 | 1 => $file1, |
||
| 216 | ]; |
||
| 217 | |||
| 218 | $request = $this->createRequest($query, $variables, $map, $files, 'op'); |
||
| 219 | |||
| 220 | $this->expectException(RequestError::class); |
||
| 221 | $this->expectExceptionMessage('GraphQL query declared an upload in `variables.uploads.1`, but no corresponding file were actually uploaded'); |
||
| 222 | $this->getProcessedRequest($request); |
||
| 223 | } |
||
| 224 | |||
| 225 | public function testCanUploadFileWithStandardServer(): void |
||
| 226 | { |
||
| 227 | $query = 'mutation TestUpload($text: String, $file: Upload) { |
||
| 228 | testUpload(text: $text, file: $file) |
||
| 229 | }'; |
||
| 230 | $variables = [ |
||
| 231 | 'text' => 'foo bar', |
||
| 232 | 'file' => null, |
||
| 233 | ]; |
||
| 234 | $map = [ |
||
| 235 | 1 => ['variables.file'], |
||
| 236 | ]; |
||
| 237 | $files = [ |
||
| 238 | 1 => new PsrUploadedFileStub('image.jpg', 'image/jpeg'), |
||
| 239 | ]; |
||
| 240 | |||
| 241 | $request = $this->createRequest($query, $variables, $map, $files, 'TestUpload'); |
||
| 242 | |||
| 243 | $processedRequest = $this->getProcessedRequest($request); |
||
| 244 | |||
| 245 | $server = $this->createServer(); |
||
| 246 | |||
| 247 | /** @var ExecutionResult $response */ |
||
| 248 | $response = $server->executePsrRequest($processedRequest); |
||
| 249 | |||
| 250 | $expected = ['testUpload' => 'Uploaded file was image.jpg (image/jpeg) with description: foo bar']; |
||
| 251 | self::assertSame($expected, $response->data); |
||
| 252 | } |
||
| 253 | |||
| 254 | private function getProcessedRequest(ServerRequestInterface $request): ServerRequestInterface |
||
| 255 | { |
||
| 256 | $result = $this->process($request); |
||
| 257 | self::assertInstanceOf(ServerRequestInterface::class, $result); |
||
| 258 | |||
| 259 | return $result; |
||
|
0 ignored issues
–
show
Bug
Best Practice
introduced
by
Loading history...
|
|||
| 260 | } |
||
| 261 | |||
| 262 | private function getResponse(ServerRequestInterface $request): ResponseInterface |
||
| 263 | { |
||
| 264 | $result = $this->process($request); |
||
| 265 | self::assertInstanceOf(ResponseInterface::class, $result); |
||
| 266 | |||
| 267 | return $result; |
||
| 268 | } |
||
| 269 | |||
| 270 | private function process(ServerRequestInterface $request): ResponseInterface|ServerRequestInterface |
||
| 271 | { |
||
| 272 | $defaultResponse = new Response\EmptyResponse(); |
||
| 273 | $handler = new class($defaultResponse) implements RequestHandlerInterface { |
||
| 274 | public ServerRequestInterface $processedRequest; |
||
| 275 | |||
| 276 | public function __construct( |
||
| 277 | private readonly ResponseInterface $response, |
||
| 278 | ) {} |
||
| 279 | |||
| 280 | public function handle(ServerRequestInterface $request): ResponseInterface |
||
| 281 | { |
||
| 282 | $this->processedRequest = $request; |
||
| 283 | |||
| 284 | return $this->response; |
||
| 285 | } |
||
| 286 | }; |
||
| 287 | |||
| 288 | $middleware = new UploadMiddleware(); |
||
| 289 | |||
| 290 | $actualResponse = $middleware->process($request, $handler); |
||
| 291 | |||
| 292 | return $actualResponse === $defaultResponse ? $handler->processedRequest : $actualResponse; |
||
| 293 | } |
||
| 294 | |||
| 295 | /** |
||
| 296 | * @param mixed[] $variables |
||
| 297 | * @param string[][] $map |
||
| 298 | * @param UploadedFile[] $files |
||
| 299 | * @param mixed[] $serverParams |
||
| 300 | */ |
||
| 301 | private function createRequest( |
||
| 302 | string $query, |
||
| 303 | array $variables, |
||
| 304 | array $map, |
||
| 305 | array $files, |
||
| 306 | string $operation, |
||
| 307 | array $serverParams = [], |
||
| 308 | ): ServerRequestInterface { |
||
| 309 | $request = new ServerRequest($serverParams); |
||
| 310 | $request = $request |
||
| 311 | ->withMethod('POST') |
||
| 312 | ->withHeader('content-type', ['multipart/form-data; boundary=----WebKitFormBoundarySl4GaqVa1r8GtAbn']) |
||
| 313 | ->withParsedBody([ |
||
| 314 | 'operations' => json_encode([ |
||
| 315 | 'query' => $query, |
||
| 316 | 'variables' => $variables, |
||
| 317 | 'operationName' => $operation, |
||
| 318 | ]), |
||
| 319 | 'map' => json_encode($map), |
||
| 320 | ]) |
||
| 321 | ->withUploadedFiles($files); |
||
| 322 | |||
| 323 | return $request; |
||
| 324 | } |
||
| 325 | |||
| 326 | private function createServer(): StandardServer |
||
| 327 | { |
||
| 328 | $all = DebugFlag::INCLUDE_DEBUG_MESSAGE |
||
| 329 | | DebugFlag::INCLUDE_TRACE |
||
| 330 | | DebugFlag::RETHROW_INTERNAL_EXCEPTIONS |
||
| 331 | | DebugFlag::RETHROW_UNSAFE_EXCEPTIONS; |
||
| 332 | |||
| 333 | return new StandardServer([ |
||
| 334 | 'debugFlag' => $all, |
||
| 335 | 'schema' => new Schema([ |
||
| 336 | 'query' => new ObjectType([ |
||
| 337 | 'name' => 'Query', |
||
| 338 | 'fields' => [], |
||
| 339 | ]), |
||
| 340 | 'mutation' => new ObjectType([ |
||
| 341 | 'name' => 'Mutation', |
||
| 342 | 'fields' => [ |
||
| 343 | 'testUpload' => [ |
||
| 344 | 'type' => Type::string(), |
||
| 345 | 'args' => [ |
||
| 346 | 'text' => Type::string(), |
||
| 347 | 'file' => new UploadType(), |
||
| 348 | ], |
||
| 349 | 'resolve' => function ($root, array $args): string { |
||
| 350 | /** @var UploadedFileInterface $file */ |
||
| 351 | $file = $args['file']; |
||
| 352 | $this->assertInstanceOf(UploadedFileInterface::class, $file); |
||
| 353 | |||
| 354 | // Do something more interesting with the file |
||
| 355 | // $file->moveTo('some/folder/in/my/project'); |
||
| 356 | |||
| 357 | return 'Uploaded file was ' . $file->getClientFilename() . ' (' . $file->getClientMediaType() . ') with description: ' . $args['text']; |
||
| 358 | }, |
||
| 359 | ], |
||
| 360 | ], |
||
| 361 | ]), |
||
| 362 | ]), |
||
| 363 | ]); |
||
| 364 | } |
||
| 365 | } |
||
| 366 |