Return-Path: Delivered-To: apmail-geronimo-scm-archive@www.apache.org Received: (qmail 93362 invoked from network); 8 Feb 2008 03:11:47 -0000 Received: from hermes.apache.org (HELO mail.apache.org) (140.211.11.2) by minotaur.apache.org with SMTP; 8 Feb 2008 03:11:47 -0000 Received: (qmail 75099 invoked by uid 500); 8 Feb 2008 03:11:39 -0000 Delivered-To: apmail-geronimo-scm-archive@geronimo.apache.org Received: (qmail 75038 invoked by uid 500); 8 Feb 2008 03:11:39 -0000 Mailing-List: contact scm-help@geronimo.apache.org; run by ezmlm Precedence: bulk list-help: list-unsubscribe: List-Post: Reply-To: dev@geronimo.apache.org List-Id: Delivered-To: mailing list scm@geronimo.apache.org Received: (qmail 75027 invoked by uid 99); 8 Feb 2008 03:11:39 -0000 Received: from athena.apache.org (HELO athena.apache.org) (140.211.11.136) by apache.org (qpsmtpd/0.29) with ESMTP; Thu, 07 Feb 2008 19:11:39 -0800 X-ASF-Spam-Status: No, hits=-2000.0 required=10.0 tests=ALL_TRUSTED X-Spam-Check-By: apache.org Received: from [140.211.11.3] (HELO eris.apache.org) (140.211.11.3) by apache.org (qpsmtpd/0.29) with ESMTP; Fri, 08 Feb 2008 03:11:18 +0000 Received: by eris.apache.org (Postfix, from userid 65534) id 1407B1A9832; Thu, 7 Feb 2008 19:11:26 -0800 (PST) Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit Subject: svn commit: r619750 - /geronimo/server/branches/2.0/modules/geronimo-security/src/test/java/org/apache/geronimo/security/realm/providers/GeronimoPropertiesFileMappedPasswordCredentialLoginModuleTest.java Date: Fri, 08 Feb 2008 03:11:25 -0000 To: scm@geronimo.apache.org From: dwoods@apache.org X-Mailer: svnmailer-1.0.8 Message-Id: <20080208031126.1407B1A9832@eris.apache.org> X-Virus-Checked: Checked by ClamAV on apache.org Author: dwoods Date: Thu Feb 7 19:11:22 2008 New Revision: 619750 URL: http://svn.apache.org/viewvc?rev=619750&view=rev Log: GERONIMO-3830 Test assumes a specific Set ordering, which fails on Java 6. Toby, thanks for the patch. Modified: geronimo/server/branches/2.0/modules/geronimo-security/src/test/java/org/apache/geronimo/security/realm/providers/GeronimoPropertiesFileMappedPasswordCredentialLoginModuleTest.java Modified: geronimo/server/branches/2.0/modules/geronimo-security/src/test/java/org/apache/geronimo/security/realm/providers/GeronimoPropertiesFileMappedPasswordCredentialLoginModuleTest.java URL: http://svn.apache.org/viewvc/geronimo/server/branches/2.0/modules/geronimo-security/src/test/java/org/apache/geronimo/security/realm/providers/GeronimoPropertiesFileMappedPasswordCredentialLoginModuleTest.java?rev=619750&r1=619749&r2=619750&view=diff ============================================================================== --- geronimo/server/branches/2.0/modules/geronimo-security/src/test/java/org/apache/geronimo/security/realm/providers/GeronimoPropertiesFileMappedPasswordCredentialLoginModuleTest.java (original) +++ geronimo/server/branches/2.0/modules/geronimo-security/src/test/java/org/apache/geronimo/security/realm/providers/GeronimoPropertiesFileMappedPasswordCredentialLoginModuleTest.java Thu Feb 7 19:11:22 2008 @@ -20,15 +20,16 @@ package org.apache.geronimo.security.realm.providers; -import java.util.Set; -import java.util.HashSet; +import java.util.Comparator; import java.util.Iterator; +import java.util.Set; +import java.util.TreeSet; import junit.framework.TestCase; import org.apache.geronimo.security.jaas.NamedUsernamePasswordCredential; /** - * @version $Rev:$ $Date:$ + * @version $Rev$ $Date$ */ public class GeronimoPropertiesFileMappedPasswordCredentialLoginModuleTest extends TestCase { private GeronimoPropertiesFileMappedPasswordCredentialLoginModule loginModule; @@ -36,7 +37,11 @@ protected void setUp() { loginModule = new GeronimoPropertiesFileMappedPasswordCredentialLoginModule(); - passwordCredentials = new HashSet(); + passwordCredentials = new TreeSet(new Comparator() { + public int compare(Object o1, Object o2) { + return ((NamedUsernamePasswordCredential)o1).getName().compareTo(((NamedUsernamePasswordCredential)o2).getName()); + } + }); } public void testParsingOne() throws Exception {