simplifying user feature in UI tests

owncloud · Sep 19, 2017 · ce7e630 · ce7e630
1 parent 8c03224
commit ce7e630
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 15 deletions.
diff --git a/tests/ui/features/bootstrap/UsersContext.php b/tests/ui/features/bootstrap/UsersContext.php
@@ -47,6 +47,15 @@ public function __construct(UsersPage $usersPage) {
 		$this->usersPage = $usersPage;
 	}
 
+	/**
+	 * @Given I am on the users page
+	 * @return void
+	 */
+	public function iAmOnTheUsersPage() {
+		$this->usersPage->open();
+		$this->usersPage->waitTillPageIsLoaded($this->getSession());
+	}
+
 	/**
 	 * substitute codes like "%regularuser%" with the actual name of the user
 	 *
@@ -59,26 +68,12 @@ public function checkUsername($username) {
 	}
 
 	/**
-	 * @Given quota of user :username is set to :quota
+	 * @Given quota of user :username is set/changed to :quota
 	 * @param string $username
 	 * @param string $quota
 	 * @return void
 	 */
 	public function quotaOfUserIsSetTo($username, $quota) {
-		$this->usersPage->open();
-		$this->usersPage->waitTillPageIsLoaded($this->getSession());
-		$this->usersPage->setQuotaOfUserTo($username, $quota, $this->getSession());
-	}
-
-	/**
-	 * @When quota of user :username is changed to :quota
-	 * @param string $username
-	 * @param string $quota
-	 * @return void
-	 */
-	public function quotaOfUserIsChangedTo($username, $quota) {
-		$this->usersPage->open();
-		$this->usersPage->waitTillPageIsLoaded($this->getSession());
 		$this->usersPage->setQuotaOfUserTo($username, $quota, $this->getSession());
 	}
 

diff --git a/tests/ui/features/other/users.feature b/tests/ui/features/other/users.feature
@@ -3,6 +3,7 @@ Feature: users
 	Background:
 		Given a regular user exists
 		And I am logged in as admin
+		And I am on the users page
 
 	Scenario Outline: change quota to a valid value
 		And quota of user "%regularuser%" is set to "<start_quota>"