check for the UndefinedInterfaceMethod psalm

Signed-off-by: nabim777 <[email protected]>
nextcloud · Sep 23, 2024 · 46bdc06 · 46bdc06
1 parent 034a2c4
commit 46bdc06
Show file tree

Hide file tree

Showing 5 changed files with 10 additions and 9 deletions.
diff --git a/psalm.xml b/psalm.xml
@@ -15,8 +15,7 @@
 		<directory name="tests" />
 		<ignoreFiles>
 			<directory name="vendor" />
-			<directory name="../../**" />
-			<directory name="../**" />
+			<directory name="../../lib" />
 		</ignoreFiles>
 	</projectFiles>
 	<stubs>

diff --git a/tests/acceptance/features/bootstrap/FeatureContext.php b/tests/acceptance/features/bootstrap/FeatureContext.php
@@ -719,7 +719,10 @@ public function getIdOfFileOrFolder(string $user, string $path): int {
 			'oc',
 			'http://owncloud.org/ns'
 		);
-		return (int)(string)$responseXmlObject->xpath('//oc:fileid')[0];
+		$fileId = $responseXmlObject->xpath('//oc:fileid')[0];
+		Assert::assertNotNull($fileId, __METHOD__ . " file $path user $user not found (the file may not exist)");
+
+		return (int)(string) $fileId;
 	}
 
 	public function fileOrFolderExists(string $user, string $path): bool {

diff --git a/tests/acceptance/features/bootstrap/FilesVersionsContext.php b/tests/acceptance/features/bootstrap/FilesVersionsContext.php
@@ -29,7 +29,6 @@ public function theVersionFolderOfFileShouldContainElements(
 		int $count
 	):void {
 		$fileId = $this->featureContext->getIdOfFileOrFolder($user, $path);
-		Assert::assertNotNull($fileId, __METHOD__ . " file $path user $user not found (the file may not exist)");
 		$this->theVersionFolderOfFileIdShouldContainElements($user, $fileId, $count);
 	}
 

diff --git a/tests/lib/Controller/OpenProjectAPIControllerTest.php b/tests/lib/Controller/OpenProjectAPIControllerTest.php
@@ -31,10 +31,10 @@
 use Psr\Log\LoggerInterface;
 
 class OpenProjectAPIControllerTest extends TestCase {
-	/** @var IConfig $configMock */
+	/** @var IConfig */
 	private $configMock;
 
-	/** @var IRequest $requestMock */
+	/** @var IRequest */
 	private $requestMock;
 
 	/**
@@ -79,9 +79,10 @@ public function setUpMocks(): void {
 
 	/**
 	 * @param string $token
+	 * @psalm-suppress UndefinedInterfaceMethod
 	 * @return void
 	 */
-	public function getUserValueMock($token = '123') {
+	public function getUserValueMock(string $token = '123'): void {
 		$this->configMock
 			->method('getUserValue')
 			->withConsecutive(
@@ -93,7 +94,7 @@ public function getUserValueMock($token = '123') {
 	/**
 	 * @return void
 	 */
-	public function testGetNotifications() {
+	public function testGetNotifications(): void {
 		$this->getUserValueMock();
 		$service = $this->getMockBuilder(OpenProjectAPIService::class)
 			->disableOriginalConstructor()

diff --git a/tests/lib/Service/OpenProjectAPIServiceTest.php b/tests/lib/Service/OpenProjectAPIServiceTest.php
@@ -635,7 +635,6 @@ private function getOpenProjectAPIService(
 	) {
 		$certificateManager = $this->getMockBuilder('\OCP\ICertificateManager')->getMock();
 		$certificateManager->method('getAbsoluteBundlePath')->willReturn('/');
-		$ocClient = null;
 		$client = new GuzzleClient();
 		$clientConfigMock = $this->getMockBuilder(IConfig::class)->getMock();
 		$clientConfigMock