PHPStan fixes

inc/Configuration.php

@@ -33,7 +33,7 @@ class Configuration {
 	 *
 	 * @phpstan-var ProjectConfig
 	 */
-	protected $config = [];
+	protected array $config = [];
 
 	/**
 	 * Class constructor.
@@ -84,7 +84,7 @@ public function get_config(): array {
 	 * @phpstan-param T $key
 	 * @phpstan-return ProjectConfig[T] | null
 	 */
-	public function get_config_value( string $key ): string|array|null {
+	public function get_config_value( string $key ): mixed {
 		if ( isset( $this->config[ $key ] ) ) {
 			return $this->config[ $key ];
 		}

tests/phpunit/tests/Export.php

@@ -12,10 +12,7 @@
  * Test cases for \Required\Traduttore\Export.
  */
 class Export extends TestCase {
-	/**
-	 * @var \GP_Translation_Set
-	 */
-	protected $translation_set;
+	protected \GP_Translation_Set $translation_set;
 
 	public function setUp(): void {
 		parent::setUp();
@@ -149,7 +146,7 @@ public function test_creates_multiple_json_files(): void {
 	 * Modify the mapping of sources to translation entries.
 	 *
 	 * @param array<string, \Translation_Entry[]> $mapping The mapping of sources to translation entries.
-	 * @return array<string, \Required\Traduttore\Tests\Translation_Entry[]> The maybe modified mapping.
+	 * @return array<string, \Translation_Entry[]> The maybe modified mapping.
 	 */
 	public function filter_map_entries_to_source( array $mapping ): array {
 		$mapping['build.js'] = array_merge( $mapping['my-super-script.js'], $mapping['my-other-script.js'] );

tests/phpunit/tests/Loader/Git.php

@@ -16,10 +16,7 @@
  * @todo Mock shell execution
  */
 class Git extends TestCase {
-	/**
-	 * @var \Required\Traduttore\Tests\Loader\Project
-	 */
-	protected $project;
+	protected Project $project;
 
 	public function setUp(): void {
 		parent::setUp();

tests/phpunit/tests/Loader/Mercurial.php

@@ -16,10 +16,7 @@
  * @todo Mock shell execution
  */
 class Mercurial extends TestCase {
-	/**
-	 * @var \Required\Traduttore\Tests\Loader\Project
-	 */
-	protected $project;
+	protected Project $project;
 
 	public function setUp(): void {
 		parent::setUp();

tests/phpunit/tests/Loader/Subversion.php

@@ -16,10 +16,7 @@
  * @todo Mock shell execution
  */
 class Subversion extends TestCase {
-	/**
-	 * @var \Required\Traduttore\Tests\Loader\Project
-	 */
-	protected $project;
+	protected Project $project;
 
 	public function setUp(): void {
 		parent::setUp();

tests/phpunit/tests/Project.php

@@ -7,20 +7,16 @@
 
 use DateTime;
 use DateTimeZone;
+use GP_Project;
 use Required\Traduttore\Project as TraduttoreProject;
 
 /**
  * Test cases for \Required\Traduttore\Project.
  */
 class Project extends TestCase {
-	/**
-	 * @var \Required\Traduttore\Tests\GP_Project
-	 */
-	protected $gp_project;
-	/**
-	 * @var \Required\Traduttore\Tests\TraduttoreProject
-	 */
-	protected $project;
+	protected GP_Project $gp_project;
+
+	protected TraduttoreProject $project;
 
 	public function setUp(): void {
 		parent::setUp();

tests/phpunit/tests/Runner.php

@@ -14,20 +14,11 @@
  * Test cases for \Required\Traduttore\Runner.
  */
 class Runner extends TestCase {
-	/**
-	 * @var \Required\Traduttore\Tests\Project
-	 */
-	protected $project;
-
-	/**
-	 * @var \Required\Traduttore\Tests\R
-	 */
-	protected $runner;
-
-	/**
-	 * @var \Required\Traduttore\Loader
-	 */
-	protected $loader;
+	protected Project $project;
+
+	protected R $runner;
+
+	protected Loader $loader;
 
 	public function setUp(): void {
 		parent::setUp();

tests/phpunit/tests/TestCase.php

@@ -7,6 +7,7 @@
 
 use GP_UnitTest_Factory;
 use GP_UnitTestCase;
+use WP_Error;
 use WP_REST_Response;
 
 /**

tests/phpunit/tests/Updater.php

@@ -14,15 +14,9 @@
  * Test cases for \Required\Traduttore\Updater.
  */
 class Updater extends TestCase {
-	/**
-	 * @var \Required\Traduttore\Tests\Project
-	 */
-	protected $project;
-
-	/**
-	 * @var \Required\Traduttore\Tests\U
-	 */
-	protected $updater;
+	protected Project $project;
+
+	protected U $updater;
 
 	public function setUp(): void {
 		parent::setUp();