chore: psalm fixes

Signed-off-by: Robin Appelman <robin@icewind.nl>

apps/sharing/tests/Command/CommandTest.php

@@ -33,6 +33,7 @@ use OCP\Sharing\ISharingRegistry;
 use OCP\Sharing\Permission\SharePermission;
 use OCP\Sharing\Property\ShareProperty;
 use OCP\Sharing\Recipient\ShareRecipient;
+use OCP\Sharing\Share;
 use OCP\Sharing\ShareAccessContext;
 use OCP\Sharing\ShareState;
 use OCP\Sharing\Source\ShareSource;
@@ -42,6 +43,9 @@ use Symfony\Component\Console\Output\ConsoleOutput;
 use Symfony\Component\Console\Output\Output;
 use Test\Sharing\AbstractSharingManagerTests;
 
+/**
+ * @psalm-import-type SharingShare from Share
+ */
 #[Group(name: 'DB')]
 final class CommandTest extends AbstractSharingManagerTests {
 	/** @var list<class-string<SharingBase>> */
@@ -363,7 +367,7 @@ final class CommandTest extends AbstractSharingManagerTests {
 	}
 
 	/**
-	 * @return array<string, mixed>
+	 * @return SharingShare
 	 */
 	#[Override]
 	protected function getShare(ShareAccessContext $accessContext, string $id): array {
@@ -380,7 +384,7 @@ final class CommandTest extends AbstractSharingManagerTests {
 	}
 
 	/**
-	 * @return array<string, mixed>
+	 * @return SharingShare[]
 	 */
 	#[Override]
 	protected function getShares(ShareAccessContext $accessContext, ?string $filterSourceTypeClass, ?string $filterSourceTypeValue, ?string $lastShareID, ?int $limit): array {

apps/sharing/tests/Controller/ApiV1ControllerTest.php

@@ -22,6 +22,7 @@ use OCP\Sharing\ISharingRegistry;
 use OCP\Sharing\Permission\SharePermission;
 use OCP\Sharing\Property\ShareProperty;
 use OCP\Sharing\Recipient\ShareRecipient;
+use OCP\Sharing\Share;
 use OCP\Sharing\ShareAccessContext;
 use OCP\Sharing\ShareState;
 use OCP\Sharing\Source\ShareSource;
@@ -29,6 +30,10 @@ use PHPUnit\Framework\Attributes\Group;
 use Test\Sharing\AbstractSharingManagerTests;
 
 // TODO: Use Dispatcher
+
+/**
+ * @psalm-import-type SharingShare from Share
+ */
 #[Group(name: 'DB')]
 final class ApiV1ControllerTest extends AbstractSharingManagerTests {
 	public function testDefaultShareAccessContext(): void {
@@ -146,11 +151,17 @@ final class ApiV1ControllerTest extends AbstractSharingManagerTests {
 		$this->executeRequest($accessContext, fn (ApiV1Controller $controller): DataResponse => $controller->deleteShare($id));
 	}
 
+	/**
+	 * @psalm-suppress MixedReturnTypeCoercion
+	 */
 	#[Override]
 	protected function getShare(ShareAccessContext $accessContext, string $id): array {
 		return $this->executeRequest(new ShareAccessContext($accessContext->currentUser, null, [], $accessContext->overrideChecks), fn (ApiV1Controller $controller): DataResponse => $controller->getShare($id, $accessContext->secret, $accessContext->arguments));
 	}
 
+	/**
+	 * @psalm-suppress MixedReturnTypeCoercion
+	 */
 	#[Override]
 	protected function getShares(ShareAccessContext $accessContext, ?string $filterSourceTypeClass, ?string $filterSourceTypeValue, ?string $lastShareID, ?int $limit): array {
 		return $this->executeRequest($accessContext, function (ApiV1Controller $controller) use ($filterSourceTypeClass, $filterSourceTypeValue, $lastShareID, $limit): DataResponse {

tests/lib/Sharing/AbstractSharingManagerTests.php

@@ -35,6 +35,9 @@ use Test\TestCase;
 /**
  * @psalm-import-type SharingShare from Share
  * @psalm-import-type SharingRecipient from Share
+ *
+ * @psalm-suppress RedundantConditionGivenDocblockType
+ * @psalm-suppress PossiblyUndefinedArrayOffset
  */
 abstract class AbstractSharingManagerTests extends TestCase {
 	abstract protected function searchRecipients(ShareAccessContext $accessContext, ?array $filterRecipientTypeClasses, string $query, int $limit, int $offset, ?string $id = null): array;
@@ -61,8 +64,14 @@ abstract class AbstractSharingManagerTests extends TestCase {
 
 	abstract protected function deleteShare(ShareAccessContext $accessContext, string $id): void;
 
+	/**
+	 * @return SharingShare
+	 */
 	abstract protected function getShare(ShareAccessContext $accessContext, string $id): array;
 
+	/**
+	 * @return SharingShare[]
+	 */
 	abstract protected function getShares(ShareAccessContext $accessContext, ?string $filterSourceTypeClass, ?string $filterSourceTypeValue, ?string $lastShareID, ?int $limit): array;
 
 	protected IDBConnection $dbConnection;

tests/lib/Sharing/SharingManagerTest.php

@@ -207,9 +207,6 @@ final class SharingManagerTest extends AbstractSharingManagerTests {
 		}
 	}
 
-	/**
-	 * @return mixed[][]
-	 */
 	#[\Override]
 	protected function getShares(ShareAccessContext $accessContext, ?string $filterSourceTypeClass, ?string $filterSourceTypeValue, ?string $lastShareID, ?int $limit): array {
 		try {