keycloak-aplcache

Details

diff --git a/forms/common-themes/src/main/resources/theme/admin/base/resources/partials/user-federated-identity.html b/forms/common-themes/src/main/resources/theme/admin/base/resources/partials/user-federated-identity.html
index 70a2691..dd23d7d 100644
--- a/forms/common-themes/src/main/resources/theme/admin/base/resources/partials/user-federated-identity.html
+++ b/forms/common-themes/src/main/resources/theme/admin/base/resources/partials/user-federated-identity.html
@@ -17,7 +17,7 @@
         <table class="table table-striped table-bordered">
             <thead>
             <tr>
-                <th>Identity Provider Name</th>
+                <th>Identity Provider Alias</th>
                 <th>Username</th>
             </tr>
             </thead>
diff --git a/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java b/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java
index 5732688..6b694a7 100755
--- a/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java
+++ b/services/src/main/java/org/keycloak/services/resources/admin/UsersResource.java
@@ -267,9 +267,6 @@ public class UsersResource {
             for (IdentityProviderModel identityProviderModel : realm.getIdentityProviders()) {
                 if (identityProviderModel.getId().equals(identity.getIdentityProvider())) {
                     FederatedIdentityRepresentation rep = ModelToRepresentation.toRepresentation(identity);
-
-                    rep.setIdentityProvider(identityProviderModel.getName());
-
                     result.add(rep);
                 }
             }
diff --git a/testsuite/integration/src/test/java/org/keycloak/testsuite/admin/UserTest.java b/testsuite/integration/src/test/java/org/keycloak/testsuite/admin/UserTest.java
index d2c76dd..976223c 100755
--- a/testsuite/integration/src/test/java/org/keycloak/testsuite/admin/UserTest.java
+++ b/testsuite/integration/src/test/java/org/keycloak/testsuite/admin/UserTest.java
@@ -1,9 +1,12 @@
 package org.keycloak.testsuite.admin;
 
+import org.junit.Assert;
 import org.junit.Ignore;
 import org.junit.Test;
+import org.keycloak.admin.client.resource.IdentityProviderResource;
 import org.keycloak.admin.client.resource.UserResource;
 import org.keycloak.representations.idm.FederatedIdentityRepresentation;
+import org.keycloak.representations.idm.IdentityProviderRepresentation;
 import org.keycloak.representations.idm.UserRepresentation;
 
 import javax.ws.rs.ClientErrorException;
@@ -187,44 +190,53 @@ public class UserTest extends AbstractClientTest {
     }
 
     @Test
-    public void addFederatedIdentity() {
-        createUser();
+    public void getFederatedIdentities() {
+        // Add sample identity provider
+        addSampleIdentityProvider();
 
+        // Add sample user
+        createUser();
         UserResource user = realm.users().get("user1");
+        assertEquals(0, user.getFederatedIdentity().size());
 
+        // Add social link to the user
         FederatedIdentityRepresentation link = new FederatedIdentityRepresentation();
         link.setUserId("social-user-id");
         link.setUserName("social-username");
-
         Response response = user.addFederatedIdentity("social-provider-id", link);
         assertEquals(204, response.getStatus());
-    }
-
-    @Test
-    @Ignore("Refactor based on KEYCLOAK-883")
-    public void getFederatedIdentities() {
-        addFederatedIdentity();
-
-        UserResource user = realm.users().get("user1");
-        assertEquals(1, user.getFederatedIdentity().size());
 
-        FederatedIdentityRepresentation link = user.getFederatedIdentity().get(0);
+        // Verify social link is here
+        user = realm.users().get("user1");
+        List<FederatedIdentityRepresentation> federatedIdentities = user.getFederatedIdentity();
+        assertEquals(1, federatedIdentities.size());
+        link = federatedIdentities.get(0);
         assertEquals("social-provider-id", link.getIdentityProvider());
         assertEquals("social-user-id", link.getUserId());
         assertEquals("social-username", link.getUserName());
-    }
 
-    @Test
-    @Ignore("Refactor based on KEYCLOAK-883")
-    public void removeFederatedIdentity() {
-        addFederatedIdentity();
+        // Remove social link now
+        user.removeFederatedIdentity("social-provider-id");
+        assertEquals(0, user.getFederatedIdentity().size());
 
-        UserResource user = realm.users().get("user1");
-        assertEquals(1, user.getFederatedIdentity().size());
+        removeSampleIdentityProvider();
+    }
 
-        user.removeFederatedIdentity("social-provider-id");
+    private void addSampleIdentityProvider() {
+        List<IdentityProviderRepresentation> providers = realm.identityProviders().findAll();
+        Assert.assertEquals(0, providers.size());
 
-        assertEquals(0, user.getFederatedIdentity().size());
+        IdentityProviderRepresentation rep = new IdentityProviderRepresentation();
+        rep.setId("social-provider-id");
+        rep.setName("social-provider-name");
+        rep.setProviderId("social-provider-type");
+        realm.identityProviders().create(rep);
+    }
+
+    private void removeSampleIdentityProvider() {
+        IdentityProviderResource resource = realm.identityProviders().get("social-provider-id");
+        Assert.assertNotNull(resource);
+        resource.remove();
     }
 
     @Test