Fix user backend unit tests

Signed-off-by: Giuliano Mele <[email protected]>

tests/unit/UserBackendTest.php

@@ -117,10 +117,6 @@ public function testUpdateAttributesWithoutAttributes() {
 			->method('get')
 			->with('ExistingUser')
 			->willReturn($user);
-		$user
-			->expects($this->once())
-			->method('getEMailAddress')
-			->willReturn(null);
 		$user
 			->expects($this->never())
 			->method('setEMailAddress');
@@ -129,6 +125,17 @@ public function testUpdateAttributesWithoutAttributes() {
 			->method('getDisplayName')
 			->with('ExistingUser')
 			->willReturn('');
+		$user->expects($this->once())
+			->method('getUID')
+			->willReturn('ExistingUser');
+		$this->groupManager
+			->expects($this->once())
+			->method('replaceGroups')
+			->with('ExistingUser', []);
+		$this->groupManager
+			->expects($this->once())
+			->method('evaluateGroupMigrations')
+			->with([]);
 		$this->userBackend->updateAttributes('ExistingUser', []);
 	}
 
@@ -173,10 +180,6 @@ public function testUpdateAttributes() {
 			->method('get')
 			->with('ExistingUser')
 			->willReturn($user);
-		$user
-			->expects($this->once())
-			->method('getEMailAddress')
-			->willReturn('[email protected]');
 		$user
 			->expects($this->once())
 			->method('setEMailAddress')
@@ -194,12 +197,21 @@ public function testUpdateAttributes() {
 			->expects($this->once())
 			->method('setDisplayName')
 			->with('ExistingUser', 'New Displayname');
-
 		$this->groupManager
 			->expects($this->once())
 			->method('replaceGroups')
-			->with($user->getUID(), ['groupB', 'groupC']);
-
+			->with('ExistingUser', ['groupB', 'groupC']);
+		$user->expects($this->once())
+			->method('getUID')
+			->willReturn('ExistingUser');
+		$this->groupManager
+			->expects($this->once())
+			->method('evaluateGroupMigrations')
+			->with(['groupB', 'groupC']);
+		$this->groupManager
+			->expects($this->once())
+			->method('evaluateGroupMigrations')
+			->with(['groupB', 'groupC']);
 		$this->userBackend->updateAttributes('ExistingUser', [
 			'email' => '[email protected]',
 			'displayname' => 'New Displayname',
@@ -234,10 +246,6 @@ public function testUpdateAttributesQuotaDefaultFallback() {
 			->method('get')
 			->with('ExistingUser')
 			->willReturn($user);
-		$user
-			->expects($this->once())
-			->method('getEMailAddress')
-			->willReturn('[email protected]');
 		$user
 			->expects($this->once())
 			->method('setEMailAddress')
@@ -255,6 +263,17 @@ public function testUpdateAttributesQuotaDefaultFallback() {
 			->expects($this->once())
 			->method('setDisplayName')
 			->with('ExistingUser', 'New Displayname');
+		$user->expects($this->once())
+			->method('getUID')
+			->willReturn('ExistingUser');
+		$this->groupManager
+			->expects($this->once())
+			->method('replaceGroups')
+			->with('ExistingUser', []);
+		$this->groupManager
+			->expects($this->once())
+			->method('evaluateGroupMigrations')
+			->with([]);
 		$this->userBackend->updateAttributes('ExistingUser', ['email' => '[email protected]', 'displayname' => 'New Displayname', 'quota' => '']);
 	}