From 2bc21263177a4fa2a8be125ffe179f2fd34637d1 Mon Sep 17 00:00:00 2001 From: Stian Thorgersen Date: Thu, 22 Aug 2013 12:05:34 +0100 Subject: [PATCH] Added test for changing password, and a small fix --- .../services/resources/AccountService.java | 2 - .../org/keycloak/testsuite/AccountTest.java | 56 +++++++++++++++++++ 2 files changed, 56 insertions(+), 2 deletions(-) create mode 100644 testsuite/src/test/java/org/keycloak/testsuite/AccountTest.java diff --git a/services/src/main/java/org/keycloak/services/resources/AccountService.java b/services/src/main/java/org/keycloak/services/resources/AccountService.java index 30428af419..96f8564761 100644 --- a/services/src/main/java/org/keycloak/services/resources/AccountService.java +++ b/services/src/main/java/org/keycloak/services/resources/AccountService.java @@ -182,8 +182,6 @@ public class AccountService { realm.updateCredential(user, credentials); - authManager.expireIdentityCookie(realm, uriInfo); - return Flows.forms(realm, request).setUser(user).forwardToPassword(); } else { return Response.status(Status.FORBIDDEN).build(); diff --git a/testsuite/src/test/java/org/keycloak/testsuite/AccountTest.java b/testsuite/src/test/java/org/keycloak/testsuite/AccountTest.java new file mode 100644 index 0000000000..17ba285fe1 --- /dev/null +++ b/testsuite/src/test/java/org/keycloak/testsuite/AccountTest.java @@ -0,0 +1,56 @@ +/* + * JBoss, Home of Professional Open Source. + * Copyright 2012, Red Hat, Inc., and individual contributors + * as indicated by the @author tags. See the copyright.txt file in the + * distribution for a full listing of individual contributors. + * + * This is free software; you can redistribute it and/or modify it + * under the terms of the GNU Lesser General Public License as + * published by the Free Software Foundation; either version 2.1 of + * the License, or (at your option) any later version. + * + * This software 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 + * Lesser General Public License for more details. + * + * You should have received a copy of the GNU Lesser General Public + * License along with this software; if not, write to the Free + * Software Foundation, Inc., 51 Franklin St, Fifth Floor, Boston, MA + * 02110-1301 USA, or see the FSF site: http://www.fsf.org. + */ +package org.keycloak.testsuite; + +import org.jboss.arquillian.junit.Arquillian; +import org.junit.Assert; +import org.junit.Test; +import org.junit.runner.RunWith; + +/** + * @author Stian Thorgersen + */ +@RunWith(Arquillian.class) +public class AccountTest extends AbstractDroneTest { + + @Test + public void changePassword() { + registerUser("changePassword", "password"); + + selenium.open(authServerUrl + "/rest/realms/demo/account/password"); + selenium.waitForPageToLoad("10000"); + + Assert.assertTrue(selenium.isTextPresent("Change Password")); + + selenium.type("id=password", "password"); + selenium.type("id=password-new", "newpassword"); + selenium.type("id=password-confirm", "newpassword"); + selenium.click("css=input[type=\"submit\"]"); + selenium.waitForPageToLoad("30000"); + + logout(); + + login("changePassword", "password", "Invalid username or password"); + login("changePassword", "newpassword"); + } + +}