Skip to content

Commit

Permalink
Fix tests and dependencies
Browse files Browse the repository at this point in the history
  • Loading branch information
Jose Manuel Cardona committed Mar 1, 2024
1 parent 5dee6bf commit 4c86f2c
Show file tree
Hide file tree
Showing 3 changed files with 17 additions and 6 deletions.
2 changes: 1 addition & 1 deletion composer.json
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@
"guzzlehttp/guzzle": "^6.3 || ^7.0",
"softonic/guzzle-oauth2-middleware": "^2.1",
"ext-json": "*",
"symfony/console": "^5.0 || ^6.0 || ^7.0"
"symfony/console": "^6.0 || ^7.0"
},
"require-dev": {
"friendsofphp/php-cs-fixer": "^3.9",
Expand Down
2 changes: 1 addition & 1 deletion src/Console/Mutation/GenerateConfig.php
Original file line number Diff line number Diff line change
Expand Up @@ -54,7 +54,7 @@ protected function configure()
);
}

protected function execute(InputInterface $input, OutputInterface $output)
protected function execute(InputInterface $input, OutputInterface $output): int
{
if (!$this->checkArguments($input, $output)) {
return 1;
Expand Down
19 changes: 15 additions & 4 deletions tests/ResponseBuilderTest.php
Original file line number Diff line number Diff line change
Expand Up @@ -2,8 +2,10 @@

namespace Softonic\GraphQL;

use GuzzleHttp\Psr7\BufferStream;
use PHPUnit\Framework\TestCase;
use Psr\Http\Message\ResponseInterface;
use Psr\Http\Message\StreamInterface;
use UnexpectedValueException;

class ResponseBuilderTest extends TestCase
Expand All @@ -24,7 +26,7 @@ public function testBuildMalformedResponse()
$mockHttpResponse = $this->createMock(ResponseInterface::class);
$mockHttpResponse->expects($this->once())
->method('getBody')
->willReturn('malformed response');
->willReturn($this->stringToStream('malformed response'));

$this->expectException(UnexpectedValueException::class);
$this->expectExceptionMessage('Invalid JSON response. Response body: ');
Expand Down Expand Up @@ -53,7 +55,7 @@ public function testBuildInvalidGraphqlJsonResponse(string $body)

$mockHttpResponse->expects($this->once())
->method('getBody')
->willReturn($body);
->willReturn($this->stringToStream($body));

$this->expectException(UnexpectedValueException::class);
$this->expectExceptionMessage('Invalid GraphQL JSON response. Response body: ');
Expand All @@ -67,7 +69,7 @@ public function testBuildValidGraphqlJsonWithoutErrors()

$mockHttpResponse->expects($this->once())
->method('getBody')
->willReturn('{"data": {"foo": "bar"}}');
->willReturn($this->stringToStream('{"data": {"foo": "bar"}}'));

$expectedData = ['foo' => 'bar'];
$dataObjectMock = [
Expand Down Expand Up @@ -107,7 +109,7 @@ public function testBuildValidGraphqlJsonWithErrors(string $body)

$mockHttpResponse->expects($this->once())
->method('getBody')
->willReturn($body);
->willReturn($this->stringToStream($body));

$this->dataObjectBuilder->expects($this->once())
->method('buildQuery')
Expand All @@ -121,4 +123,13 @@ public function testBuildValidGraphqlJsonWithErrors(string $body)
$this->assertTrue($response->hasErrors());
$this->assertEquals([['foo' => 'bar']], $response->getErrors());
}

public function stringToStream(string $string): StreamInterface
{
$buffer = new BufferStream();

$buffer->write($string);

return $buffer;
}
}

0 comments on commit 4c86f2c

Please sign in to comment.