aboutsummaryrefslogtreecommitdiff
path: root/tests
diff options
context:
space:
mode:
authorGravatar Alexis Degrugillier <aledeg@users.noreply.github.com> 2025-09-05 09:56:46 -0400
committerGravatar GitHub <noreply@github.com> 2025-09-05 15:56:46 +0200
commit23ba48c71f0d41bbe012d668349f6516dad527b4 (patch)
treeb5680e834c2b87becdded07e0555227ade9d7629 /tests
parent3d85eb19380021031e9d1193919ba516bbc99573 (diff)
Change how files are included (#7916)
1. `include`, `include_once`, `require` and `require_once` are expressions not functions, parentheses are not necessary. 2. to move up the directory tree, it's better to use the `dirname` function instead of relying on `/..`.
Diffstat (limited to 'tests')
-rw-r--r--tests/app/Models/SearchTest.php2
-rw-r--r--tests/bootstrap.php4
-rw-r--r--tests/cli/CliOptionsParserTest.php4
-rwxr-xr-xtests/cli/cli-parser-test.php4
-rw-r--r--tests/cli/i18n/I18nCompletionValidatorTest.php4
-rw-r--r--tests/cli/i18n/I18nDataTest.php4
-rw-r--r--tests/cli/i18n/I18nFileTest.php2
-rw-r--r--tests/cli/i18n/I18nUsageValidatorTest.php4
-rw-r--r--tests/cli/i18n/I18nValueTest.php2
9 files changed, 15 insertions, 15 deletions
diff --git a/tests/app/Models/SearchTest.php b/tests/app/Models/SearchTest.php
index 8b4c9d2b5..e6efdbe55 100644
--- a/tests/app/Models/SearchTest.php
+++ b/tests/app/Models/SearchTest.php
@@ -3,7 +3,7 @@ declare(strict_types=1);
use PHPUnit\Framework\Attributes\DataProvider;
-require_once(LIB_PATH . '/lib_date.php');
+require_once LIB_PATH . '/lib_date.php';
class SearchTest extends PHPUnit\Framework\TestCase {
diff --git a/tests/bootstrap.php b/tests/bootstrap.php
index f988855e6..6aee8b96f 100644
--- a/tests/bootstrap.php
+++ b/tests/bootstrap.php
@@ -5,5 +5,5 @@ ini_set('display_errors', '1');
const COPY_LOG_TO_SYSLOG = false;
-require(__DIR__ . '/../constants.php');
-require(LIB_PATH . '/lib_rss.php'); //Includes class autoloader
+require dirname(__DIR__) . '/constants.php';
+require LIB_PATH . '/lib_rss.php'; //Includes class autoloader
diff --git a/tests/cli/CliOptionsParserTest.php b/tests/cli/CliOptionsParserTest.php
index 6a67fa761..2fe0b7924 100644
--- a/tests/cli/CliOptionsParserTest.php
+++ b/tests/cli/CliOptionsParserTest.php
@@ -2,8 +2,8 @@
declare(strict_types=1);
use PHPUnit\Framework\TestCase;
-require_once __DIR__ . '/../../cli/CliOption.php';
-require_once __DIR__ . '/../../cli/CliOptionsParser.php';
+require_once dirname(__DIR__, 2) . '/cli/CliOption.php';
+require_once dirname(__DIR__, 2) . '/cli/CliOptionsParser.php';
final class CliOptionsOptionalTest extends CliOptionsParser {
public string $string = '';
diff --git a/tests/cli/cli-parser-test.php b/tests/cli/cli-parser-test.php
index 6806d29ce..a6b157474 100755
--- a/tests/cli/cli-parser-test.php
+++ b/tests/cli/cli-parser-test.php
@@ -2,8 +2,8 @@
<?php
declare(strict_types=1);
-require(__DIR__ . '/../../vendor/autoload.php');
-require(__DIR__ . '/CliOptionsParserTest.php');
+require dirname(__DIR__, 2) . '/vendor/autoload.php';
+require __DIR__ . '/CliOptionsParserTest.php';
$optionsClass = getenv('CLI_PARSER_TEST_OPTIONS_CLASS');
if (!is_string($optionsClass) || !class_exists($optionsClass)) {
diff --git a/tests/cli/i18n/I18nCompletionValidatorTest.php b/tests/cli/i18n/I18nCompletionValidatorTest.php
index f66d9e04f..4d62662aa 100644
--- a/tests/cli/i18n/I18nCompletionValidatorTest.php
+++ b/tests/cli/i18n/I18nCompletionValidatorTest.php
@@ -1,7 +1,7 @@
<?php
declare(strict_types=1);
-require_once __DIR__ . '/../../../cli/i18n/I18nCompletionValidator.php';
-require_once __DIR__ . '/../../../cli/i18n/I18nValue.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nCompletionValidator.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nValue.php';
class I18nCompletionValidatorTest extends PHPUnit\Framework\TestCase {
/** @var I18nValue&PHPUnit\Framework\MockObject\MockObject */
diff --git a/tests/cli/i18n/I18nDataTest.php b/tests/cli/i18n/I18nDataTest.php
index afdaacf1e..1f9e257fc 100644
--- a/tests/cli/i18n/I18nDataTest.php
+++ b/tests/cli/i18n/I18nDataTest.php
@@ -1,7 +1,7 @@
<?php
declare(strict_types=1);
-require_once __DIR__ . '/../../../cli/i18n/I18nData.php';
-require_once __DIR__ . '/../../../cli/i18n/I18nValue.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nData.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nValue.php';
class I18nDataTest extends PHPUnit\Framework\TestCase {
/** @var array<string,array<string,array<string,I18nValue>>> */
diff --git a/tests/cli/i18n/I18nFileTest.php b/tests/cli/i18n/I18nFileTest.php
index 34abdb9a3..b1aca49ea 100644
--- a/tests/cli/i18n/I18nFileTest.php
+++ b/tests/cli/i18n/I18nFileTest.php
@@ -1,6 +1,6 @@
<?php
declare(strict_types=1);
-require_once __DIR__ . '/../../../cli/i18n/I18nFile.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nFile.php';
class I18nFileTest extends PHPUnit\Framework\TestCase {
public function test(): void {
diff --git a/tests/cli/i18n/I18nUsageValidatorTest.php b/tests/cli/i18n/I18nUsageValidatorTest.php
index 5a2268141..0429d1277 100644
--- a/tests/cli/i18n/I18nUsageValidatorTest.php
+++ b/tests/cli/i18n/I18nUsageValidatorTest.php
@@ -1,7 +1,7 @@
<?php
declare(strict_types=1);
-require_once __DIR__ . '/../../../cli/i18n/I18nValue.php';
-require_once __DIR__ . '/../../../cli/i18n/I18nUsageValidator.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nValue.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nUsageValidator.php';
class I18nUsageValidatorTest extends PHPUnit\Framework\TestCase {
diff --git a/tests/cli/i18n/I18nValueTest.php b/tests/cli/i18n/I18nValueTest.php
index 85ff00fee..fb1fca804 100644
--- a/tests/cli/i18n/I18nValueTest.php
+++ b/tests/cli/i18n/I18nValueTest.php
@@ -1,6 +1,6 @@
<?php
declare(strict_types=1);
-require_once __DIR__ . '/../../../cli/i18n/I18nValue.php';
+require_once dirname(__DIR__, 3) . '/cli/i18n/I18nValue.php';
class I18nValueTest extends PHPUnit\Framework\TestCase {
public static function testConstructorWithoutState(): void {