[#JBPAPP-259] EJBTHREE-997 - implement and create test for security-role-ref

[ Permlink | « Hide ]

William DeCoste [09/Aug/07 03:41 PM]

This was already merged

[ Permlink | « Hide ]

William DeCoste [10/Aug/07 01:10 PM]

was only in JBPAPP_4_2_0_CP

[ Permlink | « Hide ]

Alex Pinkin [28/Aug/07 12:03 AM]

re-opening just to set release notes flag

[ Permlink | « Hide ]

Shelly McGowan [07/Sep/07 10:16 AM]

The test has been added and passes. However, looking at how the test was implemented, I noticed there were differences in the EnterpriseBean.java code between Branch_4_2 and JBPAPP_4_2; i.e.,

diff EnterpriseBean.java EnterpriseBean-JBPAPP.java
31d30
< import org.jboss.metamodel.descriptor.SecurityRoleRef;
36a36
> import org.jboss.metamodel.descriptor.SecurityRoleRef;
43c43
< * @version <tt>$Revision: 63839 $</tt>
---
> * @version <tt>$Revision: 63832 $</tt>
97c97
< private Collection<SecurityRoleRef> securityRoleRefs = new HashSet<SecurityRoleRef>();
---
> private ArrayList<SecurityRoleRef> securityRoleRefs = new ArrayList<SecurityRoleRef>();
400d399
< sb.append(",securityRoleRefs=").append(securityRoleRefs);

[ Permlink | « Hide ]

Shelly McGowan [08/Sep/07 08:36 AM]

Fix committed:

Revision: 65234

Changes

1. [~~JBPAPP-259~~] corrected merge problems - synced with Branch_4_2 (detail)

will verify in CP01.2 release

[ Permlink | « Hide ]

Shelly McGowan [12/Sep/07 02:18 PM]

verified in 4.2.0.GA_CP01.2