Make acceptance test function getUserDisplayName more robust

Signed-off-by: Phil Davis <[email protected]>
owncloud · Jun 13, 2018 · 2683a3b · 2683a3b
1 parent 1cc01ee
commit 2683a3b
Showing 1 changed file with 7 additions and 6 deletions.
diff --git a/tests/acceptance/features/bootstrap/Provisioning.php b/tests/acceptance/features/bootstrap/Provisioning.php
@@ -85,7 +85,7 @@ public function getCreatedGroups() {
 	 * returns the display name of the current user
 	 * if no "Display Name" is set the user-name is returned instead
 	 *
-	 * @return array
+	 * @return string
 	 */
 	public function getCurrentUserDisplayName() {
 		return $this->getUserDisplayName($this->getCurrentUser());
@@ -97,14 +97,15 @@ public function getCurrentUserDisplayName() {
 	 *
 	 * @param string $username
 	 *
-	 * @return array
+	 * @return string
 	 */
 	public function getUserDisplayName($username) {
-		$displayName = $this->createdUsers[$username]['displayname'];
-		if (($displayName === null) || ($displayName === '')) {
-			$displayName = $username;
+		if (isset($this->createdUsers[$username]['displayname']))  {
+			if ($this->createdUsers[$username]['displayname'] != '') {
+				return $this->createdUsers[$username]['displayname'];
+			}
 		}
-		return $displayName;
+		return $username;
 	}
 
 	/**