Skip to content
This repository was archived by the owner on Mar 23, 2024. It is now read-only.

Commit 9708930

Browse files
committed
:octocat: getTestResponses()
1 parent 9e79642 commit 9708930

File tree

3 files changed

+20
-19
lines changed

3 files changed

+20
-19
lines changed

tests/Providers/OAuth1ProviderTestAbstract.php

Lines changed: 8 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -27,12 +27,6 @@
2727
*/
2828
abstract class OAuth1ProviderTestAbstract extends ProviderTestAbstract{
2929

30-
protected $responses = [
31-
'/oauth1/request_token' => 'oauth_token=test_request_token&oauth_token_secret=test_request_token_secret&oauth_callback_confirmed=true',
32-
'/oauth1/access_token' => 'oauth_token=test_access_token&oauth_token_secret=test_access_token_secret&oauth_callback_confirmed=true',
33-
'/oauth1/api/request' => '{"data":"such data! much wow!"}',
34-
];
35-
3630
protected function setUp():void{
3731
parent::setUp();
3832

@@ -42,6 +36,14 @@ protected function setUp():void{
4236

4337
}
4438

39+
protected function getTestResponses():array{
40+
return [
41+
'/oauth1/request_token' => 'oauth_token=test_request_token&oauth_token_secret=test_request_token_secret&oauth_callback_confirmed=true',
42+
'/oauth1/access_token' => 'oauth_token=test_access_token&oauth_token_secret=test_access_token_secret&oauth_callback_confirmed=true',
43+
'/oauth1/api/request' => '{"data":"such data! much wow!"}',
44+
];
45+
}
46+
4547
public function testOAuth1Instance(){
4648
$this->assertInstanceOf(OAuth1Interface::class, $this->provider);
4749
}

tests/Providers/OAuth2ProviderTestAbstract.php

Lines changed: 9 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -28,13 +28,6 @@
2828
*/
2929
abstract class OAuth2ProviderTestAbstract extends ProviderTestAbstract{
3030

31-
protected $responses = [
32-
'/oauth2/access_token' => '{"access_token":"test_access_token","expires_in":3600,"state":"test_state"}',
33-
'/oauth2/refresh_token' => '{"access_token":"test_refreshed_access_token","expires_in":60,"state":"test_state"}',
34-
'/oauth2/client_credentials' => '{"access_token":"test_client_credentials_token","expires_in":30,"state":"test_state"}',
35-
'/oauth2/api/request' => '{"data":"such data! much wow!"}',
36-
];
37-
3831
protected function setUp():void{
3932
parent::setUp();
4033

@@ -46,6 +39,15 @@ protected function setUp():void{
4639
$this->storage->storeCSRFState($this->provider->serviceName, 'test_state');
4740
}
4841

42+
protected function getTestResponses():array{
43+
return [
44+
'/oauth2/access_token' => '{"access_token":"test_access_token","expires_in":3600,"state":"test_state"}',
45+
'/oauth2/refresh_token' => '{"access_token":"test_refreshed_access_token","expires_in":60,"state":"test_state"}',
46+
'/oauth2/client_credentials' => '{"access_token":"test_client_credentials_token","expires_in":30,"state":"test_state"}',
47+
'/oauth2/api/request' => '{"data":"such data! much wow!"}',
48+
];
49+
}
50+
4951
public function testOAuth2Instance(){
5052
$this->assertInstanceOf(OAuth2Interface::class, $this->provider);
5153
}

tests/Providers/ProviderTestAbstract.php

Lines changed: 3 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -68,11 +68,6 @@ abstract class ProviderTestAbstract extends TestCase{
6868
*/
6969
protected $dotEnv;
7070

71-
/**
72-
* @var array
73-
*/
74-
protected $responses;
75-
7671
/**
7772
* @var bool
7873
*/
@@ -99,12 +94,14 @@ protected function setUp():void{
9994
$this->storage->storeAccessToken($this->provider->serviceName, new AccessToken(['accessToken' => 'foo']));
10095
}
10196

97+
abstract protected function getTestResponses():array;
98+
10299
/**
103100
* @return \Psr\Http\Client\ClientInterface
104101
*/
105102
protected function initHttp():ClientInterface{
106103

107-
$client = new class($this->responses) implements ClientInterface{
104+
$client = new class($this->getTestResponses()) implements ClientInterface{
108105

109106
/** @var array */
110107
protected $responses;

0 commit comments

Comments
 (0)