Merge pull request owncloud#40451 from owncloud/add-step-implementation

[tests-only] Refactored step implementation for adding public link to server

tests/acceptance/features/bootstrap/WebUISharingContext.php

@@ -1283,18 +1283,17 @@ public function thePublicAddsThePublicLinkToUsingTheWebUI(
 		if (!$this->publicLinkFilesPage->isOpen()) {
 			throw new Exception('Not on public link page!');
 		}
-		if ($server === '%remote_server%') {
+		$servers = ['%local_server%', '%remote_server%'];
+		if (\in_array($server, $servers)) {
 			$server = $this->featureContext->substituteInLineCodes($server);
 			$this->publicLinkFilesPage->addToServer($server);
-		} elseif ($server === '%local_server%') {
-			$this->publicLinkFilesPage->saveToSameServer();
 		} else {
 			throw new Exception(
 				"Invalid server provided '" . $server . "'. " .
 				"Either should be '%remote_server%' or '%local_server%'"
 			);
 		}
-		// addToServer and saveToSameServer takes us from the public link page to the login page
+		// addToServer takes us from the public link page to the login page
 		// of the remote and local server respectively, waiting for us to login.
 		$actualUsername = $this->featureContext->getActualUsername($username);
 		$password = $this->featureContext->getUserPassword($actualUsername);

tests/acceptance/features/lib/PublicLinkFilesPage.php

@@ -576,22 +576,4 @@ public function getCompletelyUploadedElements(): array {
 		}
 		return $uploadedElements;
 	}
-
-	/**
-	 * adding public link share to same server instance
-	 *
-	 * @return void
-	 * @throws ElementNotFoundException
-	 */
-	public function saveToSameServer(): void {
-		$saveToElement = $this->findById($this->saveToOcButtonId);
-
-		$this->assertElementNotNull(
-			$saveToElement,
-			__METHOD__ .
-			" id $this->saveToOcButtonId could not find 'Add To' button"
-		);
-
-		$saveToElement->click();
-	}
 }