You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@tomee.apache.org by Mohammad Nour El-Din <no...@gmail.com> on 2006/09/02 13:32:00 UTC
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Hi Dave...
Sorry for being late to reply, I added the missing files and made the
changes you asked for.
Here is the new patch http://jira.codehaus.org/browse/OPENEJB-146. I will
work on *OPENEJB-147* & *OPENEJB-172*, please assign them to me. One more
favor, please take a look at this mail
http://marc.theaimsgroup.com/?l=openejb-development&m=115699229501566&w=2 .
Thanks and best regards...
Mohammad Nour El-Din
On 8/31/06, David Blevins <da...@visi.com> wrote:
> Hi Mohammad!
>
> On Aug 30, 2006, at 8:12 PM, Mohammad Nour El-Din wrote:
>
> > Hi Dave...
> >
> > I have submited the patch of OPENEJB-146, please review and assign
> > to me
> > OPENEJB-147 :)
>
> Excellent! I reviewed the patch and have a couple of notes:
>
> The first is simple, you forgot to 'svn add' the component local
> interfaces :)
>
> Also, go ahead and put the local interfaces on the
> BasicStatelessPojoBean that we created in the last set of tests. No
> need to add inheritance and copy the bean def in the descriptors. My
> original note in that jira was actually about splitting the component
> local and business local interface tests into two beans, but we don't
> need to do that for the component remote and the component local tests.
>
> Looking great! You're starting to cook :)
>
> -David
>
> > Thanks and best regards...
> > Mohammad Nour El-Din
> >
> >
> > On 8/30/06, Mohammed Nour < nour.mohammad@gmail.com> wrote:
> >
> >> Hi Daves...
> >>
> >> On 8/30/06, David Blevins <da...@visi.com> wrote:
> >> >
> >> > Hi Mohammads... :)
> >> >
> >> > Now that's a big sentence! Edgar Allan Poe would be jealous.
> >> >
> >> > Comments below.
> >> >
> >> > On Aug 29, 2006, at 3:11 PM, Mohammed Nour wrote:
> >> >
> >> > > Hi Daves...
> >> > >
> >> > > Sure you can assign it to me, it will be my honor and
> >> pleasure :),
> >> > > but -
> >> > > correct me if I am wrong - I have noticed that there is no iTests
> >> > > for using
> >> > > local interfaces, so I have some simple re-factoring ideas so we
> >> > > can make
> >> > > iTests for using local interfaces, I am working on them and will
> >> > > send you a
> >> > > patch, this of caurse will include the re-factored impl of the
> >> two
> >> > > iTests
> >> > > that you and me did.
> >> >
> >> > Ok. I updated OPENEJB-146 "iTest: StatelessLocalJndiTests" so
> >> it no
> >> > longer includes testing the local business interface -- probably
> >> > should have been that way from the beginning.
> >> >
> >> > I'll leave 146 assigned to you until I hear otherwise.
> >> >
> >> > The tests OPENEJB-145 and OPENEJB-171 don't need to be updated as
> >> > they intentionally are only for remote interfaces. Unless when you
> >> > say "refactor" you mean you simply adding a local and local-home
> >> > interfaces to the POJOs we created. That would be fine.
> >> >
> >> > > I want to ask you a favor, can I have a JIRA of a
> >> > > feature to impl beside the iTests, if it is OK with you I will
> >> be very
> >> > > happy, but if u see that I rush to have more tasks so I will
> >> stick
> >> > > with the
> >> > > iTests till I finish them all and then go into new features
> >> JIRAs.
> >> >
> >> > You can work on whatever you think you can do. I personally don't
> >> > take more than one or two at a time cause I just can't do more than
> >> > one or two at a time. I'd love to do them all.
> >>
> >>
> >> Thanks for the advice, and I will stick with it :)
> >>
> >> -David
> >> >
> >> > > Thanks and best regards...
> >> > > Mohammad Nour El-Din
> >> > >
> >> > >
> >> > > On 8/29/06, David Blevins < david.blevins@visi.com> wrote:
> >> > >
> >> > >> Great patch, Mohammad. It applied, built and ran perfectly.
> >> > >>
> >> > >> Regarding OPENEJB-146, as I've said here (http://
> >> > >> marc.theaimsgroup.com/?l=openejb-
> >> development&m=115561420703000&w=2)
> >> > >> doing that one requires a whole bunch of functionality we
> >> don't have
> >> > >> yet, so there are probably better jiras to work on.
> >> > >>
> >> > >> OPENEJB-147 would be a good one. That will be easier than
> >> the one
> >> > >> you just did as you won't even have to create a new bean and
> >> add it
> >> > >> to the various deployment descriptors.
> >> > >>
> >> > >> Let me know and I'll assign it to you.
> >> > >>
> >> > >> Thanks!
> >> > >> David
> >> > >>
> >> > >>
> >> > >> On Aug 27, 2006, at 5:12 AM, Mohammed Nour wrote:
> >> > >>
> >> > >> > Hi Daves...
> >> > >> >
> >> > >> > I attached the new diff file to OPENEJB-171, I will work on
> >> > >> > OPENEJB-146 with
> >> > >> > the same procedure.
> >> > >> >
> >> > >> > Thanks and best regards...
> >> > >> > Mohammad Nour El-Din
> >> > >> >
> >> > >> >
> >> > >> > On 8/27/06, David Blevins <david.blevins@visi.com > wrote:
> >> > >> >
> >> > >> >> On Aug 26, 2006, at 1:31 PM, David Blevins wrote:
> >> > >> >> > I'll take a look at this and get back to you later today.
> >> > >> >>
> >> > >> >> The patch looks good, you're just missing a few files.
> >> Not a big
> >> > >> >> deal, you just need to 'svn add' them so svn will include
> >> them
> >> > >> in the
> >> > >> >> patch. Give that a try and attach a new patch file here:
> >> http://
> >> > >> >> jira.codehaus.org/browse/OPENEJB-171
> >> > >> >>
> >> > >> >> > On Aug 26, 2006, at 10:59 AM, Mohammed Nour wrote:
> >> > >> >> >
> >> > >> >> >> Hi Daves...
> >> > >> >> >> I followed the steps you explained to me, but I have a
> >> > >> couple of
> >> > >> >> >> stupid questions :) :
> >> > >> >> >> I don't see any thing the build output which indicates
> >> that the
> >> > >> >> >> test case was running, why?
> >> > >> >>
> >> > >> >> It's just the way the surefire plugin works. It only
> >> displays the
> >> > >> >> names of the tests that are in that module and we have all
> >> the
> >> > >> itests
> >> > >> >> in another module. If it fails though, there will be a
> >> report
> >> > >> under
> >> > >> >> openejb-core/target/surefire-reports/
> >> > >> >>
> >> > >> >> >> What is the use of spring.xml, is it for assembling the
> >> > >> components
> >> > >> >> >> of the EJB container? If yes, do I have to read about
> >> spring to
> >> >
> >> > >> >> >> know the details of this XML file ?
> >> > >> >>
> >> > >> >> Yes, and it's experimental. You don't need to understand
> >> it to
> >> > >> work
> >> > >> >> on OpenEJB. You'll need to add OpenEJB deployments to it
> >> whenever
> >> >
> >> > >> >> you add them to the ejb-jar.xml, but we can help with that
> >> if the
> >> > >> >> need comes up again.
> >> > >> >>
> >> > >> >> -David
> >> > >> >>
> >> > >> >>
> >> > >>
> >> > >>
> >> >
> >> >
> >>
> >>
>
>
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Mohammad Nour El-Din <no...@gmail.com>.
Sorry I mean I had pressure at work :D
On 9/14/06, Mohammad Nour El-Din <no...@gmail.com> wrote:
>
> Hi Dave...
>
> Sorry for being late to submit patches but I has pressure at work, I
> submitted the patch of OPENEJB-147
> http://jira.codehaus.org/browse/OPENEJB-147, waiting for ur review.
>
>
>
> On 9/3/06, Mohammad Nour El-Din <no...@gmail.com> wrote:
>
> > Hi Dave...
> >
> > On 9/2/06, David Blevins <david.blevins@visi.com > wrote:
> > >
> > > Excellent work Mohammad! I applied an updated version of the patch
> > > which contains the modifications I mentioned before. I went ahead
> > > and assigned you OPENEJB-147 and OPENEJB-172.
> > >
> > > Will take a look at that email you mention.
> > >
> > > It's so great to have your help! Thank you!
> >
> >
> > You are so welcome :)
> >
> > -David
> > >
> > >
> > > On Sep 2, 2006, at 4:32 AM, Mohammad Nour El-Din wrote:
> > >
> > > > Hi Dave...
> > > >
> > > > Sorry for being late to reply, I added the missing files and made
> > > the
> > > > changes you asked for.
> > > > Here is the new patch http://jira.codehaus.org/browse/OPENEJB-146 .
> > > > I will
> > > > work on *OPENEJB-147* & *OPENEJB-172*, please assign them to me.
> > > > One more
> > > > favor, please take a look at this mail
> > > > http://marc.theaimsgroup.com/?l=openejb-
> > > > development&m=115699229501566&w=2 .
> > > >
> > > > Thanks and best regards...
> > > > Mohammad Nour El-Din
> > > >
> > > >
> > > > On 8/31/06, David Blevins < david.blevins@visi.com> wrote:
> > > >
> > > >> Hi Mohammad!
> > > >>
> > > >> On Aug 30, 2006, at 8:12 PM, Mohammad Nour El-Din wrote:
> > > >>
> > > >> > Hi Dave...
> > > >> >
> > > >> > I have submited the patch of OPENEJB-146, please review and
> > > assign
> > > >> > to me
> > > >> > OPENEJB-147 :)
> > > >>
> > > >> Excellent! I reviewed the patch and have a couple of notes:
> > > >>
> > > >> The first is simple, you forgot to 'svn add' the component local
> > > >> interfaces :)
> > > >>
> > > >> Also, go ahead and put the local interfaces on the
> > > >> BasicStatelessPojoBean that we created in the last set of
> > > tests. No
> > > >> need to add inheritance and copy the bean def in the
> > > descriptors. My
> > > >> original note in that jira was actually about splitting the
> > > component
> > > >> local and business local interface tests into two beans, but we
> > > don't
> > > >> need to do that for the component remote and the component local
> > > >> tests.
> > > >>
> > > >> Looking great! You're starting to cook :)
> > > >>
> > > >> -David
> > > >>
> > > >> > Thanks and best regards...
> > > >> > Mohammad Nour El-Din
> > > >> >
> > > >> >
> > > >> > On 8/30/06, Mohammed Nour < nour.mohammad@gmail.com> wrote:
> > > >> >
> > > >> >> Hi Daves...
> > > >> >>
> > > >> >> On 8/30/06, David Blevins < david.blevins@visi.com> wrote:
> > > >> >> >
> > > >> >> > Hi Mohammads... :)
> > > >> >> >
> > > >> >> > Now that's a big sentence! Edgar Allan Poe would be jealous.
> > > >> >> >
> > > >> >> > Comments below.
> > > >> >> >
> > > >> >> > On Aug 29, 2006, at 3:11 PM, Mohammed Nour wrote:
> > > >> >> >
> > > >> >> > > Hi Daves...
> > > >> >> > >
> > > >> >> > > Sure you can assign it to me, it will be my honor and
> > > >> >> pleasure :),
> > > >> >> > > but -
> > > >> >> > > correct me if I am wrong - I have noticed that there is no
> > > >> iTests
> > > >> >> > > for using
> > > >> >> > > local interfaces, so I have some simple re-factoring ideas
> > > >> so we
> > > >> >> > > can make
> > > >> >> > > iTests for using local interfaces, I am working on them and
> > > >> will
> > > >> >> > > send you a
> > > >> >> > > patch, this of caurse will include the re-factored impl of
> > > the
> > > >> >> two
> > > >> >> > > iTests
> > > >> >> > > that you and me did.
> > > >> >> >
> > > >> >> > Ok. I updated OPENEJB-146 "iTest: StatelessLocalJndiTests" so
> > >
> > > >> >> it no
> > > >> >> > longer includes testing the local business interface --
> > > probably
> > > >> >> > should have been that way from the beginning.
> > > >> >> >
> > > >> >> > I'll leave 146 assigned to you until I hear otherwise.
> > > >> >> >
> > > >> >> > The tests OPENEJB-145 and OPENEJB-171 don't need to be
> > > >> updated as
> > > >> >> > they intentionally are only for remote interfaces. Unless
> > > >> when you
> > > >> >> > say "refactor" you mean you simply adding a local and
> > > local-home
> > > >> >> > interfaces to the POJOs we created. That would be fine.
> > > >> >> >
> > > >> >> > > I want to ask you a favor, can I have a JIRA of a
> > > >> >> > > feature to impl beside the iTests, if it is OK with you I
> > > will
> > > >> >> be very
> > > >> >> > > happy, but if u see that I rush to have more tasks so I will
> > > >> >> stick
> > > >> >> > > with the
> > > >> >> > > iTests till I finish them all and then go into new features
> > > >> >> JIRAs.
> > > >> >> >
> > > >> >> > You can work on whatever you think you can do. I personally
> > > >> don't
> > > >> >> > take more than one or two at a time cause I just can't do
> > > >> more than
> > > >> >> > one or two at a time. I'd love to do them all.
> > > >> >>
> > > >> >>
> > > >> >> Thanks for the advice, and I will stick with it :)
> > > >> >>
> > > >> >> -David
> > > >> >> >
> > > >> >> > > Thanks and best regards...
> > > >> >> > > Mohammad Nour El-Din
> > > >> >> > >
> > > >> >> > >
> > > >> >> > > On 8/29/06, David Blevins < david.blevins@visi.com > wrote:
> > > >> >> > >
> > > >> >> > >> Great patch, Mohammad. It applied, built and ran
> > > perfectly.
> > > >> >> > >>
> > > >> >> > >> Regarding OPENEJB-146, as I've said here (http://
> > > >> >> > >> marc.theaimsgroup.com/?l=openejb-
> > > >> >> development&m=115561420703000&w=2)
> > > >> >> > >> doing that one requires a whole bunch of functionality we
> > > >> >> don't have
> > > >> >> > >> yet, so there are probably better jiras to work on.
> > > >> >> > >>
> > > >> >> > >> OPENEJB-147 would be a good one. That will be easier than
> > > >> >> the one
> > > >> >> > >> you just did as you won't even have to create a new bean
> > > and
> > > >> >> add it
> > > >> >> > >> to the various deployment descriptors.
> > > >> >> > >>
> > > >> >> > >> Let me know and I'll assign it to you.
> > > >> >> > >>
> > > >> >> > >> Thanks!
> > > >> >> > >> David
> > > >> >> > >>
> > > >> >> > >>
> > > >> >> > >> On Aug 27, 2006, at 5:12 AM, Mohammed Nour wrote:
> > > >> >> > >>
> > > >> >> > >> > Hi Daves...
> > > >> >> > >> >
> > > >> >> > >> > I attached the new diff file to OPENEJB-171, I will work
> > > on
> > > >> >> > >> > OPENEJB-146 with
> > > >> >> > >> > the same procedure.
> > > >> >> > >> >
> > > >> >> > >> > Thanks and best regards...
> > > >> >> > >> > Mohammad Nour El-Din
> > > >> >> > >> >
> > > >> >> > >> >
> > > >> >> > >> > On 8/27/06, David Blevins < david.blevins@visi.com >
> > > wrote:
> > > >> >> > >> >
> > > >> >> > >> >> On Aug 26, 2006, at 1:31 PM, David Blevins wrote:
> > > >> >> > >> >> > I'll take a look at this and get back to you later
> > > >> today.
> > > >> >> > >> >>
> > > >> >> > >> >> The patch looks good, you're just missing a few files.
> > > >> >> Not a big
> > > >> >> > >> >> deal, you just need to 'svn add' them so svn will
> > > include
> > > >> >> them
> > > >> >> > >> in the
> > > >> >> > >> >> patch. Give that a try and attach a new patch file
> > > here:
> > > >> >> http://
> > > >> >> > >> >> jira.codehaus.org/browse/OPENEJB-171
> > > >> >> > >> >>
> > > >> >> > >> >> > On Aug 26, 2006, at 10:59 AM, Mohammed Nour wrote:
> > > >> >> > >> >> >
> > > >> >> > >> >> >> Hi Daves...
> > > >> >> > >> >> >> I followed the steps you explained to me, but I have
> > > a
> > > >> >> > >> couple of
> > > >> >> > >> >> >> stupid questions :) :
> > > >> >> > >> >> >> I don't see any thing the build output which
> > > indicates
> > > >> >> that the
> > > >> >> > >> >> >> test case was running, why?
> > > >> >> > >> >>
> > > >> >> > >> >> It's just the way the surefire plugin works. It only
> > > >> >> displays the
> > > >> >> > >> >> names of the tests that are in that module and we have
> > > all
> > > >> >> the
> > > >> >> > >> itests
> > > >> >> > >> >> in another module. If it fails though, there will be a
> > > >> >> report
> > > >> >> > >> under
> > > >> >> > >> >> openejb-core/target/surefire-reports/
> > > >> >> > >> >>
> > > >> >> > >> >> >> What is the use of spring.xml, is it for assembling
> > > the
> > > >> >> > >> components
> > > >> >> > >> >> >> of the EJB container? If yes, do I have to read about
> > > >> >> spring to
> > > >> >> >
> > > >> >> > >> >> >> know the details of this XML file ?
> > > >> >> > >> >>
> > > >> >> > >> >> Yes, and it's experimental. You don't need to
> > > understand
> > > >> >> it to
> > > >> >> > >> work
> > > >> >> > >> >> on OpenEJB. You'll need to add OpenEJB deployments to
> > > it
> > > >> >> whenever
> > > >> >> >
> > > >> >> > >> >> you add them to the ejb-jar.xml, but we can help with
> > > that
> > > >> >> if the
> > > >> >> > >> >> need comes up again.
> > > >> >> > >> >>
> > > >> >> > >> >> -David
> > > >> >> > >> >>
> > > >> >> > >> >>
> > > >> >> > >>
> > > >> >> > >>
> > > >> >> >
> > > >> >> >
> > > >> >>
> > > >> >>
> > > >>
> > > >>
> > >
> > >
> >
> >
>
>
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Jacek Laskowski <ja...@laskowski.net.pl>.
On 9/14/06, Mohammad Nour El-Din <no...@gmail.com> wrote:
>
> Hi Jacek...
>
> Yeah this error appears to me too, this is why I created this JIRA
> http://jira.codehaus.org/browse/OPENEJB-264, but the itests
> run successfully when using spring.xml .
So, how can/should I test your patch? Why is it not failing for the
other tests? Just looking for more information before we decide to
"enhance" spring2.xml with stuff that eventually turns out
unnecesssary.
Jacek
--
Jacek Laskowski
http://www.laskowski.net.pl
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Mohammad Nour El-Din <no...@gmail.com>.
Hi Jacek...
Yeah this error appears to me too, this is why I created this JIRA
http://jira.codehaus.org/browse/OPENEJB-264, but the itests run successfully
when using spring.xml .
On 9/14/06, Jacek Laskowski <ja...@laskowski.net.pl> wrote:
> On 9/14/06, Jacek Laskowski <ja...@laskowski.net.pl> wrote:
> > On 9/14/06, David Blevins <da...@visi.com> wrote:
> >
> > > I'm rebuilding gbuild today and will be working on moving the svn
> > > tomorrow. Is there someone who can look at this patch today?
> >
> > But of course! ;-) I'm doing it now...
>
> When I run the itests after the patch had been applied, I got a tests
> failure:
>
> $ less
> container/openejb-core/target/surefire-reports/org.openejb.SpringAssembler2Test.txt
>
> -------------------------------------------------------------------------------
> Test set: org.openejb.SpringAssembler2Test
>
> -------------------------------------------------------------------------------
> Tests run: 398, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
> 5.796 sec <<< FAILURE!
> Stateless.PojoLocalHomeIntfc..setUp() Time elapsed: 0 sec <<< ERROR!
> javax.naming.NameNotFoundException: Name
> "client/tests/stateless/BasicStatelessPojoLocalHome" not found.
> at org.openejb.core.ivm.naming.IvmContext.federate(IvmContext.java
> :138)
> at org.openejb.core.ivm.naming.IvmContext.lookup(IvmContext.java
> :101)
> at javax.naming.InitialContext.lookup(InitialContext.java:351)
> at
> org.openejb.test.stateless.StatelessPojoLocalHomeIntfcTests.setUp(
> StatelessPojoLocalHomeIntfcTests.java:19)
> at org.openejb.test.NumberedTestCase.run(NumberedTestCase.java:77)
> at junit.framework.TestSuite.runTest(TestSuite.java:208)
> at junit.framework.TestSuite.run(TestSuite.java:203)
> at org.openejb.test.TestSuite.run(TestSuite.java:29)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at sun.reflect.NativeMethodAccessorImpl.invoke(
> NativeMethodAccessorImpl.java:39)
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(
> DelegatingMethodAccessorImpl.java:25)
> at java.lang.reflect.Method.invoke(Method.java:585)
> at org.apache.maven.surefire.junit.JUnitTestSet.execute(
> JUnitTestSet.java:210)
> at
> org.apache.maven.surefire.suite.AbstractDirectoryTestSuite.executeTestSet(
> AbstractDirectoryTestSuite.java:135)
> at
> org.apache.maven.surefire.suite.AbstractDirectoryTestSuite.execute(
> AbstractDirectoryTestSuite.java:160)
> at org.apache.maven.surefire.Surefire.run(Surefire.java:81)
> at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
> at sun.reflect.NativeMethodAccessorImpl.invoke(
> NativeMethodAccessorImpl.java:39)
> at sun.reflect.DelegatingMethodAccessorImpl.invoke(
> DelegatingMethodAccessorImpl.java:25)
> at java.lang.reflect.Method.invoke(Method.java:585)
> at
> org.apache.maven.surefire.booter.SurefireBooter.runSuitesInProcess(
> SurefireBooter.java:182)
> at org.apache.maven.surefire.booter.SurefireBooter.main(
> SurefireBooter.java:743)
>
> I wonder whether or not spring.xml
>
> (container\openejb-core\src\main\resources\META-INF\org.openejb\spring.xml)
> needs to be updated so that the JNDI name is available. It looks to me
> that spring2.xml is used instead and don't know how to get rid of the
> above test failure.
>
> Jacek
>
> --
> Jacek Laskowski
> http://www.laskowski.net.pl
>
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Jacek Laskowski <ja...@laskowski.net.pl>.
On 9/14/06, Jacek Laskowski <ja...@laskowski.net.pl> wrote:
> On 9/14/06, David Blevins <da...@visi.com> wrote:
>
> > I'm rebuilding gbuild today and will be working on moving the svn
> > tomorrow. Is there someone who can look at this patch today?
>
> But of course! ;-) I'm doing it now...
When I run the itests after the patch had been applied, I got a tests failure:
$ less container/openejb-core/target/surefire-reports/org.openejb.SpringAssembler2Test.txt
-------------------------------------------------------------------------------
Test set: org.openejb.SpringAssembler2Test
-------------------------------------------------------------------------------
Tests run: 398, Failures: 0, Errors: 1, Skipped: 0, Time elapsed:
5.796 sec <<< FAILURE!
Stateless.PojoLocalHomeIntfc..setUp() Time elapsed: 0 sec <<< ERROR!
javax.naming.NameNotFoundException: Name
"client/tests/stateless/BasicStatelessPojoLocalHome" not found.
at org.openejb.core.ivm.naming.IvmContext.federate(IvmContext.java:138)
at org.openejb.core.ivm.naming.IvmContext.lookup(IvmContext.java:101)
at javax.naming.InitialContext.lookup(InitialContext.java:351)
at org.openejb.test.stateless.StatelessPojoLocalHomeIntfcTests.setUp(StatelessPojoLocalHomeIntfcTests.java:19)
at org.openejb.test.NumberedTestCase.run(NumberedTestCase.java:77)
at junit.framework.TestSuite.runTest(TestSuite.java:208)
at junit.framework.TestSuite.run(TestSuite.java:203)
at org.openejb.test.TestSuite.run(TestSuite.java:29)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:585)
at org.apache.maven.surefire.junit.JUnitTestSet.execute(JUnitTestSet.java:210)
at org.apache.maven.surefire.suite.AbstractDirectoryTestSuite.executeTestSet(AbstractDirectoryTestSuite.java:135)
at org.apache.maven.surefire.suite.AbstractDirectoryTestSuite.execute(AbstractDirectoryTestSuite.java:160)
at org.apache.maven.surefire.Surefire.run(Surefire.java:81)
at sun.reflect.NativeMethodAccessorImpl.invoke0(Native Method)
at sun.reflect.NativeMethodAccessorImpl.invoke(NativeMethodAccessorImpl.java:39)
at sun.reflect.DelegatingMethodAccessorImpl.invoke(DelegatingMethodAccessorImpl.java:25)
at java.lang.reflect.Method.invoke(Method.java:585)
at org.apache.maven.surefire.booter.SurefireBooter.runSuitesInProcess(SurefireBooter.java:182)
at org.apache.maven.surefire.booter.SurefireBooter.main(SurefireBooter.java:743)
I wonder whether or not spring.xml
(container\openejb-core\src\main\resources\META-INF\org.openejb\spring.xml)
needs to be updated so that the JNDI name is available. It looks to me
that spring2.xml is used instead and don't know how to get rid of the
above test failure.
Jacek
--
Jacek Laskowski
http://www.laskowski.net.pl
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Mohammad Nour El-Din <no...@gmail.com>.
On 9/14/06, Mohammad Nour El-Din <no...@gmail.com> wrote:
>
> Hi Dave...
>
> Sorry for being late to submit patches but I has pressure at work, I
> submitted the patch of OPENEJB-147
> http://jira.codehaus.org/browse/OPENEJB-147, waiting for ur review.
>
>
Use attached patch #1
>
> On 9/3/06, Mohammad Nour El-Din <no...@gmail.com> wrote:
>
> > Hi Dave...
> >
> > On 9/2/06, David Blevins <david.blevins@visi.com > wrote:
> > >
> > > Excellent work Mohammad! I applied an updated version of the patch
> > > which contains the modifications I mentioned before. I went ahead
> > > and assigned you OPENEJB-147 and OPENEJB-172.
> > >
> > > Will take a look at that email you mention.
> > >
> > > It's so great to have your help! Thank you!
> >
> >
> > You are so welcome :)
> >
> > -David
> > >
> > >
> > > On Sep 2, 2006, at 4:32 AM, Mohammad Nour El-Din wrote:
> > >
> > > > Hi Dave...
> > > >
> > > > Sorry for being late to reply, I added the missing files and made
> > > the
> > > > changes you asked for.
> > > > Here is the new patch http://jira.codehaus.org/browse/OPENEJB-146 .
> > > > I will
> > > > work on *OPENEJB-147* & *OPENEJB-172*, please assign them to me.
> > > > One more
> > > > favor, please take a look at this mail
> > > > http://marc.theaimsgroup.com/?l=openejb-
> > > > development&m=115699229501566&w=2 .
> > > >
> > > > Thanks and best regards...
> > > > Mohammad Nour El-Din
> > > >
> > > >
> > > > On 8/31/06, David Blevins < david.blevins@visi.com> wrote:
> > > >
> > > >> Hi Mohammad!
> > > >>
> > > >> On Aug 30, 2006, at 8:12 PM, Mohammad Nour El-Din wrote:
> > > >>
> > > >> > Hi Dave...
> > > >> >
> > > >> > I have submited the patch of OPENEJB-146, please review and
> > > assign
> > > >> > to me
> > > >> > OPENEJB-147 :)
> > > >>
> > > >> Excellent! I reviewed the patch and have a couple of notes:
> > > >>
> > > >> The first is simple, you forgot to 'svn add' the component local
> > > >> interfaces :)
> > > >>
> > > >> Also, go ahead and put the local interfaces on the
> > > >> BasicStatelessPojoBean that we created in the last set of
> > > tests. No
> > > >> need to add inheritance and copy the bean def in the
> > > descriptors. My
> > > >> original note in that jira was actually about splitting the
> > > component
> > > >> local and business local interface tests into two beans, but we
> > > don't
> > > >> need to do that for the component remote and the component local
> > > >> tests.
> > > >>
> > > >> Looking great! You're starting to cook :)
> > > >>
> > > >> -David
> > > >>
> > > >> > Thanks and best regards...
> > > >> > Mohammad Nour El-Din
> > > >> >
> > > >> >
> > > >> > On 8/30/06, Mohammed Nour < nour.mohammad@gmail.com> wrote:
> > > >> >
> > > >> >> Hi Daves...
> > > >> >>
> > > >> >> On 8/30/06, David Blevins < david.blevins@visi.com> wrote:
> > > >> >> >
> > > >> >> > Hi Mohammads... :)
> > > >> >> >
> > > >> >> > Now that's a big sentence! Edgar Allan Poe would be jealous.
> > > >> >> >
> > > >> >> > Comments below.
> > > >> >> >
> > > >> >> > On Aug 29, 2006, at 3:11 PM, Mohammed Nour wrote:
> > > >> >> >
> > > >> >> > > Hi Daves...
> > > >> >> > >
> > > >> >> > > Sure you can assign it to me, it will be my honor and
> > > >> >> pleasure :),
> > > >> >> > > but -
> > > >> >> > > correct me if I am wrong - I have noticed that there is no
> > > >> iTests
> > > >> >> > > for using
> > > >> >> > > local interfaces, so I have some simple re-factoring ideas
> > > >> so we
> > > >> >> > > can make
> > > >> >> > > iTests for using local interfaces, I am working on them and
> > > >> will
> > > >> >> > > send you a
> > > >> >> > > patch, this of caurse will include the re-factored impl of
> > > the
> > > >> >> two
> > > >> >> > > iTests
> > > >> >> > > that you and me did.
> > > >> >> >
> > > >> >> > Ok. I updated OPENEJB-146 "iTest: StatelessLocalJndiTests" so
> > >
> > > >> >> it no
> > > >> >> > longer includes testing the local business interface --
> > > probably
> > > >> >> > should have been that way from the beginning.
> > > >> >> >
> > > >> >> > I'll leave 146 assigned to you until I hear otherwise.
> > > >> >> >
> > > >> >> > The tests OPENEJB-145 and OPENEJB-171 don't need to be
> > > >> updated as
> > > >> >> > they intentionally are only for remote interfaces. Unless
> > > >> when you
> > > >> >> > say "refactor" you mean you simply adding a local and
> > > local-home
> > > >> >> > interfaces to the POJOs we created. That would be fine.
> > > >> >> >
> > > >> >> > > I want to ask you a favor, can I have a JIRA of a
> > > >> >> > > feature to impl beside the iTests, if it is OK with you I
> > > will
> > > >> >> be very
> > > >> >> > > happy, but if u see that I rush to have more tasks so I will
> > > >> >> stick
> > > >> >> > > with the
> > > >> >> > > iTests till I finish them all and then go into new features
> > > >> >> JIRAs.
> > > >> >> >
> > > >> >> > You can work on whatever you think you can do. I personally
> > > >> don't
> > > >> >> > take more than one or two at a time cause I just can't do
> > > >> more than
> > > >> >> > one or two at a time. I'd love to do them all.
> > > >> >>
> > > >> >>
> > > >> >> Thanks for the advice, and I will stick with it :)
> > > >> >>
> > > >> >> -David
> > > >> >> >
> > > >> >> > > Thanks and best regards...
> > > >> >> > > Mohammad Nour El-Din
> > > >> >> > >
> > > >> >> > >
> > > >> >> > > On 8/29/06, David Blevins < david.blevins@visi.com > wrote:
> > > >> >> > >
> > > >> >> > >> Great patch, Mohammad. It applied, built and ran
> > > perfectly.
> > > >> >> > >>
> > > >> >> > >> Regarding OPENEJB-146, as I've said here (http://
> > > >> >> > >> marc.theaimsgroup.com/?l=openejb-
> > > >> >> development&m=115561420703000&w=2)
> > > >> >> > >> doing that one requires a whole bunch of functionality we
> > > >> >> don't have
> > > >> >> > >> yet, so there are probably better jiras to work on.
> > > >> >> > >>
> > > >> >> > >> OPENEJB-147 would be a good one. That will be easier than
> > > >> >> the one
> > > >> >> > >> you just did as you won't even have to create a new bean
> > > and
> > > >> >> add it
> > > >> >> > >> to the various deployment descriptors.
> > > >> >> > >>
> > > >> >> > >> Let me know and I'll assign it to you.
> > > >> >> > >>
> > > >> >> > >> Thanks!
> > > >> >> > >> David
> > > >> >> > >>
> > > >> >> > >>
> > > >> >> > >> On Aug 27, 2006, at 5:12 AM, Mohammed Nour wrote:
> > > >> >> > >>
> > > >> >> > >> > Hi Daves...
> > > >> >> > >> >
> > > >> >> > >> > I attached the new diff file to OPENEJB-171, I will work
> > > on
> > > >> >> > >> > OPENEJB-146 with
> > > >> >> > >> > the same procedure.
> > > >> >> > >> >
> > > >> >> > >> > Thanks and best regards...
> > > >> >> > >> > Mohammad Nour El-Din
> > > >> >> > >> >
> > > >> >> > >> >
> > > >> >> > >> > On 8/27/06, David Blevins < david.blevins@visi.com >
> > > wrote:
> > > >> >> > >> >
> > > >> >> > >> >> On Aug 26, 2006, at 1:31 PM, David Blevins wrote:
> > > >> >> > >> >> > I'll take a look at this and get back to you later
> > > >> today.
> > > >> >> > >> >>
> > > >> >> > >> >> The patch looks good, you're just missing a few files.
> > > >> >> Not a big
> > > >> >> > >> >> deal, you just need to 'svn add' them so svn will
> > > include
> > > >> >> them
> > > >> >> > >> in the
> > > >> >> > >> >> patch. Give that a try and attach a new patch file
> > > here:
> > > >> >> http://
> > > >> >> > >> >> jira.codehaus.org/browse/OPENEJB-171
> > > >> >> > >> >>
> > > >> >> > >> >> > On Aug 26, 2006, at 10:59 AM, Mohammed Nour wrote:
> > > >> >> > >> >> >
> > > >> >> > >> >> >> Hi Daves...
> > > >> >> > >> >> >> I followed the steps you explained to me, but I have
> > > a
> > > >> >> > >> couple of
> > > >> >> > >> >> >> stupid questions :) :
> > > >> >> > >> >> >> I don't see any thing the build output which
> > > indicates
> > > >> >> that the
> > > >> >> > >> >> >> test case was running, why?
> > > >> >> > >> >>
> > > >> >> > >> >> It's just the way the surefire plugin works. It only
> > > >> >> displays the
> > > >> >> > >> >> names of the tests that are in that module and we have
> > > all
> > > >> >> the
> > > >> >> > >> itests
> > > >> >> > >> >> in another module. If it fails though, there will be a
> > > >> >> report
> > > >> >> > >> under
> > > >> >> > >> >> openejb-core/target/surefire-reports/
> > > >> >> > >> >>
> > > >> >> > >> >> >> What is the use of spring.xml, is it for assembling
> > > the
> > > >> >> > >> components
> > > >> >> > >> >> >> of the EJB container? If yes, do I have to read about
> > > >> >> spring to
> > > >> >> >
> > > >> >> > >> >> >> know the details of this XML file ?
> > > >> >> > >> >>
> > > >> >> > >> >> Yes, and it's experimental. You don't need to
> > > understand
> > > >> >> it to
> > > >> >> > >> work
> > > >> >> > >> >> on OpenEJB. You'll need to add OpenEJB deployments to
> > > it
> > > >> >> whenever
> > > >> >> >
> > > >> >> > >> >> you add them to the ejb-jar.xml, but we can help with
> > > that
> > > >> >> if the
> > > >> >> > >> >> need comes up again.
> > > >> >> > >> >>
> > > >> >> > >> >> -David
> > > >> >> > >> >>
> > > >> >> > >> >>
> > > >> >> > >>
> > > >> >> > >>
> > > >> >> >
> > > >> >> >
> > > >> >>
> > > >> >>
> > > >>
> > > >>
> > >
> > >
> >
> >
>
>
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Jacek Laskowski <ja...@laskowski.net.pl>.
On 9/14/06, David Blevins <da...@visi.com> wrote:
> I'm rebuilding gbuild today and will be working on moving the svn
> tomorrow. Is there someone who can look at this patch today?
But of course! ;-) I'm doing it now...
Jacek
--
Jacek Laskowski
http://www.laskowski.net.pl
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by David Blevins <da...@visi.com>.
On Sep 13, 2006, at 4:23 PM, Mohammad Nour El-Din wrote:
> Hi Dave...
>
> Sorry for being late to submit patches but I has pressure at work, I
> submitted the patch of OPENEJB-147
> http://jira.codehaus.org/browse/OPENEJB-147, waiting for ur review.
I'm rebuilding gbuild today and will be working on moving the svn
tomorrow. Is there someone who can look at this patch today?
-David
>
>
> On 9/3/06, Mohammad Nour El-Din <no...@gmail.com> wrote:
>
>> Hi Dave...
>>
>> On 9/2/06, David Blevins <da...@visi.com> wrote:
>> >
>> > Excellent work Mohammad! I applied an updated version of the patch
>> > which contains the modifications I mentioned before. I went ahead
>> > and assigned you OPENEJB-147 and OPENEJB-172.
>> >
>> > Will take a look at that email you mention.
>> >
>> > It's so great to have your help! Thank you!
>>
>>
>> You are so welcome :)
>>
>> -David
>> >
>> >
>> > On Sep 2, 2006, at 4:32 AM, Mohammad Nour El-Din wrote:
>> >
>> > > Hi Dave...
>> > >
>> > > Sorry for being late to reply, I added the missing files and
>> made the
>> > > changes you asked for.
>> > > Here is the new patch http://jira.codehaus.org/browse/
>> OPENEJB-146 .
>> > > I will
>> > > work on *OPENEJB-147* & *OPENEJB-172*, please assign them to me.
>> > > One more
>> > > favor, please take a look at this mail
>> > > http://marc.theaimsgroup.com/?l=openejb-
>> > > development&m=115699229501566&w=2 .
>> > >
>> > > Thanks and best regards...
>> > > Mohammad Nour El-Din
>> > >
>> > >
>> > > On 8/31/06, David Blevins < david.blevins@visi.com> wrote:
>> > >
>> > >> Hi Mohammad!
>> > >>
>> > >> On Aug 30, 2006, at 8:12 PM, Mohammad Nour El-Din wrote:
>> > >>
>> > >> > Hi Dave...
>> > >> >
>> > >> > I have submited the patch of OPENEJB-146, please review and
>> assign
>> > >> > to me
>> > >> > OPENEJB-147 :)
>> > >>
>> > >> Excellent! I reviewed the patch and have a couple of notes:
>> > >>
>> > >> The first is simple, you forgot to 'svn add' the component local
>> > >> interfaces :)
>> > >>
>> > >> Also, go ahead and put the local interfaces on the
>> > >> BasicStatelessPojoBean that we created in the last set of
>> tests. No
>> > >> need to add inheritance and copy the bean def in the
>> descriptors. My
>> > >> original note in that jira was actually about splitting the
>> component
>> > >> local and business local interface tests into two beans, but
>> we don't
>> >
>> > >> need to do that for the component remote and the component local
>> > >> tests.
>> > >>
>> > >> Looking great! You're starting to cook :)
>> > >>
>> > >> -David
>> > >>
>> > >> > Thanks and best regards...
>> > >> > Mohammad Nour El-Din
>> > >> >
>> > >> >
>> > >> > On 8/30/06, Mohammed Nour < nour.mohammad@gmail.com> wrote:
>> > >> >
>> > >> >> Hi Daves...
>> > >> >>
>> > >> >> On 8/30/06, David Blevins <da...@visi.com> wrote:
>> > >> >> >
>> > >> >> > Hi Mohammads... :)
>> > >> >> >
>> > >> >> > Now that's a big sentence! Edgar Allan Poe would be
>> jealous.
>> > >> >> >
>> > >> >> > Comments below.
>> > >> >> >
>> > >> >> > On Aug 29, 2006, at 3:11 PM, Mohammed Nour wrote:
>> > >> >> >
>> > >> >> > > Hi Daves...
>> > >> >> > >
>> > >> >> > > Sure you can assign it to me, it will be my honor and
>> > >> >> pleasure :),
>> > >> >> > > but -
>> > >> >> > > correct me if I am wrong - I have noticed that there
>> is no
>> > >> iTests
>> > >> >> > > for using
>> > >> >> > > local interfaces, so I have some simple re-factoring
>> ideas
>> > >> so we
>> > >> >> > > can make
>> > >> >> > > iTests for using local interfaces, I am working on
>> them and
>> > >> will
>> > >> >> > > send you a
>> > >> >> > > patch, this of caurse will include the re-factored
>> impl of the
>> >
>> > >> >> two
>> > >> >> > > iTests
>> > >> >> > > that you and me did.
>> > >> >> >
>> > >> >> > Ok. I updated OPENEJB-146 "iTest:
>> StatelessLocalJndiTests" so
>> > >> >> it no
>> > >> >> > longer includes testing the local business interface --
>> probably
>> > >> >> > should have been that way from the beginning.
>> > >> >> >
>> > >> >> > I'll leave 146 assigned to you until I hear otherwise.
>> > >> >> >
>> > >> >> > The tests OPENEJB-145 and OPENEJB-171 don't need to be
>> > >> updated as
>> > >> >> > they intentionally are only for remote interfaces. Unless
>> > >> when you
>> > >> >> > say "refactor" you mean you simply adding a local and
>> local-home
>> > >> >> > interfaces to the POJOs we created. That would be fine.
>> > >> >> >
>> > >> >> > > I want to ask you a favor, can I have a JIRA of a
>> > >> >> > > feature to impl beside the iTests, if it is OK with
>> you I will
>> > >> >> be very
>> > >> >> > > happy, but if u see that I rush to have more tasks so
>> I will
>> > >> >> stick
>> > >> >> > > with the
>> > >> >> > > iTests till I finish them all and then go into new
>> features
>> > >> >> JIRAs.
>> > >> >> >
>> > >> >> > You can work on whatever you think you can do. I
>> personally
>> > >> don't
>> > >> >> > take more than one or two at a time cause I just can't do
>> > >> more than
>> > >> >> > one or two at a time. I'd love to do them all.
>> > >> >>
>> > >> >>
>> > >> >> Thanks for the advice, and I will stick with it :)
>> > >> >>
>> > >> >> -David
>> > >> >> >
>> > >> >> > > Thanks and best regards...
>> > >> >> > > Mohammad Nour El-Din
>> > >> >> > >
>> > >> >> > >
>> > >> >> > > On 8/29/06, David Blevins < david.blevins@visi.com >
>> wrote:
>> > >> >> > >
>> > >> >> > >> Great patch, Mohammad. It applied, built and ran
>> perfectly.
>> > >> >> > >>
>> > >> >> > >> Regarding OPENEJB-146, as I've said here (http://
>> > >> >> > >> marc.theaimsgroup.com/?l=openejb-
>> > >> >> development&m=115561420703000&w=2)
>> > >> >> > >> doing that one requires a whole bunch of
>> functionality we
>> > >> >> don't have
>> > >> >> > >> yet, so there are probably better jiras to work on.
>> > >> >> > >>
>> > >> >> > >> OPENEJB-147 would be a good one. That will be easier
>> than
>> > >> >> the one
>> > >> >> > >> you just did as you won't even have to create a new
>> bean and
>> > >> >> add it
>> > >> >> > >> to the various deployment descriptors.
>> > >> >> > >>
>> > >> >> > >> Let me know and I'll assign it to you.
>> > >> >> > >>
>> > >> >> > >> Thanks!
>> > >> >> > >> David
>> > >> >> > >>
>> > >> >> > >>
>> > >> >> > >> On Aug 27, 2006, at 5:12 AM, Mohammed Nour wrote:
>> > >> >> > >>
>> > >> >> > >> > Hi Daves...
>> > >> >> > >> >
>> > >> >> > >> > I attached the new diff file to OPENEJB-171, I will
>> work on
>> > >> >> > >> > OPENEJB-146 with
>> > >> >> > >> > the same procedure.
>> > >> >> > >> >
>> > >> >> > >> > Thanks and best regards...
>> > >> >> > >> > Mohammad Nour El-Din
>> > >> >> > >> >
>> > >> >> > >> >
>> > >> >> > >> > On 8/27/06, David Blevins <david.blevins@visi.com >
>> wrote:
>> > >> >> > >> >
>> > >> >> > >> >> On Aug 26, 2006, at 1:31 PM, David Blevins wrote:
>> > >> >> > >> >> > I'll take a look at this and get back to you later
>> > >> today.
>> > >> >> > >> >>
>> > >> >> > >> >> The patch looks good, you're just missing a few
>> files.
>> > >> >> Not a big
>> > >> >> > >> >> deal, you just need to 'svn add' them so svn will
>> include
>> > >> >> them
>> > >> >> > >> in the
>> > >> >> > >> >> patch. Give that a try and attach a new patch
>> file here:
>> > >> >> http://
>> > >> >> > >> >> jira.codehaus.org/browse/OPENEJB-171
>> > >> >> > >> >>
>> > >> >> > >> >> > On Aug 26, 2006, at 10:59 AM, Mohammed Nour wrote:
>> > >> >> > >> >> >
>> > >> >> > >> >> >> Hi Daves...
>> > >> >> > >> >> >> I followed the steps you explained to me, but I
>> have a
>> > >> >> > >> couple of
>> > >> >> > >> >> >> stupid questions :) :
>> > >> >> > >> >> >> I don't see any thing the build output which
>> indicates
>> > >> >> that the
>> > >> >> > >> >> >> test case was running, why?
>> > >> >> > >> >>
>> > >> >> > >> >> It's just the way the surefire plugin works. It only
>> > >> >> displays the
>> > >> >> > >> >> names of the tests that are in that module and we
>> have all
>> > >> >> the
>> > >> >> > >> itests
>> > >> >> > >> >> in another module. If it fails though, there will
>> be a
>> > >> >> report
>> > >> >> > >> under
>> > >> >> > >> >> openejb-core/target/surefire-reports/
>> > >> >> > >> >>
>> > >> >> > >> >> >> What is the use of spring.xml, is it for
>> assembling the
>> > >> >> > >> components
>> > >> >> > >> >> >> of the EJB container? If yes, do I have to read
>> about
>> > >> >> spring to
>> > >> >> >
>> > >> >> > >> >> >> know the details of this XML file ?
>> > >> >> > >> >>
>> > >> >> > >> >> Yes, and it's experimental. You don't need to
>> understand
>> > >> >> it to
>> > >> >> > >> work
>> > >> >> > >> >> on OpenEJB. You'll need to add OpenEJB
>> deployments to it
>> > >> >> whenever
>> > >> >> >
>> > >> >> > >> >> you add them to the ejb-jar.xml, but we can help
>> with that
>> > >> >> if the
>> > >> >> > >> >> need comes up again.
>> > >> >> > >> >>
>> > >> >> > >> >> -David
>> > >> >> > >> >>
>> > >> >> > >> >>
>> > >> >> > >>
>> > >> >> > >>
>> > >> >> >
>> > >> >> >
>> > >> >>
>> > >> >>
>> > >>
>> > >>
>> >
>> >
>>
>>
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Mohammad Nour El-Din <no...@gmail.com>.
Hi Dave...
Sorry for being late to submit patches but I has pressure at work, I
submitted the patch of OPENEJB-147
http://jira.codehaus.org/browse/OPENEJB-147, waiting for ur review.
On 9/3/06, Mohammad Nour El-Din <no...@gmail.com> wrote:
> Hi Dave...
>
> On 9/2/06, David Blevins <da...@visi.com> wrote:
> >
> > Excellent work Mohammad! I applied an updated version of the patch
> > which contains the modifications I mentioned before. I went ahead
> > and assigned you OPENEJB-147 and OPENEJB-172.
> >
> > Will take a look at that email you mention.
> >
> > It's so great to have your help! Thank you!
>
>
> You are so welcome :)
>
> -David
> >
> >
> > On Sep 2, 2006, at 4:32 AM, Mohammad Nour El-Din wrote:
> >
> > > Hi Dave...
> > >
> > > Sorry for being late to reply, I added the missing files and made the
> > > changes you asked for.
> > > Here is the new patch http://jira.codehaus.org/browse/OPENEJB-146 .
> > > I will
> > > work on *OPENEJB-147* & *OPENEJB-172*, please assign them to me.
> > > One more
> > > favor, please take a look at this mail
> > > http://marc.theaimsgroup.com/?l=openejb-
> > > development&m=115699229501566&w=2 .
> > >
> > > Thanks and best regards...
> > > Mohammad Nour El-Din
> > >
> > >
> > > On 8/31/06, David Blevins < david.blevins@visi.com> wrote:
> > >
> > >> Hi Mohammad!
> > >>
> > >> On Aug 30, 2006, at 8:12 PM, Mohammad Nour El-Din wrote:
> > >>
> > >> > Hi Dave...
> > >> >
> > >> > I have submited the patch of OPENEJB-146, please review and assign
> > >> > to me
> > >> > OPENEJB-147 :)
> > >>
> > >> Excellent! I reviewed the patch and have a couple of notes:
> > >>
> > >> The first is simple, you forgot to 'svn add' the component local
> > >> interfaces :)
> > >>
> > >> Also, go ahead and put the local interfaces on the
> > >> BasicStatelessPojoBean that we created in the last set of tests. No
> > >> need to add inheritance and copy the bean def in the descriptors. My
> > >> original note in that jira was actually about splitting the component
> > >> local and business local interface tests into two beans, but we don't
> >
> > >> need to do that for the component remote and the component local
> > >> tests.
> > >>
> > >> Looking great! You're starting to cook :)
> > >>
> > >> -David
> > >>
> > >> > Thanks and best regards...
> > >> > Mohammad Nour El-Din
> > >> >
> > >> >
> > >> > On 8/30/06, Mohammed Nour < nour.mohammad@gmail.com> wrote:
> > >> >
> > >> >> Hi Daves...
> > >> >>
> > >> >> On 8/30/06, David Blevins <da...@visi.com> wrote:
> > >> >> >
> > >> >> > Hi Mohammads... :)
> > >> >> >
> > >> >> > Now that's a big sentence! Edgar Allan Poe would be jealous.
> > >> >> >
> > >> >> > Comments below.
> > >> >> >
> > >> >> > On Aug 29, 2006, at 3:11 PM, Mohammed Nour wrote:
> > >> >> >
> > >> >> > > Hi Daves...
> > >> >> > >
> > >> >> > > Sure you can assign it to me, it will be my honor and
> > >> >> pleasure :),
> > >> >> > > but -
> > >> >> > > correct me if I am wrong - I have noticed that there is no
> > >> iTests
> > >> >> > > for using
> > >> >> > > local interfaces, so I have some simple re-factoring ideas
> > >> so we
> > >> >> > > can make
> > >> >> > > iTests for using local interfaces, I am working on them and
> > >> will
> > >> >> > > send you a
> > >> >> > > patch, this of caurse will include the re-factored impl of the
> >
> > >> >> two
> > >> >> > > iTests
> > >> >> > > that you and me did.
> > >> >> >
> > >> >> > Ok. I updated OPENEJB-146 "iTest: StatelessLocalJndiTests" so
> > >> >> it no
> > >> >> > longer includes testing the local business interface -- probably
> > >> >> > should have been that way from the beginning.
> > >> >> >
> > >> >> > I'll leave 146 assigned to you until I hear otherwise.
> > >> >> >
> > >> >> > The tests OPENEJB-145 and OPENEJB-171 don't need to be
> > >> updated as
> > >> >> > they intentionally are only for remote interfaces. Unless
> > >> when you
> > >> >> > say "refactor" you mean you simply adding a local and local-home
> > >> >> > interfaces to the POJOs we created. That would be fine.
> > >> >> >
> > >> >> > > I want to ask you a favor, can I have a JIRA of a
> > >> >> > > feature to impl beside the iTests, if it is OK with you I will
> > >> >> be very
> > >> >> > > happy, but if u see that I rush to have more tasks so I will
> > >> >> stick
> > >> >> > > with the
> > >> >> > > iTests till I finish them all and then go into new features
> > >> >> JIRAs.
> > >> >> >
> > >> >> > You can work on whatever you think you can do. I personally
> > >> don't
> > >> >> > take more than one or two at a time cause I just can't do
> > >> more than
> > >> >> > one or two at a time. I'd love to do them all.
> > >> >>
> > >> >>
> > >> >> Thanks for the advice, and I will stick with it :)
> > >> >>
> > >> >> -David
> > >> >> >
> > >> >> > > Thanks and best regards...
> > >> >> > > Mohammad Nour El-Din
> > >> >> > >
> > >> >> > >
> > >> >> > > On 8/29/06, David Blevins < david.blevins@visi.com > wrote:
> > >> >> > >
> > >> >> > >> Great patch, Mohammad. It applied, built and ran perfectly.
> > >> >> > >>
> > >> >> > >> Regarding OPENEJB-146, as I've said here (http://
> > >> >> > >> marc.theaimsgroup.com/?l=openejb-
> > >> >> development&m=115561420703000&w=2)
> > >> >> > >> doing that one requires a whole bunch of functionality we
> > >> >> don't have
> > >> >> > >> yet, so there are probably better jiras to work on.
> > >> >> > >>
> > >> >> > >> OPENEJB-147 would be a good one. That will be easier than
> > >> >> the one
> > >> >> > >> you just did as you won't even have to create a new bean and
> > >> >> add it
> > >> >> > >> to the various deployment descriptors.
> > >> >> > >>
> > >> >> > >> Let me know and I'll assign it to you.
> > >> >> > >>
> > >> >> > >> Thanks!
> > >> >> > >> David
> > >> >> > >>
> > >> >> > >>
> > >> >> > >> On Aug 27, 2006, at 5:12 AM, Mohammed Nour wrote:
> > >> >> > >>
> > >> >> > >> > Hi Daves...
> > >> >> > >> >
> > >> >> > >> > I attached the new diff file to OPENEJB-171, I will work on
> > >> >> > >> > OPENEJB-146 with
> > >> >> > >> > the same procedure.
> > >> >> > >> >
> > >> >> > >> > Thanks and best regards...
> > >> >> > >> > Mohammad Nour El-Din
> > >> >> > >> >
> > >> >> > >> >
> > >> >> > >> > On 8/27/06, David Blevins <david.blevins@visi.com > wrote:
> > >> >> > >> >
> > >> >> > >> >> On Aug 26, 2006, at 1:31 PM, David Blevins wrote:
> > >> >> > >> >> > I'll take a look at this and get back to you later
> > >> today.
> > >> >> > >> >>
> > >> >> > >> >> The patch looks good, you're just missing a few files.
> > >> >> Not a big
> > >> >> > >> >> deal, you just need to 'svn add' them so svn will include
> > >> >> them
> > >> >> > >> in the
> > >> >> > >> >> patch. Give that a try and attach a new patch file here:
> > >> >> http://
> > >> >> > >> >> jira.codehaus.org/browse/OPENEJB-171
> > >> >> > >> >>
> > >> >> > >> >> > On Aug 26, 2006, at 10:59 AM, Mohammed Nour wrote:
> > >> >> > >> >> >
> > >> >> > >> >> >> Hi Daves...
> > >> >> > >> >> >> I followed the steps you explained to me, but I have a
> > >> >> > >> couple of
> > >> >> > >> >> >> stupid questions :) :
> > >> >> > >> >> >> I don't see any thing the build output which indicates
> > >> >> that the
> > >> >> > >> >> >> test case was running, why?
> > >> >> > >> >>
> > >> >> > >> >> It's just the way the surefire plugin works. It only
> > >> >> displays the
> > >> >> > >> >> names of the tests that are in that module and we have all
> > >> >> the
> > >> >> > >> itests
> > >> >> > >> >> in another module. If it fails though, there will be a
> > >> >> report
> > >> >> > >> under
> > >> >> > >> >> openejb-core/target/surefire-reports/
> > >> >> > >> >>
> > >> >> > >> >> >> What is the use of spring.xml, is it for assembling the
> > >> >> > >> components
> > >> >> > >> >> >> of the EJB container? If yes, do I have to read about
> > >> >> spring to
> > >> >> >
> > >> >> > >> >> >> know the details of this XML file ?
> > >> >> > >> >>
> > >> >> > >> >> Yes, and it's experimental. You don't need to understand
> > >> >> it to
> > >> >> > >> work
> > >> >> > >> >> on OpenEJB. You'll need to add OpenEJB deployments to it
> > >> >> whenever
> > >> >> >
> > >> >> > >> >> you add them to the ejb-jar.xml, but we can help with that
> > >> >> if the
> > >> >> > >> >> need comes up again.
> > >> >> > >> >>
> > >> >> > >> >> -David
> > >> >> > >> >>
> > >> >> > >> >>
> > >> >> > >>
> > >> >> > >>
> > >> >> >
> > >> >> >
> > >> >>
> > >> >>
> > >>
> > >>
> >
> >
>
>
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by Mohammad Nour El-Din <no...@gmail.com>.
Hi Dave...
On 9/2/06, David Blevins <da...@visi.com> wrote:
>
> Excellent work Mohammad! I applied an updated version of the patch
> which contains the modifications I mentioned before. I went ahead
> and assigned you OPENEJB-147 and OPENEJB-172.
>
> Will take a look at that email you mention.
>
> It's so great to have your help! Thank you!
You are so welcome :)
-David
>
>
> On Sep 2, 2006, at 4:32 AM, Mohammad Nour El-Din wrote:
>
> > Hi Dave...
> >
> > Sorry for being late to reply, I added the missing files and made the
> > changes you asked for.
> > Here is the new patch http://jira.codehaus.org/browse/OPENEJB-146.
> > I will
> > work on *OPENEJB-147* & *OPENEJB-172*, please assign them to me.
> > One more
> > favor, please take a look at this mail
> > http://marc.theaimsgroup.com/?l=openejb-
> > development&m=115699229501566&w=2 .
> >
> > Thanks and best regards...
> > Mohammad Nour El-Din
> >
> >
> > On 8/31/06, David Blevins <da...@visi.com> wrote:
> >
> >> Hi Mohammad!
> >>
> >> On Aug 30, 2006, at 8:12 PM, Mohammad Nour El-Din wrote:
> >>
> >> > Hi Dave...
> >> >
> >> > I have submited the patch of OPENEJB-146, please review and assign
> >> > to me
> >> > OPENEJB-147 :)
> >>
> >> Excellent! I reviewed the patch and have a couple of notes:
> >>
> >> The first is simple, you forgot to 'svn add' the component local
> >> interfaces :)
> >>
> >> Also, go ahead and put the local interfaces on the
> >> BasicStatelessPojoBean that we created in the last set of tests. No
> >> need to add inheritance and copy the bean def in the descriptors. My
> >> original note in that jira was actually about splitting the component
> >> local and business local interface tests into two beans, but we don't
> >> need to do that for the component remote and the component local
> >> tests.
> >>
> >> Looking great! You're starting to cook :)
> >>
> >> -David
> >>
> >> > Thanks and best regards...
> >> > Mohammad Nour El-Din
> >> >
> >> >
> >> > On 8/30/06, Mohammed Nour < nour.mohammad@gmail.com> wrote:
> >> >
> >> >> Hi Daves...
> >> >>
> >> >> On 8/30/06, David Blevins <da...@visi.com> wrote:
> >> >> >
> >> >> > Hi Mohammads... :)
> >> >> >
> >> >> > Now that's a big sentence! Edgar Allan Poe would be jealous.
> >> >> >
> >> >> > Comments below.
> >> >> >
> >> >> > On Aug 29, 2006, at 3:11 PM, Mohammed Nour wrote:
> >> >> >
> >> >> > > Hi Daves...
> >> >> > >
> >> >> > > Sure you can assign it to me, it will be my honor and
> >> >> pleasure :),
> >> >> > > but -
> >> >> > > correct me if I am wrong - I have noticed that there is no
> >> iTests
> >> >> > > for using
> >> >> > > local interfaces, so I have some simple re-factoring ideas
> >> so we
> >> >> > > can make
> >> >> > > iTests for using local interfaces, I am working on them and
> >> will
> >> >> > > send you a
> >> >> > > patch, this of caurse will include the re-factored impl of the
> >> >> two
> >> >> > > iTests
> >> >> > > that you and me did.
> >> >> >
> >> >> > Ok. I updated OPENEJB-146 "iTest: StatelessLocalJndiTests" so
> >> >> it no
> >> >> > longer includes testing the local business interface -- probably
> >> >> > should have been that way from the beginning.
> >> >> >
> >> >> > I'll leave 146 assigned to you until I hear otherwise.
> >> >> >
> >> >> > The tests OPENEJB-145 and OPENEJB-171 don't need to be
> >> updated as
> >> >> > they intentionally are only for remote interfaces. Unless
> >> when you
> >> >> > say "refactor" you mean you simply adding a local and local-home
> >> >> > interfaces to the POJOs we created. That would be fine.
> >> >> >
> >> >> > > I want to ask you a favor, can I have a JIRA of a
> >> >> > > feature to impl beside the iTests, if it is OK with you I will
> >> >> be very
> >> >> > > happy, but if u see that I rush to have more tasks so I will
> >> >> stick
> >> >> > > with the
> >> >> > > iTests till I finish them all and then go into new features
> >> >> JIRAs.
> >> >> >
> >> >> > You can work on whatever you think you can do. I personally
> >> don't
> >> >> > take more than one or two at a time cause I just can't do
> >> more than
> >> >> > one or two at a time. I'd love to do them all.
> >> >>
> >> >>
> >> >> Thanks for the advice, and I will stick with it :)
> >> >>
> >> >> -David
> >> >> >
> >> >> > > Thanks and best regards...
> >> >> > > Mohammad Nour El-Din
> >> >> > >
> >> >> > >
> >> >> > > On 8/29/06, David Blevins < david.blevins@visi.com> wrote:
> >> >> > >
> >> >> > >> Great patch, Mohammad. It applied, built and ran perfectly.
> >> >> > >>
> >> >> > >> Regarding OPENEJB-146, as I've said here (http://
> >> >> > >> marc.theaimsgroup.com/?l=openejb-
> >> >> development&m=115561420703000&w=2)
> >> >> > >> doing that one requires a whole bunch of functionality we
> >> >> don't have
> >> >> > >> yet, so there are probably better jiras to work on.
> >> >> > >>
> >> >> > >> OPENEJB-147 would be a good one. That will be easier than
> >> >> the one
> >> >> > >> you just did as you won't even have to create a new bean and
> >> >> add it
> >> >> > >> to the various deployment descriptors.
> >> >> > >>
> >> >> > >> Let me know and I'll assign it to you.
> >> >> > >>
> >> >> > >> Thanks!
> >> >> > >> David
> >> >> > >>
> >> >> > >>
> >> >> > >> On Aug 27, 2006, at 5:12 AM, Mohammed Nour wrote:
> >> >> > >>
> >> >> > >> > Hi Daves...
> >> >> > >> >
> >> >> > >> > I attached the new diff file to OPENEJB-171, I will work on
> >> >> > >> > OPENEJB-146 with
> >> >> > >> > the same procedure.
> >> >> > >> >
> >> >> > >> > Thanks and best regards...
> >> >> > >> > Mohammad Nour El-Din
> >> >> > >> >
> >> >> > >> >
> >> >> > >> > On 8/27/06, David Blevins <david.blevins@visi.com > wrote:
> >> >> > >> >
> >> >> > >> >> On Aug 26, 2006, at 1:31 PM, David Blevins wrote:
> >> >> > >> >> > I'll take a look at this and get back to you later
> >> today.
> >> >> > >> >>
> >> >> > >> >> The patch looks good, you're just missing a few files.
> >> >> Not a big
> >> >> > >> >> deal, you just need to 'svn add' them so svn will include
> >> >> them
> >> >> > >> in the
> >> >> > >> >> patch. Give that a try and attach a new patch file here:
> >> >> http://
> >> >> > >> >> jira.codehaus.org/browse/OPENEJB-171
> >> >> > >> >>
> >> >> > >> >> > On Aug 26, 2006, at 10:59 AM, Mohammed Nour wrote:
> >> >> > >> >> >
> >> >> > >> >> >> Hi Daves...
> >> >> > >> >> >> I followed the steps you explained to me, but I have a
> >> >> > >> couple of
> >> >> > >> >> >> stupid questions :) :
> >> >> > >> >> >> I don't see any thing the build output which indicates
> >> >> that the
> >> >> > >> >> >> test case was running, why?
> >> >> > >> >>
> >> >> > >> >> It's just the way the surefire plugin works. It only
> >> >> displays the
> >> >> > >> >> names of the tests that are in that module and we have all
> >> >> the
> >> >> > >> itests
> >> >> > >> >> in another module. If it fails though, there will be a
> >> >> report
> >> >> > >> under
> >> >> > >> >> openejb-core/target/surefire-reports/
> >> >> > >> >>
> >> >> > >> >> >> What is the use of spring.xml, is it for assembling the
> >> >> > >> components
> >> >> > >> >> >> of the EJB container? If yes, do I have to read about
> >> >> spring to
> >> >> >
> >> >> > >> >> >> know the details of this XML file ?
> >> >> > >> >>
> >> >> > >> >> Yes, and it's experimental. You don't need to understand
> >> >> it to
> >> >> > >> work
> >> >> > >> >> on OpenEJB. You'll need to add OpenEJB deployments to it
> >> >> whenever
> >> >> >
> >> >> > >> >> you add them to the ejb-jar.xml, but we can help with that
> >> >> if the
> >> >> > >> >> need comes up again.
> >> >> > >> >>
> >> >> > >> >> -David
> >> >> > >> >>
> >> >> > >> >>
> >> >> > >>
> >> >> > >>
> >> >> >
> >> >> >
> >> >>
> >> >>
> >>
> >>
>
>
Re: [PTACH] - OPENEJB-171 - Re: Component interface iTests (was: Re: POJO Style-beans)
Posted by David Blevins <da...@visi.com>.
Excellent work Mohammad! I applied an updated version of the patch
which contains the modifications I mentioned before. I went ahead
and assigned you OPENEJB-147 and OPENEJB-172.
Will take a look at that email you mention.
It's so great to have your help! Thank you!
-David
On Sep 2, 2006, at 4:32 AM, Mohammad Nour El-Din wrote:
> Hi Dave...
>
> Sorry for being late to reply, I added the missing files and made the
> changes you asked for.
> Here is the new patch http://jira.codehaus.org/browse/OPENEJB-146.
> I will
> work on *OPENEJB-147* & *OPENEJB-172*, please assign them to me.
> One more
> favor, please take a look at this mail
> http://marc.theaimsgroup.com/?l=openejb-
> development&m=115699229501566&w=2 .
>
> Thanks and best regards...
> Mohammad Nour El-Din
>
>
> On 8/31/06, David Blevins <da...@visi.com> wrote:
>
>> Hi Mohammad!
>>
>> On Aug 30, 2006, at 8:12 PM, Mohammad Nour El-Din wrote:
>>
>> > Hi Dave...
>> >
>> > I have submited the patch of OPENEJB-146, please review and assign
>> > to me
>> > OPENEJB-147 :)
>>
>> Excellent! I reviewed the patch and have a couple of notes:
>>
>> The first is simple, you forgot to 'svn add' the component local
>> interfaces :)
>>
>> Also, go ahead and put the local interfaces on the
>> BasicStatelessPojoBean that we created in the last set of tests. No
>> need to add inheritance and copy the bean def in the descriptors. My
>> original note in that jira was actually about splitting the component
>> local and business local interface tests into two beans, but we don't
>> need to do that for the component remote and the component local
>> tests.
>>
>> Looking great! You're starting to cook :)
>>
>> -David
>>
>> > Thanks and best regards...
>> > Mohammad Nour El-Din
>> >
>> >
>> > On 8/30/06, Mohammed Nour < nour.mohammad@gmail.com> wrote:
>> >
>> >> Hi Daves...
>> >>
>> >> On 8/30/06, David Blevins <da...@visi.com> wrote:
>> >> >
>> >> > Hi Mohammads... :)
>> >> >
>> >> > Now that's a big sentence! Edgar Allan Poe would be jealous.
>> >> >
>> >> > Comments below.
>> >> >
>> >> > On Aug 29, 2006, at 3:11 PM, Mohammed Nour wrote:
>> >> >
>> >> > > Hi Daves...
>> >> > >
>> >> > > Sure you can assign it to me, it will be my honor and
>> >> pleasure :),
>> >> > > but -
>> >> > > correct me if I am wrong - I have noticed that there is no
>> iTests
>> >> > > for using
>> >> > > local interfaces, so I have some simple re-factoring ideas
>> so we
>> >> > > can make
>> >> > > iTests for using local interfaces, I am working on them and
>> will
>> >> > > send you a
>> >> > > patch, this of caurse will include the re-factored impl of the
>> >> two
>> >> > > iTests
>> >> > > that you and me did.
>> >> >
>> >> > Ok. I updated OPENEJB-146 "iTest: StatelessLocalJndiTests" so
>> >> it no
>> >> > longer includes testing the local business interface -- probably
>> >> > should have been that way from the beginning.
>> >> >
>> >> > I'll leave 146 assigned to you until I hear otherwise.
>> >> >
>> >> > The tests OPENEJB-145 and OPENEJB-171 don't need to be
>> updated as
>> >> > they intentionally are only for remote interfaces. Unless
>> when you
>> >> > say "refactor" you mean you simply adding a local and local-home
>> >> > interfaces to the POJOs we created. That would be fine.
>> >> >
>> >> > > I want to ask you a favor, can I have a JIRA of a
>> >> > > feature to impl beside the iTests, if it is OK with you I will
>> >> be very
>> >> > > happy, but if u see that I rush to have more tasks so I will
>> >> stick
>> >> > > with the
>> >> > > iTests till I finish them all and then go into new features
>> >> JIRAs.
>> >> >
>> >> > You can work on whatever you think you can do. I personally
>> don't
>> >> > take more than one or two at a time cause I just can't do
>> more than
>> >> > one or two at a time. I'd love to do them all.
>> >>
>> >>
>> >> Thanks for the advice, and I will stick with it :)
>> >>
>> >> -David
>> >> >
>> >> > > Thanks and best regards...
>> >> > > Mohammad Nour El-Din
>> >> > >
>> >> > >
>> >> > > On 8/29/06, David Blevins < david.blevins@visi.com> wrote:
>> >> > >
>> >> > >> Great patch, Mohammad. It applied, built and ran perfectly.
>> >> > >>
>> >> > >> Regarding OPENEJB-146, as I've said here (http://
>> >> > >> marc.theaimsgroup.com/?l=openejb-
>> >> development&m=115561420703000&w=2)
>> >> > >> doing that one requires a whole bunch of functionality we
>> >> don't have
>> >> > >> yet, so there are probably better jiras to work on.
>> >> > >>
>> >> > >> OPENEJB-147 would be a good one. That will be easier than
>> >> the one
>> >> > >> you just did as you won't even have to create a new bean and
>> >> add it
>> >> > >> to the various deployment descriptors.
>> >> > >>
>> >> > >> Let me know and I'll assign it to you.
>> >> > >>
>> >> > >> Thanks!
>> >> > >> David
>> >> > >>
>> >> > >>
>> >> > >> On Aug 27, 2006, at 5:12 AM, Mohammed Nour wrote:
>> >> > >>
>> >> > >> > Hi Daves...
>> >> > >> >
>> >> > >> > I attached the new diff file to OPENEJB-171, I will work on
>> >> > >> > OPENEJB-146 with
>> >> > >> > the same procedure.
>> >> > >> >
>> >> > >> > Thanks and best regards...
>> >> > >> > Mohammad Nour El-Din
>> >> > >> >
>> >> > >> >
>> >> > >> > On 8/27/06, David Blevins <david.blevins@visi.com > wrote:
>> >> > >> >
>> >> > >> >> On Aug 26, 2006, at 1:31 PM, David Blevins wrote:
>> >> > >> >> > I'll take a look at this and get back to you later
>> today.
>> >> > >> >>
>> >> > >> >> The patch looks good, you're just missing a few files.
>> >> Not a big
>> >> > >> >> deal, you just need to 'svn add' them so svn will include
>> >> them
>> >> > >> in the
>> >> > >> >> patch. Give that a try and attach a new patch file here:
>> >> http://
>> >> > >> >> jira.codehaus.org/browse/OPENEJB-171
>> >> > >> >>
>> >> > >> >> > On Aug 26, 2006, at 10:59 AM, Mohammed Nour wrote:
>> >> > >> >> >
>> >> > >> >> >> Hi Daves...
>> >> > >> >> >> I followed the steps you explained to me, but I have a
>> >> > >> couple of
>> >> > >> >> >> stupid questions :) :
>> >> > >> >> >> I don't see any thing the build output which indicates
>> >> that the
>> >> > >> >> >> test case was running, why?
>> >> > >> >>
>> >> > >> >> It's just the way the surefire plugin works. It only
>> >> displays the
>> >> > >> >> names of the tests that are in that module and we have all
>> >> the
>> >> > >> itests
>> >> > >> >> in another module. If it fails though, there will be a
>> >> report
>> >> > >> under
>> >> > >> >> openejb-core/target/surefire-reports/
>> >> > >> >>
>> >> > >> >> >> What is the use of spring.xml, is it for assembling the
>> >> > >> components
>> >> > >> >> >> of the EJB container? If yes, do I have to read about
>> >> spring to
>> >> >
>> >> > >> >> >> know the details of this XML file ?
>> >> > >> >>
>> >> > >> >> Yes, and it's experimental. You don't need to understand
>> >> it to
>> >> > >> work
>> >> > >> >> on OpenEJB. You'll need to add OpenEJB deployments to it
>> >> whenever
>> >> >
>> >> > >> >> you add them to the ejb-jar.xml, but we can help with that
>> >> if the
>> >> > >> >> need comes up again.
>> >> > >> >>
>> >> > >> >> -David
>> >> > >> >>
>> >> > >> >>
>> >> > >>
>> >> > >>
>> >> >
>> >> >
>> >>
>> >>
>>
>>