Issue Details (XML | Word | Printable)

Key: OPENEJB-172
Type: Sub-task Sub-task
Status: Closed Closed
Resolution: Fixed
Priority: Major Major
Assignee: Mohammad Nour
Reporter: David Blevins
Votes: 0
Watchers: 0
Operations

If you were logged in you would be able to see more operations.
OpenEJB
OPENEJB-142

iTest: StatefulLocalJndiTests

Created: 21/Jun/06 08:09 AM   Updated: 27/Sep/07 02:25 AM  Due: 25/Sep/06
Component/s: ejb3 simplified
Affects Version/s: (trunk/openejb3)
Fix Version/s: (trunk/openejb3)

Time Tracking:
Not Specified

File Attachments:
  Size
Text File build_r448689_OPENEJB-172_and_other_fixes_patch.txt 2006-09-21 12:49 PM Mohammad Nour 185 kB
Text File r448689_OPENEJB-172_and_other_fixes_patch.txt 2006-09-22 02:22 PM Mohammad Nour 186 kB

Resolution Date: 13/Oct/06 06:19 AM
Labels:


 Description  « Hide
tests that an ejb3 stateful bean can be looked up in global JNDI via it's backwards compatible local home interface.

 All   Comments   Work Log   Change History   Subversion Commits      Sort Order: Ascending order - Click to sort in descending order
Mohammad Nour added a comment - 21/Sep/06 12:49 PM
- This patch solves the iTests errors that were showing as a result of the patch of OPENEJB-147.
- *Completed* waiting for review.

Mohammad Nour added a comment - 22/Sep/06 02:22 PM
Use this patch not the other one.

Mohammad Nour added a comment - 22/Sep/06 02:26 PM
- *Use* patch#*2*, *don't use* patch#*1*.

Jacek Laskowski added a comment - 23/Sep/06 08:51 AM
After some changes to your latest patch (classes/interfaces doubled in their files) had been applied, the build went file (!)

[INFO] ------------------------------------------------------------------------
[INFO] ------------------------------------------------------------------------
[INFO] BUILD SUCCESSFUL
[INFO] ------------------------------------------------------------------------
[INFO] Total time: 2 minutes 25 seconds
[INFO] Finished at: Sat Sep 23 08:49:26 CEST 2006
[INFO] Final Memory: 26M/254M
[INFO] ------------------------------------------------------------------------

I'm looking at the changes and if they look fine, I'll commit them. Stay tuned.

Mohammad Nour added a comment - 23/Sep/06 09:01 AM
Completed

Mohammad Nour added a comment - 23/Sep/06 09:34 AM
Resolved

Jacek Laskowski added a comment - 23/Sep/06 11:57 AM
Patch committed.

A few comments to improve your patching skills ;-)

1/ Do not use tabs as indent characters
2/ Make sure that a patch works before submitting it to jira (double- or triple-check it with an empty m2 local repo)
3/ Add ASF 2.0 license header to every source file
4/ Do not use @author tags - @version $Rev$ $Date$ is enough
5/ Remove (svn rm) rather than empty files so they won't pollute the repo or 'svn mv' them to keep their history

$ svn ci -m 'OPENEJB-172 - iTest: StatefulLocalJndiTests' .
Sending container/openejb-core/src/main/java/org/openejb/util/Index.java
Sending container/openejb-core/src/main/resources/META-INF/org.openejb/spring.xml
Sending openejb-itests/src/main/java/org/openejb/test/stateful/BasicStatefulTestClient.java
Sending openejb-itests/src/main/java/org/openejb/test/stateful/StatefulLocalTestSuite.java
Adding openejb-itests/src/main/java/org/openejb/test/stateless/BasicStatelessLocalTestClient.java
Deleting openejb-itests/src/main/java/org/openejb/test/stateless/BasicStatelessTestLocalClient.java
Sending openejb-itests/src/main/java/org/openejb/test/stateless/StatelessLocalTestSuite.java
Sending openejb-itests/src/main/java/org/openejb/test/stateless/StatelessPojoLocalJndiTests.java
Sending openejb-itests/src/main/java/org/openejb/test/stateless/StatelessTestSuite.java
Sending openejb-itests/src/main/resources/META-INF/ejb-jar.xml
Transmitting file data .........
Committed revision 449211.