Adjust tests to open ShareActionsDropDown to be able to see 'notify by email' button

phil-davis · phil-davis · commit 16cb9fb7dcbb · 2020-01-29T10:04:14.000+05:45
diff --git a/tests/acceptance/features/bootstrap/WebUISharingContext.php b/tests/acceptance/features/bootstrap/WebUISharingContext.php
@@ -1020,15 +1020,24 @@ public function removesPublicLinkAtCertainPosition($number, $entryName) {
 	}
 
 	/**
-	 * @When the user sends the share notification by email using the webUI
-	 *
 	 * @return void
 	 */
-	public function theUserSendsTheShareNotificationByEmailUsingTheWebui() {
+	public function sendShareNotificationByEmailUsingTheWebui() {
 		Assert::assertNotNull(
 			$this->sharingDialog, "Sharing Dialog is not open"
 		);
+		$this->sharingDialog->openShareActionsDropDown();
 		$this->sharingDialog->sendShareNotificationByEmail($this->getSession());
+		$this->sharingDialog->closeShareActionsDropDown();
+	}
+
+	/**
+	 * @When the user sends the share notification by email using the webUI
+	 *
+	 * @return void
+	 */
+	public function theUserSendsTheShareNotificationByEmailUsingTheWebui() {
+		$this->sendShareNotificationByEmailUsingTheWebui();
 	}
 
 	/**
@@ -1038,11 +1047,8 @@ public function theUserSendsTheShareNotificationByEmailUsingTheWebui() {
 	 */
 	public function theUserShouldNotBeAbleToSendTheShareNotificationByEmailUsingTheWebui() {
 		$errorMessage = "";
-		Assert::assertNotNull(
-			$this->sharingDialog, "Sharing Dialog is not open"
-		);
 		try {
-			$this->sharingDialog->sendShareNotificationByEmail($this->getSession());
+			$this->sendShareNotificationByEmailUsingTheWebui();
 		} catch (Exception $e) {
 			$errorMessage = $e->getMessage();
 		}
