1 files changed, 15 insertions, 11 deletions

diff --git a/tests/lib/Share20/ShareTest.php b/tests/lib/Share20/ShareTest.php
index 6769683aafe..f15fbb860db 100644
--- a/tests/lib/Share20/ShareTest.php
+++ b/tests/lib/Share20/ShareTest.php
@@ -1,4 +1,5 @@
 <?php
+
 /**
  * SPDX-FileCopyrightText: 2016-2024 Nextcloud GmbH and Nextcloud contributors
  * SPDX-FileCopyrightText: 2016 ownCloud, Inc.
@@ -7,8 +8,11 @@
 
 namespace Test\Share20;
 
+use OC\Share20\Share;
 use OCP\Files\IRootFolder;
 use OCP\IUserManager;
+use OCP\Share\Exceptions\IllegalIDChangeException;
+use OCP\Share\IShare;
 use PHPUnit\Framework\MockObject\MockObject;
 
 /**
@@ -21,37 +25,37 @@ class ShareTest extends \Test\TestCase {
 	protected $rootFolder;
 	/** @var IUserManager|MockObject */
 	protected $userManager;
-	/** @var \OCP\Share\IShare */
+	/** @var IShare */
 	protected $share;
 
 	protected function setUp(): void {
 		$this->rootFolder = $this->createMock(IRootFolder::class);
 		$this->userManager = $this->createMock(IUserManager::class);
-		$this->share = new \OC\Share20\Share($this->rootFolder, $this->userManager);
+		$this->share = new Share($this->rootFolder, $this->userManager);
 	}
 
 
-	public function testSetIdInvalid() {
+	public function testSetIdInvalid(): void {
 		$this->expectException(\InvalidArgumentException::class);
 		$this->expectExceptionMessage('String expected.');
 
 		$this->share->setId(1.2);
 	}
 
-	public function testSetIdInt() {
+	public function testSetIdInt(): void {
 		$this->share->setId(42);
 		$this->assertEquals('42', $this->share->getId());
 	}
 
 
-	public function testSetIdString() {
+	public function testSetIdString(): void {
 		$this->share->setId('foo');
 		$this->assertEquals('foo', $this->share->getId());
 	}
 
 
-	public function testSetIdOnce() {
-		$this->expectException(\OCP\Share\Exceptions\IllegalIDChangeException::class);
+	public function testSetIdOnce(): void {
+		$this->expectException(IllegalIDChangeException::class);
 		$this->expectExceptionMessage('Not allowed to assign a new internal id to a share');
 
 		$this->share->setId('foo');
@@ -59,7 +63,7 @@ class ShareTest extends \Test\TestCase {
 	}
 
 
-	public function testSetProviderIdInt() {
+	public function testSetProviderIdInt(): void {
 		$this->expectException(\InvalidArgumentException::class);
 		$this->expectExceptionMessage('String expected.');
 
@@ -67,15 +71,15 @@ class ShareTest extends \Test\TestCase {
 	}
 
 
-	public function testSetProviderIdString() {
+	public function testSetProviderIdString(): void {
 		$this->share->setProviderId('foo');
 		$this->share->setId('bar');
 		$this->assertEquals('foo:bar', $this->share->getFullId());
 	}
 
 
-	public function testSetProviderIdOnce() {
-		$this->expectException(\OCP\Share\Exceptions\IllegalIDChangeException::class);
+	public function testSetProviderIdOnce(): void {
+		$this->expectException(IllegalIDChangeException::class);
 		$this->expectExceptionMessage('Not allowed to assign a new provider id to a share');
 
 		$this->share->setProviderId('foo');