Skip to content

Commit 1a1b239

Browse files
committed
Make tests compatible with newer phpunit versions
1 parent b264e08 commit 1a1b239

13 files changed

Lines changed: 72 additions & 24 deletions

Tests/DeviceDetectorTest.php

Lines changed: 12 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -247,7 +247,7 @@ public function testParse(array $fixtureData): void
247247
$this->assertEquals($fixtureData, $uaInfo, $errorMessage);
248248
}
249249

250-
public function getFixtures(): array
250+
public static function getFixtures(): array
251251
{
252252
$fixtures = [];
253253
$fixtureFiles = \glob(\realpath(__DIR__) . '/fixtures/*.yml');
@@ -301,7 +301,7 @@ public function testParseClient(array $fixtureData): void
301301
$this->assertEquals($fixtureData['client'], $uaInfo['client'], $messageError);
302302
}
303303

304-
public function getFixturesClient(): array
304+
public static function getFixturesClient(): array
305305
{
306306
$fixtures = [];
307307
$fixtureFiles = \glob(\realpath(__DIR__) . '/Parser/Client/fixtures/*.yml');
@@ -341,7 +341,7 @@ public function testParseDevice(array $fixtureData): void
341341
$this->assertEquals($fixtureData['device'], $uaInfo['device']);
342342
}
343343

344-
public function getFixturesDevice(): array
344+
public static function getFixturesDevice(): array
345345
{
346346
$fixtures = [];
347347
$fixtureFiles = \glob(\realpath(__DIR__) . '/Parser/Device/fixtures/*.yml');
@@ -403,7 +403,7 @@ public function testVersionTruncation(string $useragent, int $truncationType, st
403403
AbstractParser::setVersionTruncation(AbstractParser::VERSION_TRUNCATION_NONE);
404404
}
405405

406-
public function getVersionTruncationFixtures(): array
406+
public static function getVersionTruncationFixtures(): array
407407
{
408408
return [
409409
['Mozilla/5.0 (Linux; Android 4.2.2; ARCHOS 101 PLATINUM Build/JDQ39) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/34.0.1847.114 Safari/537.36', AbstractParser::VERSION_TRUNCATION_NONE, '4.2.2', '34.0.1847.114'],
@@ -528,7 +528,7 @@ public function testParseBots(array $fixtureData): void
528528
);
529529
}
530530

531-
public function getBotFixtures(): array
531+
public static function getBotFixtures(): array
532532
{
533533
$fixturesPath = \realpath(__DIR__ . '/fixtures/bots.yml');
534534
$fixtures = \Spyc::YAMLLoad($fixturesPath);
@@ -628,11 +628,15 @@ public function testGetClient(): void
628628
$this->assertEquals($expected, $dd->getClient());
629629
}
630630

631-
public function getTypeMethodFixtures(): array
631+
public static function getTypeMethodFixtures(): array
632632
{
633-
$fixturePath = \realpath(__DIR__ . '/Parser/fixtures/type-methods.yml');
633+
$fixtureData = \Spyc::YAMLLoad(\realpath(__DIR__ . '/Parser/fixtures/type-methods.yml'));
634634

635-
return \Spyc::YAMLLoad($fixturePath);
635+
$fixtureData = \array_map(static function (array $item): array {
636+
return ['ua' => $item['user_agent'], 'checkTypes' => $item['check']];
637+
}, $fixtureData);
638+
639+
return $fixtureData;
636640
}
637641

638642
/**

Tests/Parser/Client/BrowserTest.php

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -49,10 +49,14 @@ public function testParse(string $useragent, array $client, ?array $headers = nu
4949
self::$browsersTested[] = $client['name'];
5050
}
5151

52-
public function getFixtures(): array
52+
public static function getFixtures(): array
5353
{
5454
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/browser.yml');
5555

56+
$fixtureData = \array_map(static function (array $item): array {
57+
return ['useragent' => $item['user_agent'], 'client' => $item['client'], 'headers' => $item['headers'] ?? null];
58+
}, $fixtureData);
59+
5660
return $fixtureData;
5761
}
5862

@@ -108,7 +112,7 @@ public function testBrowserFamiliesNoDuplicates(): void
108112
/**
109113
* @return array
110114
*/
111-
public function getFixturesBrowserHints(): array
115+
public static function getFixturesBrowserHints(): array
112116
{
113117
$method = new \ReflectionMethod(BrowserHints::class, 'getRegexes');
114118
$method->setAccessible(true);

Tests/Parser/Client/FeedReaderTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,14 @@ public function testParse(string $useragent, array $client): void
2929
$this->assertEquals($client, $feedReaderParser->parse(), "UserAgent: {$useragent}");
3030
}
3131

32-
public function getFixtures(): array
32+
public static function getFixtures(): array
3333
{
3434
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/feed_reader.yml');
3535

36+
$fixtureData = \array_map(static function (array $item): array {
37+
return ['useragent' => $item['user_agent'], 'client' => $item['client']];
38+
}, $fixtureData);
39+
3640
return $fixtureData;
3741
}
3842

Tests/Parser/Client/LibraryTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,14 @@ public function testParse(string $useragent, array $client): void
2929
$this->assertEquals($client, $libraryParser->parse());
3030
}
3131

32-
public function getFixtures(): array
32+
public static function getFixtures(): array
3333
{
3434
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/library.yml');
3535

36+
$fixtureData = \array_map(static function (array $item): array {
37+
return ['useragent' => $item['user_agent'], 'client' => $item['client']];
38+
}, $fixtureData);
39+
3640
return $fixtureData;
3741
}
3842

Tests/Parser/Client/MediaPlayerTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,14 @@ public function testParse(string $useragent, array $client): void
2929
$this->assertEquals($client, $mediaPlayerParser->parse());
3030
}
3131

32-
public function getFixtures(): array
32+
public static function getFixtures(): array
3333
{
3434
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/mediaplayer.yml');
3535

36+
$fixtureData = \array_map(static function (array $item): array {
37+
return ['useragent' => $item['user_agent'], 'client' => $item['client']];
38+
}, $fixtureData);
39+
3640
return $fixtureData;
3741
}
3842

Tests/Parser/Client/MobileAppTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,14 @@ public function testParse(string $useragent, array $client): void
2929
$this->assertEquals($client, $mobileAppParser->parse());
3030
}
3131

32-
public function getFixtures(): array
32+
public static function getFixtures(): array
3333
{
3434
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/mobile_app.yml');
3535

36+
$fixtureData = \array_map(static function (array $item): array {
37+
return ['useragent' => $item['user_agent'], 'client' => $item['client']];
38+
}, $fixtureData);
39+
3640
return $fixtureData;
3741
}
3842

Tests/Parser/Client/PIMTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -29,10 +29,14 @@ public function testParse(string $useragent, array $client): void
2929
$this->assertEquals($client, $PIMParser->parse());
3030
}
3131

32-
public function getFixtures(): array
32+
public static function getFixtures(): array
3333
{
3434
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/pim.yml');
3535

36+
$fixtureData = \array_map(static function (array $item): array {
37+
return ['useragent' => $item['user_agent'], 'client' => $item['client']];
38+
}, $fixtureData);
39+
3640
return $fixtureData;
3741
}
3842

Tests/Parser/Device/CameraTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,14 @@ public function testParse(string $useragent, array $device): void
3131
$this->assertEquals($device['model'], $consoleParser->getModel());
3232
}
3333

34-
public function getFixtures(): array
34+
public static function getFixtures(): array
3535
{
3636
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/camera.yml');
3737

38+
$fixtureData = \array_map(static function (array $item): array {
39+
return ['useragent' => $item['user_agent'], 'device' => $item['device']];
40+
}, $fixtureData);
41+
3842
return $fixtureData;
3943
}
4044
}

Tests/Parser/Device/CarBrowserTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,14 @@ public function testParse(string $useragent, array $device): void
3131
$this->assertEquals($device['model'], $consoleParser->getModel());
3232
}
3333

34-
public function getFixtures(): array
34+
public static function getFixtures(): array
3535
{
3636
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/car_browser.yml');
3737

38+
$fixtureData = \array_map(static function (array $item): array {
39+
return ['useragent' => $item['user_agent'], 'device' => $item['device']];
40+
}, $fixtureData);
41+
3842
return $fixtureData;
3943
}
4044
}

Tests/Parser/Device/ConsoleTest.php

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -31,10 +31,14 @@ public function testParse(string $useragent, array $device): void
3131
$this->assertEquals($device['model'], $consoleParser->getModel());
3232
}
3333

34-
public function getFixtures(): array
34+
public static function getFixtures(): array
3535
{
3636
$fixtureData = Spyc::YAMLLoad(\realpath(__DIR__) . '/fixtures/console.yml');
3737

38+
$fixtureData = \array_map(static function (array $item): array {
39+
return ['useragent' => $item['user_agent'], 'device' => $item['device']];
40+
}, $fixtureData);
41+
3842
return $fixtureData;
3943
}
4044
}

0 commit comments

Comments
 (0)