Skip to content

Commit 2ba870b

Browse files
ghalsetvdijen
authored andcommitted
Fix lint issues (sorry)
1 parent 274ec53 commit 2ba870b

2 files changed

Lines changed: 31 additions & 8 deletions

File tree

src/Auth/Process/Fticks.php

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,6 @@
1717
use function array_key_exists;
1818
use function array_keys;
1919
use function array_map;
20-
use function boolval;
2120
use function constant;
2221
use function defined;
2322
use function gethostbyname;
@@ -78,7 +77,7 @@ class Fticks extends Auth\ProcessingFilter
7877
/** @var array F-ticks attributes to exclude */
7978
private array $exclude = [];
8079

81-
/** @var bool Enable legacy handing of PN (for backwards compatibility) */
80+
/** @var string Enable legacy handing of PN (for backwards compatibility) */
8281
private string $pnHashIsTargeted = 'none';
8382

8483

tests/src/Auth/Process/FticksTest.php

Lines changed: 30 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -68,7 +68,6 @@ protected function setUp(): void
6868
Configuration::loadFromArray([
6969
'secretsalt' => 'secretsalt',
7070
], '[ARRAY]', 'simplesaml');
71-
7271
}
7372

7473

@@ -161,7 +160,12 @@ public function testSPwithUserIdDifferentProviders(): void
161160
*/
162161
public function testSPwithUserIdLegacyBehaviour(): void
163162
{
164-
$config = ['federation' => 'ACME', 'logdest' => 'stdout', 'identifyingAttribute' => 'eduPersonPrincipalName', 'pnHashIsTargeted' => 'both',];
163+
$config = [
164+
'federation' => 'ACME',
165+
'logdest' => 'stdout',
166+
'identifyingAttribute' => 'eduPersonPrincipalName',
167+
'pnHashIsTargeted' => 'both',
168+
];
165169
$request = array_merge(self::$minRequest, self::$spRequest, [
166170
'Attributes' => [
167171
'eduPersonPrincipalName' => [ 'user2@example.net' ],
@@ -186,7 +190,12 @@ public function testSPwithUserIdLegacyBehaviour(): void
186190
*/
187191
public function testSPwithUserIdSourceTargeted(): void
188192
{
189-
$config = ['federation' => 'ACME', 'logdest' => 'stdout', 'identifyingAttribute' => 'eduPersonPrincipalName', 'pnHashIsTargeted' => 'source',];
193+
$config = [
194+
'federation' => 'ACME',
195+
'logdest' => 'stdout',
196+
'identifyingAttribute' => 'eduPersonPrincipalName',
197+
'pnHashIsTargeted' => 'source',
198+
];
190199
$request = array_merge(self::$minRequest, self::$spRequest, [
191200
'Attributes' => [
192201
'eduPersonPrincipalName' => [ 'user2@example.net' ],
@@ -211,7 +220,12 @@ public function testSPwithUserIdSourceTargeted(): void
211220
*/
212221
public function testSPwithUserIdSourceTargetedDifferentDest(): void
213222
{
214-
$config = ['federation' => 'ACME', 'logdest' => 'stdout', 'identifyingAttribute' => 'eduPersonPrincipalName', 'pnHashIsTargeted' => 'source',];
223+
$config = [
224+
'federation' => 'ACME',
225+
'logdest' => 'stdout',
226+
'identifyingAttribute' => 'eduPersonPrincipalName',
227+
'pnHashIsTargeted' => 'source',
228+
];
215229
$request = array_merge(self::$minRequest, self::$spRequest, [
216230
'Attributes' => [
217231
'eduPersonPrincipalName' => [ 'user2@example.net' ],
@@ -237,7 +251,12 @@ public function testSPwithUserIdSourceTargetedDifferentDest(): void
237251
*/
238252
public function testSPwithUserIdDestinationTargeted(): void
239253
{
240-
$config = ['federation' => 'ACME', 'logdest' => 'stdout', 'identifyingAttribute' => 'eduPersonPrincipalName', 'pnHashIsTargeted' => 'destination',];
254+
$config = [
255+
'federation' => 'ACME',
256+
'logdest' => 'stdout',
257+
'identifyingAttribute' => 'eduPersonPrincipalName',
258+
'pnHashIsTargeted' => 'destination',
259+
];
241260
$request = array_merge(self::$minRequest, self::$spRequest, [
242261
'Attributes' => [
243262
'eduPersonPrincipalName' => [ 'user2@example.net' ],
@@ -263,7 +282,12 @@ public function testSPwithUserIdDestinationTargeted(): void
263282
*/
264283
public function testSPwithUserIdDestinationTargetedDifferentSource(): void
265284
{
266-
$config = ['federation' => 'ACME', 'logdest' => 'stdout', 'identifyingAttribute' => 'eduPersonPrincipalName', 'pnHashIsTargeted' => 'destination',];
285+
$config = [
286+
'federation' => 'ACME',
287+
'logdest' => 'stdout',
288+
'identifyingAttribute' => 'eduPersonPrincipalName',
289+
'pnHashIsTargeted' => 'destination',
290+
];
267291
$request = array_merge(self::$minRequest, self::$spRequest, [
268292
'Attributes' => [
269293
'eduPersonPrincipalName' => [ 'user2@example.net' ],

0 commit comments

Comments
 (0)