1 files changed, 20 insertions, 28 deletions

diff --git a/build/integration/features/bootstrap/Avatar.php b/build/integration/features/bootstrap/Avatar.php
index 6e59beeb7de..beebf1c024a 100644
--- a/build/integration/features/bootstrap/Avatar.php
+++ b/build/integration/features/bootstrap/Avatar.php
@@ -1,24 +1,8 @@
 <?php
+
 /**
- * @copyright Copyright (c) 2020, Daniel Calviño Sánchez (danxuliu@gmail.com)
- *
- * @author Daniel Calviño Sánchez <danxuliu@gmail.com>
- *
- * @license GNU AGPL version 3 or any later version
- *
- * This program is free software: you can redistribute it and/or modify
- * it under the terms of the GNU Affero General Public License as
- * published by the Free Software Foundation, either version 3 of the
- * License, or (at your option) any later version.
- *
- * This program is distributed in the hope that it will be useful,
- * but WITHOUT ANY WARRANTY; without even the implied warranty of
- * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the
- * GNU Affero General Public License for more details.
- *
- * You should have received a copy of the GNU Affero General Public License
- * along with this program. If not, see <http://www.gnu.org/licenses/>.
- *
+ * SPDX-FileCopyrightText: 2020 Nextcloud GmbH and Nextcloud contributors
+ * SPDX-License-Identifier: AGPL-3.0-or-later
  */
 use Behat\Gherkin\Node\TableNode;
 use PHPUnit\Framework\Assert;
@@ -26,8 +10,7 @@ use PHPUnit\Framework\Assert;
 require __DIR__ . '/../../vendor/autoload.php';
 
 trait Avatar {
-
-	/** @var string **/
+	/** @var string * */
 	private $lastAvatar;
 
 	/** @AfterScenario **/
@@ -174,11 +157,20 @@ trait Avatar {
 	public function lastAvatarIsASquareOfSize(string $size) {
 		[$width, $height] = getimagesizefromstring($this->lastAvatar);
 
-		Assert::assertEquals($width, $height, 'Avatar is not a square');
+		Assert::assertEquals($width, $height, 'Expected avatar to be a square');
 		Assert::assertEquals($size, $width);
 	}
 
 	/**
+	 * @Then last avatar is not a square
+	 */
+	public function lastAvatarIsNotASquare() {
+		[$width, $height] = getimagesizefromstring($this->lastAvatar);
+
+		Assert::assertNotEquals($width, $height, 'Expected avatar to not be a square');
+	}
+
+	/**
 	 * @Then last avatar is not a single color
 	 */
 	public function lastAvatarIsNotASingleColor() {
@@ -249,10 +241,10 @@ trait Avatar {
 	}
 
 	private function isSameColor(array $firstColor, array $secondColor, int $allowedDelta = 1) {
-		if ($this->isSameColorComponent($firstColor['red'], $secondColor['red'], $allowedDelta) &&
-			$this->isSameColorComponent($firstColor['green'], $secondColor['green'], $allowedDelta) &&
-			$this->isSameColorComponent($firstColor['blue'], $secondColor['blue'], $allowedDelta) &&
-			$this->isSameColorComponent($firstColor['alpha'], $secondColor['alpha'], $allowedDelta)) {
+		if ($this->isSameColorComponent($firstColor['red'], $secondColor['red'], $allowedDelta)
+			&& $this->isSameColorComponent($firstColor['green'], $secondColor['green'], $allowedDelta)
+			&& $this->isSameColorComponent($firstColor['blue'], $secondColor['blue'], $allowedDelta)
+			&& $this->isSameColorComponent($firstColor['alpha'], $secondColor['alpha'], $allowedDelta)) {
 			return true;
 		}
 
@@ -260,8 +252,8 @@ trait Avatar {
 	}
 
 	private function isSameColorComponent(int $firstColorComponent, int $secondColorComponent, int $allowedDelta) {
-		if ($firstColorComponent >= ($secondColorComponent - $allowedDelta) &&
-			$firstColorComponent <= ($secondColorComponent + $allowedDelta)) {
+		if ($firstColorComponent >= ($secondColorComponent - $allowedDelta)
+			&& $firstColorComponent <= ($secondColorComponent + $allowedDelta)) {
 			return true;
 		}