From 1c3d165e5da57bd992a43f6a58fafafee7a94a4b Mon Sep 17 00:00:00 2001 From: Vlastimil Elias Date: Tue, 2 Jun 2015 15:41:50 +0200 Subject: [PATCH] KEYCLOAK-1372 - tests improved --- .../keycloak/testsuite/broker/AbstractIdentityProviderTest.java | 2 ++ .../broker/SAMLKeyCloakServerBrokerWithSignatureTest.java | 2 ++ 2 files changed, 4 insertions(+) diff --git a/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/AbstractIdentityProviderTest.java b/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/AbstractIdentityProviderTest.java index 1ad0207225..c145d97768 100755 --- a/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/AbstractIdentityProviderTest.java +++ b/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/AbstractIdentityProviderTest.java @@ -637,10 +637,12 @@ public abstract class AbstractIdentityProviderTest { String userLastName = "New last"; assertEquals(expectedEmail, federatedUser.getEmail()); + assertFalse(federatedUser.isEmailVerified()); assertEquals(userFirstName, federatedUser.getFirstName()); assertEquals(userLastName, federatedUser.getLastName()); } else { assertEquals(expectedEmail, federatedUser.getEmail()); + assertFalse(federatedUser.isEmailVerified()); assertEquals("Test", federatedUser.getFirstName()); assertEquals("User", federatedUser.getLastName()); } diff --git a/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/SAMLKeyCloakServerBrokerWithSignatureTest.java b/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/SAMLKeyCloakServerBrokerWithSignatureTest.java index 821fc0e51a..7252d5345a 100755 --- a/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/SAMLKeyCloakServerBrokerWithSignatureTest.java +++ b/testsuite/integration/src/test/java/org/keycloak/testsuite/broker/SAMLKeyCloakServerBrokerWithSignatureTest.java @@ -57,8 +57,10 @@ public class SAMLKeyCloakServerBrokerWithSignatureTest extends AbstractIdentityP if (expectedEmail == null) { // Need to handle differences for various databases (like Oracle) assertTrue(federatedUser.getEmail() == null || federatedUser.getEmail().equals("")); + assertFalse(federatedUser.isEmailVerified()); } else { assertEquals(expectedEmail, federatedUser.getEmail()); + assertFalse(federatedUser.isEmailVerified()); } assertNull(federatedUser.getFirstName()); assertNull(federatedUser.getLastName());