Skip to content

Commit 095f8e5

Browse files
committed
replace StyleCI with Laravel Pint
1 parent fb7e312 commit 095f8e5

6 files changed

+18
-19
lines changed

.styleci.yml

-5
This file was deleted.

composer.json

+1
Original file line numberDiff line numberDiff line change
@@ -26,6 +26,7 @@
2626
"joelbutcher/facebook-graph-sdk": "^6.1.2"
2727
},
2828
"require-dev": {
29+
"laravel/pint": "^1.20",
2930
"mockery/mockery": "^1.4.2",
3031
"orchestra/testbench": "^6.0|^7.0|^9.0",
3132
"pestphp/pest": "^1.21|^2.34",

pint.json

+3
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,3 @@
1+
{
2+
"preset": "per"
3+
}

src/FacebookServiceProvider.php

+2-2
Original file line numberDiff line numberDiff line change
@@ -67,7 +67,7 @@ protected function mergeConfig(): void
6767
*/
6868
protected function getConfigPath(): string
6969
{
70-
return __DIR__.'/../config/config.php';
70+
return __DIR__ . '/../config/config.php';
7171
}
7272

7373
/**
@@ -90,7 +90,7 @@ protected function registerDefaultHttpClient(): void
9090
protected function registerUrlDetectionHandler(): void
9191
{
9292
$this->app->singleton(UrlDetectionInterface::class, function () {
93-
return new UrlDetectionHandler;
93+
return new UrlDetectionHandler();
9494
});
9595
}
9696

tests/FacebookTest.php

+11-11
Original file line numberDiff line numberDiff line change
@@ -69,32 +69,32 @@
6969

7070
it('builds default scopes', function () {
7171
$redirect = $this->getFacebookMock()->getRedirect();
72-
$this->assertStringContainsStringIgnoringCase('scope='.urlencode('email,public_profile'), $redirect);
72+
$this->assertStringContainsStringIgnoringCase('scope=' . urlencode('email,public_profile'), $redirect);
7373
});
7474

7575
it('replaces duplicate scopes from class config', function () {
7676
$redirect = $this->getFacebookMock(['scopes' => ['email', 'public_profile']])->getRedirect();
77-
$this->assertStringContainsStringIgnoringCase('scope='.urlencode('email,public_profile'), $redirect);
77+
$this->assertStringContainsStringIgnoringCase('scope=' . urlencode('email,public_profile'), $redirect);
7878
});
7979

8080
it('replaces duplicate scopes passed to method', function () {
8181
$redirect = $this->getFacebookMock()->getRedirect(null, ['email', 'public_profile']);
82-
$this->assertStringContainsStringIgnoringCase('scope='.urlencode('email,public_profile'), $redirect);
82+
$this->assertStringContainsStringIgnoringCase('scope=' . urlencode('email,public_profile'), $redirect);
8383
});
8484

8585
it('merges config scopes with default scopes', function () {
8686
$redirect = $this->getFacebookMock(['scopes' => ['publish_pages']])->getRedirect();
87-
$this->assertStringContainsStringIgnoringCase('scope='.urlencode('publish_pages,email,public_profile'), $redirect);
87+
$this->assertStringContainsStringIgnoringCase('scope=' . urlencode('publish_pages,email,public_profile'), $redirect);
8888
});
8989

9090
it('merges default scopes with scopes passed to method', function () {
9191
$redirect = $this->getFacebookMock()->getRedirect(null, ['publish_pages']);
92-
$this->assertStringContainsStringIgnoringCase('scope='.urlencode('publish_pages,email,public_profile'), $redirect);
92+
$this->assertStringContainsStringIgnoringCase('scope=' . urlencode('publish_pages,email,public_profile'), $redirect);
9393
});
9494

9595
it('merges config scopes and scopes passed to the method with default scopes to', function () {
9696
$redirect = $this->getFacebookMock(['scopes' => ['publish_pages']])->getRedirect(null, ['publish_video']);
97-
$this->assertStringContainsStringIgnoringCase('scope='.urlencode('publish_pages,publish_video,email,public_profile'), $redirect);
97+
$this->assertStringContainsStringIgnoringCase('scope=' . urlencode('publish_pages,publish_video,email,public_profile'), $redirect);
9898
});
9999

100100
it('returns a valid redirect login helper instance', function () {
@@ -110,13 +110,13 @@
110110
$redirect = $this->getFacebookMock()->getRedirect();
111111
$this->assertStringContainsStringIgnoringCase('response_type=code', $redirect);
112112
$this->assertStringContainsStringIgnoringCase('client_id=123456789', $redirect);
113-
$this->assertStringContainsStringIgnoringCase('redirect_uri='.urlencode('http://invalid.zzz'), $redirect);
113+
$this->assertStringContainsStringIgnoringCase('redirect_uri=' . urlencode('http://invalid.zzz'), $redirect);
114114
});
115115

116116
it('returns a valid logout url', function () {
117117
$redirect = $this->getFacebookMock()->getLogoutUrl('foo-access-token', 'http://invalid.zzz');
118118
$this->assertStringContainsStringIgnoringCase('access_token=foo-access-token', $redirect);
119-
$this->assertStringContainsStringIgnoringCase('next='.urlencode('http://invalid.zzz'), $redirect);
119+
$this->assertStringContainsStringIgnoringCase('next=' . urlencode('http://invalid.zzz'), $redirect);
120120
$this->assertStringContainsStringIgnoringCase('https://www.facebook.com/logout.php', $redirect);
121121
});
122122

@@ -125,22 +125,22 @@
125125
$this->assertStringContainsStringIgnoringCase('auth_type=rerequest', $redirect);
126126
$this->assertStringContainsStringIgnoringCase('response_type=code', $redirect);
127127
$this->assertStringContainsStringIgnoringCase('client_id=123456789', $redirect);
128-
$this->assertStringContainsStringIgnoringCase('redirect_uri='.urlencode('http://invalid.zzz'), $redirect);
128+
$this->assertStringContainsStringIgnoringCase('redirect_uri=' . urlencode('http://invalid.zzz'), $redirect);
129129
});
130130

131131
it('a valid re authentication url', function () {
132132
$redirect = $this->getFacebookMock()->getReAuthenticationUrl('http://invalid.zzz');
133133
$this->assertStringContainsStringIgnoringCase('auth_type=reauthenticate', $redirect);
134134
$this->assertStringContainsStringIgnoringCase('response_type=code', $redirect);
135135
$this->assertStringContainsStringIgnoringCase('client_id=123456789', $redirect);
136-
$this->assertStringContainsStringIgnoringCase('redirect_uri='.urlencode('http://invalid.zzz'), $redirect);
136+
$this->assertStringContainsStringIgnoringCase('redirect_uri=' . urlencode('http://invalid.zzz'), $redirect);
137137
});
138138

139139
it('gets an access token', function () {
140140
$_GET['code'] = 'foo_code';
141141
$_GET['state'] = 'foo_state';
142142

143-
$persistentDataHandler = new InMemoryPersistentDataHandler;
143+
$persistentDataHandler = new InMemoryPersistentDataHandler();
144144
$persistentDataHandler->set('state', 'foo_state');
145145

146146
/** @var \Facebook\Facebook|\Mockery\MockInterface $base */

tests/Fixtures/FooOAuth2Client.php

+1-1
Original file line numberDiff line numberDiff line change
@@ -8,6 +8,6 @@ class FooOAuth2Client extends OAuth2Client
88
{
99
public function getAccessTokenFromCode($code, $redirectUri = '', $machineId = null)
1010
{
11-
return 'foo_token_from_code|'.$code.'|'.$redirectUri;
11+
return 'foo_token_from_code|' . $code . '|' . $redirectUri;
1212
}
1313
}

0 commit comments

Comments
 (0)