Skip to content

Commit ba69b3d

Browse files
authored
Merge pull request #735 from robbieaverill/pulls/2.9/fix-graphql-tests
FIX GraphQL test warnings when they do not assert anything
2 parents fe27af8 + c60cc21 commit ba69b3d

File tree

1 file changed

+4
-2
lines changed

1 file changed

+4
-2
lines changed

test/Github/Tests/Api/GraphQLTest.php

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,7 +27,8 @@ public function shouldSupportGraphQLVariables()
2727
{
2828
$api = $this->getApiMock();
2929

30-
$api->method('post')
30+
$api->expects($this->once())
31+
->method('post')
3132
->with('/graphql', $this->arrayHasKey('variables'));
3233

3334
$api->execute('bar', ['variable' => 'foo']);
@@ -40,7 +41,8 @@ public function shouldJSONEncodeGraphQLVariables()
4041
{
4142
$api = $this->getApiMock();
4243

43-
$api->method('post')
44+
$api->expects($this->once())
45+
->method('post')
4446
->with('/graphql', $this->equalTo([
4547
'query'=>'bar',
4648
'variables' => '{"variable":"foo"}',

0 commit comments

Comments
 (0)